Skip to content

Commit f886ed6

Browse files
conradleeclaude
andcommitted
Update test snapshots after removing title fields from Gemini schemas
The GoogleJsonSchemaTransformer now removes all 'title' fields to fix Gemini's MALFORMED_FUNCTION_CALL bug. Updated test snapshots to reflect this change - all 'title' fields are now absent from expected outputs. This completes the fix for issue #3483. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 85295ad commit f886ed6

File tree

1 file changed

+12
-29
lines changed

1 file changed

+12
-29
lines changed

tests/models/test_gemini.py

Lines changed: 12 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -140,26 +140,22 @@ async def test_model_tools(allow_model_requests: None):
140140
'description': 'This is foo',
141141
'parameters': {
142142
'type': 'object',
143-
'title': 'Foo',
144-
'properties': {'bar': {'type': 'number', 'title': 'Bar'}},
143+
'properties': {'bar': {'type': 'number'}},
145144
},
146145
},
147146
{
148147
'name': 'apple',
149148
'description': 'This is apple',
150149
'parameters': {
151150
'type': 'object',
152-
'properties': {
153-
'banana': {'type': 'array', 'title': 'Banana', 'items': {'type': 'number', 'title': 'Bar'}}
154-
},
151+
'properties': {'banana': {'type': 'array', 'items': {'type': 'number'}}},
155152
},
156153
},
157154
{
158155
'name': 'result',
159156
'description': 'This is the tool for the final Result',
160157
'parameters': {
161158
'type': 'object',
162-
'title': 'Result',
163159
'properties': {'spam': {'type': 'number'}},
164160
'required': ['spam'],
165161
},
@@ -193,7 +189,7 @@ async def test_require_response_tool(allow_model_requests: None):
193189
{
194190
'name': 'result',
195191
'description': 'This is the tool for the final Result',
196-
'parameters': {'type': 'object', 'title': 'Result', 'properties': {'spam': {'type': 'number'}}},
192+
'parameters': {'type': 'object', 'properties': {'spam': {'type': 'number'}}},
197193
}
198194
]
199195
}
@@ -285,40 +281,34 @@ class Locations(BaseModel):
285281
'properties': {
286282
'locations': {
287283
'items': {'$ref': '#/$defs/Location'},
288-
'title': 'Locations',
289284
'type': 'array',
290285
}
291286
},
292287
'required': ['locations'],
293-
'title': 'Locations',
294288
'type': 'object',
295289
'$defs': {
296290
'Axis': {
297291
'properties': {
298292
'label': {
299293
'default': '<unlabeled axis>',
300294
'description': 'The label of the axis',
301-
'title': 'Label',
302295
'type': 'string',
303296
}
304297
},
305-
'title': 'Axis',
306298
'type': 'object',
307299
},
308300
'Chart': {
309301
'properties': {'x_axis': {'$ref': '#/$defs/Axis'}, 'y_axis': {'$ref': '#/$defs/Axis'}},
310302
'required': ['x_axis', 'y_axis'],
311-
'title': 'Chart',
312303
'type': 'object',
313304
},
314305
'Location': {
315306
'properties': {
316-
'lat': {'title': 'Lat', 'type': 'number'},
317-
'lng': {'default': 1.1, 'title': 'Lng', 'type': 'number'},
307+
'lat': {'type': 'number'},
308+
'lng': {'default': 1.1, 'type': 'number'},
318309
'chart': {'$ref': '#/$defs/Chart'},
319310
},
320311
'required': ['lat', 'chart'],
321-
'title': 'Location',
322312
'type': 'object',
323313
},
324314
},
@@ -381,18 +371,14 @@ class QueryDetails(BaseModel):
381371
'properties': {
382372
'progress': {
383373
'default': None,
384-
'title': 'Progress',
385374
'anyOf': [
386375
{'items': {'$ref': '#/$defs/ProgressEnum'}, 'type': 'array'},
387376
{'type': 'null'},
388377
],
389378
}
390379
},
391-
'title': 'QueryDetails',
392380
'type': 'object',
393-
'$defs': {
394-
'ProgressEnum': {'enum': [100, 80, 60, 40, 20], 'title': 'ProgressEnum', 'type': 'integer'}
395-
},
381+
'$defs': {'ProgressEnum': {'enum': [100, 80, 60, 40, 20], 'type': 'integer'}},
396382
},
397383
}
398384
]
@@ -434,16 +420,14 @@ class Locations(BaseModel):
434420
'properties': {
435421
'op_location': {'default': None, 'anyOf': [{'$ref': '#/$defs/Location'}, {'type': 'null'}]}
436422
},
437-
'title': 'Locations',
438423
'type': 'object',
439424
'$defs': {
440425
'Location': {
441426
'properties': {
442-
'lat': {'title': 'Lat', 'type': 'number'},
443-
'lng': {'title': 'Lng', 'type': 'number'},
427+
'lat': {'type': 'number'},
428+
'lng': {'type': 'number'},
444429
},
445430
'required': ['lat', 'lng'],
446-
'title': 'Location',
447431
'type': 'object',
448432
}
449433
},
@@ -498,13 +482,12 @@ class FormattedStringFields(BaseModel):
498482
'description': 'This is the tool for the final Result',
499483
'parameters': {
500484
'properties': {
501-
'd': {'title': 'D', 'type': 'string', 'description': 'Format: date'},
502-
'dt': {'title': 'Dt', 'type': 'string', 'description': 'Format: date-time'},
503-
't': {'description': 'Format: time', 'title': 'T', 'type': 'string'},
504-
'td': {'description': 'my timedelta (format: duration)', 'title': 'Td', 'type': 'string'},
485+
'd': {'type': 'string', 'description': 'Format: date'},
486+
'dt': {'type': 'string', 'description': 'Format: date-time'},
487+
't': {'description': 'Format: time', 'type': 'string'},
488+
'td': {'description': 'my timedelta (format: duration)', 'type': 'string'},
505489
},
506490
'required': ['d', 'dt', 't', 'td'],
507-
'title': 'FormattedStringFields',
508491
'type': 'object',
509492
},
510493
}

0 commit comments

Comments
 (0)