Skip to content

Commit 49c3dcb

Browse files
committed
chore: Consolidate auth header creation logic
Post adding the feature of adding client-level auth headers (#178), we have the logic for creating an auth header, from the given auth token getter name, in 3 different places. This PR unifies all of that logic into a single helper to improve maintenance, and make it easier to change the way we add suffix/prefix, and reduces WET code.
1 parent ea1e95a commit 49c3dcb

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

packages/toolbox-core/tests/test_client.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -827,7 +827,6 @@ async def test_bind_param_async_callable_value_success(self, tool_name, client):
827827
assert res_payload == {"argA": passed_value_a, "argB": bound_value_result}
828828
bound_async_callable.assert_awaited_once()
829829

830-
831830
@pytest.mark.asyncio
832831
async def test_bind_param_success(self, tool_name, client):
833832
"""Tests 'bind_param' with a bound parameter specified."""

0 commit comments

Comments
 (0)