fix: reverted "archive-doc-gen" changes

Author: Kingshuk-Microsoft

archive-doc-gen/src/backend/settings.py

@@ -267,10 +267,6 @@ class _AzureSearchSettings(BaseSettings, DatasourcePayloadConstructor):
         env_ignore_empty=True,
     )
 
-    def __init__(self, *args, settings: "_AppSettings", **data):
-        # Ensure DatasourcePayloadConstructor.__init__ runs so that _settings is initialized
-        super().__init__(*args, settings=settings, **data)
-
     _type: Literal["azure_search"] = PrivateAttr(default="azure_search")
     top_k: int = Field(default=5, serialization_alias="top_n_documents")
     strictness: int = 3

archive-doc-gen/src/frontend/src/components/ChatHistory/ChatHistoryListItem.tsx

@@ -7,6 +7,7 @@ import {
   DialogType,
   IconButton,
   ITextField,
+  ITooltipHostStyles,
   List,
   PrimaryButton,
   Separator,

archive-doc-gen/src/frontend/src/components/ChatHistory/chatHistoryListItem.test.tsx

@@ -553,7 +553,8 @@ describe('ChatHistoryListItemCell', () => {
   }, 10000)
 
   test('shows error when trying to rename to an existing title', async () => {
-    (historyRename as jest.Mock).mockResolvedValueOnce({
+    const existingTitle = 'Existing Chat Title'
+    ;(historyRename as jest.Mock).mockResolvedValueOnce({
       ok: false,
       json: async () => ({ message: 'Title already exists' })
     })

archive-doc-gen/src/frontend/src/pages/chat/Chat.tsx

@@ -6,6 +6,9 @@ import {
   Dialog,
   DialogType,
   Stack,
+  IStackTokens,
+  mergeStyleSets,
+  IModalStyles,
   Spinner,
   SpinnerSize
 } from '@fluentui/react'

archive-doc-gen/tests/e2e-test/pages/draftPage.py

@@ -1,4 +1,5 @@
 import time
+import os
 from base.base import BasePage
 from pytest_check import check
 from playwright.sync_api import expect

archive-doc-gen/tests/e2e-test/tests/test_st_docgen_tc.py

@@ -6,7 +6,7 @@
 import pytest
 from pytest_check import check
 from playwright.sync_api import expect
-from config.constants import (add_section, browse_question1, browse_question2, browse_question3,
+from config.constants import (URL, add_section, browse_question1, browse_question2, browse_question3,
                               browse_question4, browse_question5, generate_question1, invalid_response, 
                               invalid_response1, remove_section)
 from pages.browsePage import BrowsePage
@@ -538,6 +538,7 @@ def test_show_hide_chat_history(login_logout, request):
 
     page = login_logout
     home_page = HomePage(page)
+    browse_page = BrowsePage(page)
     generate_page = GeneratePage(page)
 
     log_capture = io.StringIO()
@@ -2502,13 +2503,15 @@ def test_bug_7571_removed_sections_not_returning(request, login_logout):
         logger.info("Step 4: Enter a prompt to remove sections one by one 'Remove (section name)'")
         start = time.time()
 
-        # Select up to 3 sections to remove from the initial list (avoid removing first section for stability)
-        indices_to_remove = [1, 2, 3] if initial_count > 3 else list(range(1, initial_count))
-        sections_to_remove = [
-            initial_sections[idx]
-            for idx in indices_to_remove
-            if idx < len(initial_sections)
-        ]
+        # Select 3 sections to remove from the initial list
+        sections_to_remove = []
+        
+        if initial_count >= 3:
+            # Remove sections at positions 1, 2, and 3 (avoid removing first section for stability)
+            indices_to_remove = [1, 2, 3] if initial_count > 3 else list(range(1, initial_count))
+            for idx in indices_to_remove:
+                if idx < len(initial_sections):
+                    sections_to_remove.append(initial_sections[idx])
 
         logger.info("Sections selected for removal: %s", sections_to_remove)
 
@@ -3399,7 +3402,7 @@ def test_bug_10177_edit_delete_icons_disabled_during_response(login_logout, requ
                 try:
                     threads.first.wait_for(state="visible", timeout=10000)
                     logger.info("✅ Chat history created and displayed with %d thread(s)", threads.count())
-                except Exception:
+                except:
                     logger.error("❌ Chat history threads not visible after creation")
                     # Try alternative locator
                     threads_alt = page.locator('div[data-list-index]')
@@ -3449,7 +3452,7 @@ def test_bug_10177_edit_delete_icons_disabled_during_response(login_logout, requ
         try:
             delete_icon.wait_for(state="visible", timeout=2000)
             is_delete_visible = True
-        except Exception:
+        except:
             is_delete_visible = False
 
         is_delete_enabled = delete_icon.is_enabled() if is_delete_visible else False
@@ -3461,7 +3464,7 @@ def test_bug_10177_edit_delete_icons_disabled_during_response(login_logout, requ
         try:
             edit_icon.wait_for(state="visible", timeout=2000)
             is_edit_visible = True
-        except Exception:
+        except:
             is_edit_visible = False
 
         is_edit_enabled = edit_icon.is_enabled() if is_edit_visible else False
@@ -3915,9 +3918,7 @@ def test_bug_16106_tooltip_on_chat_history_hover(login_logout, request):
         with check:
             assert thread_count > 0, "No chat history threads found to hover over"
 
-        if thread_count <= 0:
-            logger.warning("Skipping hover action: no chat history threads available.")
-        else:
+        if thread_count > 0:
             # Hover over the first chat thread to trigger tooltip
             first_thread = history_threads.nth(0)
             first_thread.hover()
@@ -3992,6 +3993,8 @@ def test_bug_16106_tooltip_on_chat_history_hover(login_logout, request):
         if tooltip_found:
             logger.info("✅ Tooltip displayed successfully on chat history hover")
             logger.info("Tooltip text length: %d characters", len(tooltip_text))
+        else:
+            logger.error("❌ BUG FOUND: No tooltip displayed when hovering over chat history")
 
         duration = time.time() - start
         logger.info("Execution Time for 'Verify tooltip': %.2fs", duration)
@@ -4008,7 +4011,7 @@ def test_bug_16106_tooltip_on_chat_history_hover(login_logout, request):
                 page.keyboard.press("Escape")
                 page.wait_for_timeout(1000)
                 logger.info("Closed chat history using Escape key")
-            except Exception:
+            except:
                 logger.warning("Chat history panel may still be open")
 
         logger.info("\n%s", "="*80)
@@ -4103,10 +4106,10 @@ def test_bug_26031_validate_empty_spaces_chat_input(login_logout, request):
                 assert current_responses_empty == initial_responses, \
                     f"BUG: System accepted empty query. Response count changed from {initial_responses} to {current_responses_empty}"
 
-            if current_responses_empty != initial_responses:
-                logger.error("❌ BUG: System accepted empty query and generated response")
-            else:
+            if current_responses_empty == initial_responses:
                 logger.info("✅ System did not accept empty query - no response generated")
+            else:
+                logger.error("❌ BUG: System accepted empty query and generated response")
         else:
             logger.info("✅ Send button is properly disabled for empty input")