fix: Handle the case when parameter.required is None and fix tests

PiperOrigin-RevId: 754519323
This commit is contained in:
Hangfei Lin 2025-05-03 22:39:00 -07:00 committed by Copybara-Service
parent ad4226b3d8
commit a7b4c98253
2 changed files with 4 additions and 3 deletions

View File

@ -83,7 +83,8 @@ class OperationParser:
schema.description = ( schema.description = (
description if not schema.description else schema.description description if not schema.description else schema.description
) )
required = param.required # param.required can be None
required = param.required if param.required is not None else False
self.params.append( self.params.append(
ApiParameter( ApiParameter(

View File

@ -347,8 +347,8 @@ def test_get_json_schema(sample_operation):
assert json_schema['type'] == 'object' assert json_schema['type'] == 'object'
assert 'param1' in json_schema['properties'] assert 'param1' in json_schema['properties']
assert 'prop1' in json_schema['properties'] assert 'prop1' in json_schema['properties']
assert 'param1' in json_schema['required'] # By default nothing is required unless explicitly stated
assert 'prop1' in json_schema['required'] assert 'required' not in json_schema or json_schema['required'] == []
def test_get_signature_parameters(sample_operation): def test_get_signature_parameters(sample_operation):