Merge branch 'main' into renovate/python-nonmajor

Author: twishabansal

.github/ISSUE_TEMPLATE/bug_report.md

@@ -20,7 +20,7 @@ If you are still having issues, please be sure to include as much information as
   - OS type and version:
   - Python version: `python --version`
   - pip version: `pip --version`
-  - `toolbox-langchain` version: `pip show toolbox-langchain`
+  - Version of the package you're using. Use `pip show <package-name>`
 
 #### Steps to reproduce
 
@@ -40,4 +40,4 @@ If you are still having issues, please be sure to include as much information as
 
 Making sure to follow these steps will guarantee the quickest resolution possible.
 
-Thanks!
+Thanks!

logo.png

@@ -74,30 +74,6 @@ def static_client_header() -> dict[str, str]:
     return {"X-Client-Static": "client-static-value"}
 
 
-@pytest.fixture
-def sync_callable_client_header_value() -> str:
-    return "client-sync-callable-value"
-
-
-@pytest.fixture
-def sync_callable_client_header(sync_callable_client_header_value) -> dict[str, Mock]:
-    return {"X-Client-Sync": Mock(return_value=sync_callable_client_header_value)}
-
-
-@pytest.fixture
-def async_callable_client_header_value() -> str:
-    return "client-async-callable-value"
-
-
-@pytest.fixture
-def async_callable_client_header(
-    async_callable_client_header_value,
-) -> dict[str, AsyncMock]:
-    return {
-        "X-Client-Async": AsyncMock(return_value=async_callable_client_header_value)
-    }
-
-
 # --- Fixtures for Auth Getters ---
 
 
@@ -111,11 +87,6 @@ def auth_getters(auth_token_value) -> dict[str, Callable[[], str]]:
     return {"test-auth": lambda: auth_token_value}
 
 
-@pytest.fixture
-def auth_getters_mock(auth_token_value) -> dict[str, Mock]:
-    return {"test-auth": Mock(return_value=auth_token_value)}
-
-
 @pytest.fixture
 def auth_header_key() -> str:
     return "test-auth_token"
@@ -421,3 +392,37 @@ def test_tool_init_header_auth_conflict(
             bound_params={},
             client_headers=conflicting_client_header,
         )
+
+
+def test_tool_add_auth_token_getters_conflict_with_existing_client_header(
+    http_session: ClientSession,
+    sample_tool_params: list[ParameterSchema],
+    sample_tool_description: str,
+):
+    """
+    Tests ValueError when add_auth_token_getters introduces an auth service
+    whose token name conflicts with an existing client header.
+    """
+    tool_instance = ToolboxTool(
+        session=http_session,
+        base_url=TEST_BASE_URL,
+        name="tool_with_client_header",
+        description=sample_tool_description,
+        params=sample_tool_params,
+        required_authn_params={},
+        auth_service_token_getters={},
+        bound_params={},
+        client_headers={
+            "X-Shared-Auth-Token_token": "value_from_initial_client_headers"
+        },
+    )
+    new_auth_getters_causing_conflict = {
+        "X-Shared-Auth-Token": lambda: "token_value_from_new_getter"
+    }
+    expected_error_message = (
+        f"Client header\\(s\\) `X-Shared-Auth-Token_token` already registered in client. "
+        f"Cannot register client the same headers in the client as well as tool."
+    )
+
+    with pytest.raises(ValueError, match=expected_error_message):
+        tool_instance.add_auth_token_getters(new_auth_getters_causing_conflict)