Skip to content

use '?' to work with nullable objects #284

use '?' to work with nullable objects

use '?' to work with nullable objects #284

Re-run triggered September 27, 2024 12:00
Status Failure
Total duration 4m 4s
Artifacts

pr.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

20 errors and 22 warnings
ubuntu-latest
[xUnit.net 00:00:01.44] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: KnockoutClass, version: 4.3) [FAIL]
ubuntu-latest
[xUnit.net 00:00:01.60] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: KnockoutClass, version: 2.7) [FAIL]
ubuntu-latest
[xUnit.net 00:00:01.68] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: Class, version: 1.8) [FAIL]
ubuntu-latest
[xUnit.net 00:00:01.98] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: Class, version: 2.1) [FAIL]
ubuntu-latest
[xUnit.net 00:00:02.05] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: Class, version: 4.3) [FAIL]
ubuntu-latest
[xUnit.net 00:00:02.14] NJsonSchema.CodeGeneration.TypeScript.Tests.TypeScriptDiscriminatorTests.When_parameter_is_abstract_then_generate_union_class [FAIL]
ubuntu-latest
[xUnit.net 00:00:02.29] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: Class, version: 2.7) [FAIL]
ubuntu-latest
[xUnit.net 00:00:02.39] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: KnockoutClass, version: 2.1) [FAIL]
ubuntu-latest
[xUnit.net 00:00:02.45] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: KnockoutClass, version: 1.8) [FAIL]
ubuntu-latest
Target "Test" has thrown an exception
windows-latest
[xUnit.net 00:00:00.73] NJsonSchema.CodeGeneration.TypeScript.Tests.TypeScriptDiscriminatorTests.When_parameter_is_abstract_then_generate_union_class [FAIL]
windows-latest
[xUnit.net 00:00:00.73] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: KnockoutClass, version: 4.3) [FAIL]
windows-latest
[xUnit.net 00:00:00.77] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: KnockoutClass, version: 2.7) [FAIL]
windows-latest
[xUnit.net 00:00:00.79] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: Class, version: 1.8) [FAIL]
windows-latest
[xUnit.net 00:00:00.84] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: Class, version: 2.1) [FAIL]
windows-latest
[xUnit.net 00:00:00.86] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: Class, version: 4.3) [FAIL]
windows-latest
[xUnit.net 00:00:00.88] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: Class, version: 2.7) [FAIL]
windows-latest
[xUnit.net 00:00:00.91] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: KnockoutClass, version: 2.1) [FAIL]
windows-latest
[xUnit.net 00:00:00.94] NJsonSchema.CodeGeneration.TypeScript.Tests.ClassGenerationTests.Verify_output(style: KnockoutClass, version: 1.8) [FAIL]
windows-latest
[xUnit.net 00:00:01.87] NJsonSchema.CodeGeneration.TypeScript.Tests.TypeScriptDiscriminatorTests.When_parameter_is_abstract_then_generate_union_class [FAIL]
ubuntu-latest
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/cache@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/Generation/StructTests.cs(18,47): warning CA1051: Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051) [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/NJsonSchema.NewtonsoftJson.Tests.csproj::TargetFramework=net8.0]
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/Generation/StructTests.cs(21,48): warning CA1051: Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051) [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/NJsonSchema.NewtonsoftJson.Tests.csproj::TargetFramework=net8.0]
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/Generation/StructTests.cs(25,21): warning CA1707: Remove the underscores from member name NJsonSchema.NewtonsoftJson.Tests.Generation.StructTests.Should_have_a_shared_struct_schema() (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1707) [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/NJsonSchema.NewtonsoftJson.Tests.csproj::TargetFramework=net8.0]
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/Generation/RecordTests.cs(14,21): warning CA1707: Remove the underscores from member name NJsonSchema.NewtonsoftJson.Tests.Generation.RecordTests.Should_have_only_one_property() (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1707) [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/NJsonSchema.NewtonsoftJson.Tests.csproj::TargetFramework=net8.0]
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/Generation/StructTests.cs(34,13): warning xUnit2013: Do not use Assert.Equal() to check for collection size. Use Assert.Single instead. (https://xunit.net/xunit.analyzers/rules/xUnit2013) [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/NJsonSchema.NewtonsoftJson.Tests.csproj::TargetFramework=net8.0]
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/Generation/RecordTests.cs(23,13): warning xUnit2013: Do not use Assert.Equal() to check for collection size. Use Assert.Single instead. (https://xunit.net/xunit.analyzers/rules/xUnit2013) [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.NewtonsoftJson.Tests/NJsonSchema.NewtonsoftJson.Tests.csproj::TargetFramework=net8.0]
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.Demo.Performance/Program.cs(35,23): warning CS8618: Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable. [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.Demo.Performance/NJsonSchema.Demo.Performance.csproj]
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.Demo.Performance/Program.cs(41,23): warning CS8618: Non-nullable property 'Bar' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable. [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.Demo.Performance/NJsonSchema.Demo.Performance.csproj]
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.Demo.Performance/Program.cs(73,23): warning CS8618: Non-nullable property 'Person' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable. [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.Demo.Performance/NJsonSchema.Demo.Performance.csproj]
ubuntu-latest
/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.Demo.Performance/Program.cs(75,24): warning CS8618: Non-nullable property 'Teacher' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable. [/home/runner/work/NJsonSchema/NJsonSchema/src/NJsonSchema.Demo.Performance/NJsonSchema.Demo.Performance.csproj]
windows-latest
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/cache@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\References\YamlReferencesTests.cs(21,27): warning CA1707: Remove the underscores from member name NJsonSchema.Yaml.Tests.References.LocalReferencesTests.When_yaml_schema_has_references_it_works(string, string) (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1707) [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\NJsonSchema.Yaml.Tests.csproj::TargetFramework=net8.0]
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\References\YamlReferencesTests.cs(38,27): warning CA1707: Remove the underscores from member name NJsonSchema.Yaml.Tests.References.LocalReferencesTests.When_yaml_OpenAPI_spec_has_external_schema_refs_they_are_resolved(string, string, string) (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1707) [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\NJsonSchema.Yaml.Tests.csproj::TargetFramework=net8.0]
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\References\YamlReferencesTests.cs(69,27): warning CA1707: Remove the underscores from member name NJsonSchema.Yaml.Tests.References.LocalReferencesTests.When_yaml_OpenAPI_spec_is__served_with_gzip_compression__it_works(string, string) (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1707) [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\NJsonSchema.Yaml.Tests.csproj::TargetFramework=net8.0]
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\References\YamlReferencesTests.cs(83,27): warning CA1707: Remove the underscores from member name NJsonSchema.Yaml.Tests.References.LocalReferencesTests.When_yaml_OpenAPI_spec_has_relative_external_schema_refs_in_subdirs__they_are_resolved(string) (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1707) [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\NJsonSchema.Yaml.Tests.csproj::TargetFramework=net8.0]
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\References\YamlReferencesTests.cs(115,24): warning CA1822: Member 'GetTestDirectory' does not access instance data and can be marked as static (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1822) [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Yaml.Tests\NJsonSchema.Yaml.Tests.csproj::TargetFramework=net8.0]
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\Program.cs(73,23): warning CS8618: Non-nullable property 'Person' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable. [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\NJsonSchema.Demo.Performance.csproj]
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\Program.cs(75,24): warning CS8618: Non-nullable property 'Teacher' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable. [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\NJsonSchema.Demo.Performance.csproj]
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\Program.cs(77,34): warning CS8618: Non-nullable property 'WritingInstrument' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable. [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\NJsonSchema.Demo.Performance.csproj]
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\Program.cs(68,23): warning CS8618: Non-nullable property 'Baz' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable. [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\NJsonSchema.Demo.Performance.csproj]
windows-latest
D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\Program.cs(57,23): warning CS8618: Non-nullable property 'Bar' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable. [D:\a\NJsonSchema\NJsonSchema\src\NJsonSchema.Demo.Performance\NJsonSchema.Demo.Performance.csproj]