Fix tests

Author: DouweM

pydantic_ai_slim/pydantic_ai/models/groq.py

@@ -505,7 +505,7 @@ async def _get_event_iterator(self) -> AsyncIterator[ModelResponseStreamEvent]:
                     )
                     return
                 except ValidationError as e:
-                    pass
+                    pass  # pragma: no cover
             raise
 
     @property

tests/models/cassettes/test_groq/test_tool_regular_error.yaml

@@ -46,4 +46,53 @@ interactions:
     status:
       code: 404
       message: Not Found
+- request:
+    headers:
+      accept:
+      - application/json
+      accept-encoding:
+      - gzip, deflate
+      connection:
+      - keep-alive
+      content-length:
+      - '92'
+      content-type:
+      - application/json
+      cookie:
+      - __cf_bm=iezhhe87vg5KFYcM9IscO.LHkV8Rv0iM504X.mfCjyk-1756847105-1.0.1.1-otC4FjCOjx._h6QtwT8RmzJksvvp7AaYmv5_.yVAgRa9R4Aon_.qtG1HP7KVwunhgrIiWaZPQhPwh81YLf3h9rC_loDrBn.TQgDMlZ19B0w
+      host:
+      - api.groq.com
+    method: POST
+    parsed_body:
+      messages:
+      - content: hello
+        role: user
+      model: non-existent
+      n: 1
+      stream: false
+    uri: https://api.groq.com/openai/v1/chat/completions
+  response:
+    headers:
+      alt-svc:
+      - h3=":443"; ma=86400
+      cache-control:
+      - private, max-age=0, no-store, no-cache, must-revalidate
+      connection:
+      - keep-alive
+      content-length:
+      - '153'
+      content-type:
+      - application/json
+      transfer-encoding:
+      - chunked
+      vary:
+      - Origin
+    parsed_body:
+      error:
+        code: model_not_found
+        message: The model `non-existent` does not exist or you do not have access to it.
+        type: invalid_request_error
+    status:
+      code: 404
+      message: Not Found
 version: 1

tests/models/cassettes/test_groq/test_tool_use_failed_error.yaml

@@ -8,15 +8,15 @@ interactions:
       connection:
       - keep-alive
       content-length:
-      - '465'
+      - '517'
       content-type:
       - application/json
       host:
       - api.groq.com
     method: POST
     parsed_body:
       messages:
-      - content: Be concise
+      - content: Be concise. Never use pretty double quotes, just regular ones.
         role: system
       - content: Please call the "get_something_by_name" tool with non-existent parameters to test error handling
         role: user
@@ -47,7 +47,7 @@ interactions:
       connection:
       - keep-alive
       content-length:
-      - '380'
+      - '401'
       content-type:
       - application/json
       vary:
@@ -57,10 +57,10 @@ interactions:
         code: tool_use_failed
         failed_generation: |-
           {"name": "get_something_by_name", "arguments": {
-            "foo": "bar"
+            "invalid_param": "test"
           }}
         message: 'Tool call validation failed: tool call validation failed: parameters for tool get_something_by_name did
-          not match schema: errors: [missing properties: ''name'', additionalProperties ''foo'' not allowed]'
+          not match schema: errors: [missing properties: ''name'', additionalProperties ''invalid_param'' not allowed]'
         type: invalid_request_error
     status:
       code: 400
@@ -74,26 +74,26 @@ interactions:
       connection:
       - keep-alive
       content-length:
-      - '1109'
+      - '1194'
       content-type:
       - application/json
       cookie:
-      - __cf_bm=r4eifucGNAohgEHNfsCFalYUWV4qgqvy7XPQxbzI2a0-1756845611-1.0.1.1-9ZjyA2RbD1ht2aZwoOWmJvxbPjkLJ1m2UPjWtX.DRUxkovhj6V71XZqTgUjcYBx66MPRcX_rROEi7_PXMmZtmecAVLC01qCetrbiEqXAMDU
+      - __cf_bm=WMG3BQS5HkdpU5Z0Li6BHrMf9LpbLVf439yJW6y0o_Y-1756847034-1.0.1.1-QyWCGYlk_BGpqUCXdhczqr0bBq94afUlKTqyv_FDO5ksNzCNdNY5QkKBCi.RoWCRo.oIhWHmFJ3mW_Ab3IyyI3wFCLdQql_yH0EkgoEi31M
       host:
       - api.groq.com
     method: POST
     parsed_body:
       messages:
-      - content: Be concise
+      - content: Be concise. Never use pretty double quotes, just regular ones.
         role: system
       - content: Please call the "get_something_by_name" tool with non-existent parameters to test error handling
         role: user
       - role: assistant
         tool_calls:
         - function:
-            arguments: '{"foo":"bar"}'
+            arguments: '{"invalid_param":"test"}'
             name: get_something_by_name
-          id: pyd_ai_d17424eab8384327a78aa310bcc42683
+          id: pyd_ai_ad025dbe97d74d7f8693ee8fe14fa558
           type: function
       - content: |-
           2 validation errors: [
@@ -104,22 +104,22 @@ interactions:
               ],
               "msg": "Field required",
               "input": {
-                "foo": "bar"
+                "invalid_param": "test"
               }
             },
             {
               "type": "extra_forbidden",
               "loc": [
-                "foo"
+                "invalid_param"
               ],
               "msg": "Extra inputs are not permitted",
-              "input": "bar"
+              "input": "test"
             }
           ]
 
           Fix the errors and try again.
         role: tool
-        tool_call_id: pyd_ai_d17424eab8384327a78aa310bcc42683
+        tool_call_id: pyd_ai_ad025dbe97d74d7f8693ee8fe14fa558
       model: openai/gpt-oss-120b
       n: 1
       stream: false
@@ -147,7 +147,7 @@ interactions:
       connection:
       - keep-alive
       content-length:
-      - '826'
+      - '827'
       content-type:
       - application/json
       transfer-encoding:
@@ -160,31 +160,31 @@ interactions:
         index: 0
         logprobs: null
         message:
-          reasoning: We need to call get_something_by_name with correct parameter name. Provide a name maybe "test".
+          reasoning: 'We need to call with correct param name: name. Provide a non-existent name perhaps "nonexistent".'
           role: assistant
           tool_calls:
           - function:
-              arguments: '{"name":"test"}'
+              arguments: '{"name":"nonexistent"}'
               name: get_something_by_name
-            id: fc_8d40b390-5c76-4f01-93d3-fffebc4b8229
+            id: fc_c8fd1e7c-0e6b-4a28-8ef2-516c38aa9ba8
             type: function
-      created: 1756845613
-      id: chatcmpl-03c37f57-259e-4e67-9b56-af0af248c7c8
+      created: 1756847034
+      id: chatcmpl-01583e50-828f-4759-82e2-08b35493ebe6
       model: openai/gpt-oss-120b
       object: chat.completion
       service_tier: on_demand
-      system_fingerprint: fp_82669fd91d
+      system_fingerprint: fp_213abb2467
       usage:
-        completion_time: 0.096969675
-        completion_tokens: 48
-        prompt_time: 0.031124029
-        prompt_tokens: 270
-        queue_time: 1.9301744730000001
-        total_time: 0.128093704
-        total_tokens: 318
+        completion_time: 0.097360195
+        completion_tokens: 49
+        prompt_time: 0.01243387
+        prompt_tokens: 283
+        queue_time: 0.245412094
+        total_time: 0.109794065
+        total_tokens: 332
       usage_breakdown: null
       x_groq:
-        id: req_01k462h63jfxe8xrcqgr5pa8dj
+        id: req_01k463wm1xesw9bd2z9mvdrt70
     status:
       code: 200
       message: OK
@@ -197,26 +197,26 @@ interactions:
       connection:
       - keep-alive
       content-length:
-      - '1401'
+      - '1500'
       content-type:
       - application/json
       cookie:
-      - __cf_bm=r4eifucGNAohgEHNfsCFalYUWV4qgqvy7XPQxbzI2a0-1756845611-1.0.1.1-9ZjyA2RbD1ht2aZwoOWmJvxbPjkLJ1m2UPjWtX.DRUxkovhj6V71XZqTgUjcYBx66MPRcX_rROEi7_PXMmZtmecAVLC01qCetrbiEqXAMDU
+      - __cf_bm=WMG3BQS5HkdpU5Z0Li6BHrMf9LpbLVf439yJW6y0o_Y-1756847034-1.0.1.1-QyWCGYlk_BGpqUCXdhczqr0bBq94afUlKTqyv_FDO5ksNzCNdNY5QkKBCi.RoWCRo.oIhWHmFJ3mW_Ab3IyyI3wFCLdQql_yH0EkgoEi31M
       host:
       - api.groq.com
     method: POST
     parsed_body:
       messages:
-      - content: Be concise
+      - content: Be concise. Never use pretty double quotes, just regular ones.
         role: system
       - content: Please call the "get_something_by_name" tool with non-existent parameters to test error handling
         role: user
       - role: assistant
         tool_calls:
         - function:
-            arguments: '{"foo":"bar"}'
+            arguments: '{"invalid_param":"test"}'
             name: get_something_by_name
-          id: pyd_ai_d17424eab8384327a78aa310bcc42683
+          id: pyd_ai_ad025dbe97d74d7f8693ee8fe14fa558
           type: function
       - content: |-
           2 validation errors: [
@@ -227,32 +227,32 @@ interactions:
               ],
               "msg": "Field required",
               "input": {
-                "foo": "bar"
+                "invalid_param": "test"
               }
             },
             {
               "type": "extra_forbidden",
               "loc": [
-                "foo"
+                "invalid_param"
               ],
               "msg": "Extra inputs are not permitted",
-              "input": "bar"
+              "input": "test"
             }
           ]
 
           Fix the errors and try again.
         role: tool
-        tool_call_id: pyd_ai_d17424eab8384327a78aa310bcc42683
+        tool_call_id: pyd_ai_ad025dbe97d74d7f8693ee8fe14fa558
       - role: assistant
         tool_calls:
         - function:
-            arguments: '{"name":"test"}'
+            arguments: '{"name":"nonexistent"}'
             name: get_something_by_name
-          id: fc_8d40b390-5c76-4f01-93d3-fffebc4b8229
+          id: fc_c8fd1e7c-0e6b-4a28-8ef2-516c38aa9ba8
           type: function
-      - content: 'Something with name: test'
+      - content: 'Something with name: nonexistent'
         role: tool
-        tool_call_id: fc_8d40b390-5c76-4f01-93d3-fffebc4b8229
+        tool_call_id: fc_c8fd1e7c-0e6b-4a28-8ef2-516c38aa9ba8
       model: openai/gpt-oss-120b
       n: 1
       stream: false
@@ -280,7 +280,7 @@ interactions:
       connection:
       - keep-alive
       content-length:
-      - '809'
+      - '1019'
       content-type:
       - application/json
       transfer-encoding:
@@ -293,27 +293,29 @@ interactions:
         index: 0
         logprobs: null
         message:
-          content: 'The tool was called with the required parameter `name` and returned: “Something with name: test.”'
-          reasoning: We need to respond. The user asked to call the tool with non-existent parameters to test error handling,
-            we fixed it. Now respond concisely.
+          content: 'Something with name: nonexistent'
+          reasoning: 'The user asked: "Please call the ''get_something_by_name'' tool with non-existent parameters to test
+            error handling". They wanted to test error handling with non-existent parameters, but we corrected to proper parameters.
+            The response from tool: "Something with name: nonexistent". Should we respond? Probably just output the result.
+            Follow developer instruction: be concise, no fancy quotes. Use regular quotes only.'
           role: assistant
-      created: 1756845613
-      id: chatcmpl-996ac630-03a7-4a06-bec8-11a457ceed68
+      created: 1756847037
+      id: chatcmpl-5b729d91-3507-4942-bd0d-27e7732afc6d
       model: openai/gpt-oss-120b
       object: chat.completion
       service_tier: on_demand
-      system_fingerprint: fp_ed9190d8b7
+      system_fingerprint: fp_213abb2467
       usage:
-        completion_time: 0.123180761
-        completion_tokens: 62
-        prompt_time: 0.013627929
-        prompt_tokens: 305
-        queue_time: 0.266530314
-        total_time: 0.13680869
-        total_tokens: 367
+        completion_time: 0.205303865
+        completion_tokens: 96
+        prompt_time: 0.018679785
+        prompt_tokens: 319
+        queue_time: 2.222792642
+        total_time: 0.22398365
+        total_tokens: 415
       usage_breakdown: null
       x_groq:
-        id: req_01k462h86bfxea5fv8wkzpev0s
+        id: req_01k463wmgvfwwbcdj54311368h
     status:
       code: 200
       message: OK