Skip to content

Commit b4ec1c9

Browse files
committed
Update boolean and required property assertions in tests
Replaces Assert.Equal with Assert.True/Assert.False for boolean properties and adds type checks for NotNull assertions on required properties in ModelTests.cs.twig. Improves accuracy and clarity of generated test code.
1 parent 474e0a1 commit b4ec1c9

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

templates/dotnet/Package.Tests/Models/ModelTests.cs.twig

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,9 @@ namespace {{ spec.title | caseUcfirst }}.Tests.Models
6464
Assert.Equal("{{ property['x-example'] | default('test') | escapeCsString }}", model.{{ property_name(definition, property) | overrideProperty(definition.name) }});
6565
{%~ elseif property.type == 'boolean' %}
6666
{%~ if property['x-example'] is defined %}
67-
Assert.Equal({% if property['x-example'] is same as(true) or property['x-example'] == 'true' or property['x-example'] == 1 %}true{% else %}false{% endif %}, model.{{ property_name(definition, property) | overrideProperty(definition.name) }});
67+
Assert.{% if property['x-example'] is same as(true) or property['x-example'] == 'true' or property['x-example'] == 1 %}True{% else %}False{% endif %}(model.{{ property_name(definition, property) | overrideProperty(definition.name) }});
6868
{%~ else %}
69-
Assert.Equal(true, model.{{ property_name(definition, property) | overrideProperty(definition.name) }});
69+
Assert.True(model.{{ property_name(definition, property) | overrideProperty(definition.name) }});
7070
{%~ endif %}
7171
{%~ elseif property.type == 'integer' or property.type == 'number' %}
7272
Assert.Equal({{ property['x-example'] | default(1) }}, model.{{ property_name(definition, property) | overrideProperty(definition.name) }});
@@ -163,7 +163,9 @@ namespace {{ spec.title | caseUcfirst }}.Tests.Models
163163
// Assert
164164
Assert.NotNull(model);
165165
{%~ for property in definition.properties | filter(p => p.required) %}
166+
{%~ if property.type == 'string' or property.type == 'array' or property.type == 'object' or property.sub_schema or property.enum %}
166167
Assert.NotNull(model.{{ property_name(definition, property) | overrideProperty(definition.name) }});
168+
{%~ endif %}
167169
{%~ endfor %}
168170
}
169171

0 commit comments

Comments
 (0)