update

Author: pablonyx

Diff for: backend/onyx/server/documents/connector.py

@@ -845,10 +845,6 @@ def create_connector_from_model(
     db_session: Session = Depends(get_session),
 ) -> ObjectCreationIdResponse:
     tenant_id = get_current_tenant_id()
-    print("Creating connector from model")
-    print("connector_data: ", connector_data)
-    print(connector_data.connector_specific_config)
-    print(connector_data.connector_specific_config)
 
     try:
         _validate_connector_allowed(connector_data.source)

Diff for: backend/tests/integration/common_utils/connectors.py

@@ -37,33 +37,22 @@ def upload_file(
     if "Content-Type" in headers:
         del headers["Content-Type"]
 
-    # Print debug info
-    print(f"Uploading file: {file_path} with name: {file_name}")
-    print(f"Headers: {headers}")
-
     # Make the request
     response = requests.post(
         f"{API_SERVER_URL}/manage/admin/connector/file/upload",
         files=files,
         headers=headers,
     )
 
-    # Print detailed response for debugging
-    print(f"Response status: {response.status_code}")
-    print(f"Response text: {response.text}")
-
     if not response.ok:
         try:
             error_detail = response.json().get("detail", "Unknown error")
-            print(f"Error detail: {error_detail}")
-        except Exception as e:
+        except Exception:
             error_detail = response.text
-            print(f"Failed to parse error as JSON: {e}")
 
         raise Exception(
             f"Unable to upload files - {error_detail} (Status code: {response.status_code})"
         )
 
     response_json = response.json()
-    print(f"Response JSON: {response_json}")
     return response_json

Diff for: backend/tests/integration/common_utils/managers/settings.py

@@ -55,8 +55,6 @@ def update_settings(
         )
 
         if not response.ok:
-            print("Failed to update settings")
-            print(response.json())
             return (
                 {},
                 f"Failed to update settings - {response.json().get('detail', 'Unknown error')}",

Diff for: backend/tests/integration/tests/image_indexing/indexing_images.py

@@ -29,7 +29,6 @@ def test_image_indexing(reset: None) -> None:
     test_file_dir = "tests/integration/common_utils/test_data"
     os.makedirs(test_file_dir, exist_ok=True)
     test_file_path = os.path.join(test_file_dir, "sample.pdf")
-    print(f"Using test file: {test_file_path}")
 
     # Use our fixed upload_file function to upload the test file
     file_name = "sample.pdf"
@@ -41,15 +40,14 @@ def test_image_indexing(reset: None) -> None:
         name="test_llm",
         user_performing_action=admin_user,
     )
-    print("Created LLM provider")
+
     SettingsManager.update_settings(
         DATestSettings(
             search_time_image_analysis_enabled=True,
             image_extraction_and_analysis_enabled=True,
         )
     )
 
-    print(f"File upload response: {upload_response}")
     file_paths = upload_response.get("file_paths", [])
 
     if not file_paths:
@@ -62,11 +60,6 @@ def test_image_indexing(reset: None) -> None:
         user_performing_action=admin_user,
     )
 
-    print("Created credential")
-
-    print("Creating connector")
-    print("file_paths: ", file_paths)
-    print({"file_locations": file_paths})
     # Create the connector
     connector_name = f"FileConnector-{int(datetime.now().timestamp())}"
     connector = ConnectorManager.create(
@@ -79,8 +72,6 @@ def test_image_indexing(reset: None) -> None:
         user_performing_action=admin_user,
     )
 
-    print(f"Created connector with ID: {connector.id}")
-
     # Link the credential to the connector
     cc_pair = CCPairManager.create(
         credential_id=credential.id,
@@ -89,8 +80,6 @@ def test_image_indexing(reset: None) -> None:
         user_performing_action=admin_user,
     )
 
-    print(f"Created CC pair with ID: {cc_pair.id}")
-
     # Explicitly run the connector to start indexing
     CCPairManager.run_once(
         cc_pair=cc_pair,
@@ -103,25 +92,16 @@ def test_image_indexing(reset: None) -> None:
         user_performing_action=admin_user,
     )
 
-    print("Started indexing")
-
-    # Give the system some time to index the document
-    print("Sleeping for 15 seconds to allow indexing to progress...")
-
     # Create a chat session and ask about dogs
-    print("Creating chat session to ask about dogs...")
     chat_session: DATestChatSession = ChatSessionManager.create(
         user_performing_action=admin_user
     )
 
     # Ask about dogs in the document
-    response = ChatSessionManager.send_message(
+    ChatSessionManager.send_message(
         chat_session_id=chat_session.id,
         message="How many dogs?",
         user_performing_action=admin_user,
     )
 
-    print(f"Response to 'How many dogs?': {response}")
-
     # Test passed if we were able to complete the workflow
-    print("Test successful - file uploaded and chat query completed")