Fixed tool_choice injection in test_sucessful_memory_agent

tests/test_agents.py

@@ -353,7 +353,7 @@ async def llm_model_call(*args, **kwargs):
         # https://docs.anthropic.com/en/docs/build-with-claude/tool-use#chain-of-thought
         # kwargs.pop("tool_choice", LiteLLMModel.TOOL_CHOICE_REQUIRED)
         # tool_choice is now a arg, not a kwarg
-        args.pop()  # removing it from args # ASK: I accept ideas on how to handle this better
+        args = args[:-1]  # removing last element from args
         return await orig_llm_model_call(*args, tool_choice="auto", **kwargs)  # type: ignore[misc]
 
     with patch.object(LiteLLMModel, "call", side_effect=llm_model_call, autospec=True):

tests/test_paperqa.py

@@ -427,9 +427,7 @@ def test_llm_parse_json_newlines() -> None:
 
 @pytest.mark.asyncio
 async def test_chain_completion() -> None:
-    s = Settings(
-        llm="babbage-002", temperature=0.2
-    )
+    s = Settings(llm="babbage-002", temperature=0.2)
     outputs = []
 
     def accum(x) -> None: