Skip to content
This repository was archived by the owner on Jul 28, 2025. It is now read-only.
CI/CD pull request

refactor: DTOSS-9159 - split file-level validation into four separate… #487

Sign in to view logs

refactor: DTOSS-9159 - split file-level validation into four separate…

refactor: DTOSS-9159 - split file-level validation into four separate… #487

Triggered via push June 10, 2025 11:15
@ianfnelsonianfnelson
pushed ed4d9b8
main
Status Failure
Total duration 7m 16s
Artifacts 4

cicd-1-pull-request.yaml

on: push
Set CI/CD metadata
5s
Set CI/CD metadata
Validate PR title
0s
Validate PR title
Commit stage  /  Scan secrets
7s
Commit stage / Scan secrets
Commit stage  /  Check file format
7s
Commit stage / Check file format
Commit stage  /  Check Markdown format
4s
Commit stage / Check Markdown format
Commit stage  /  Check English usage
9s
Commit stage / Check English usage
Commit stage  /  Lint Terraform
5s
Commit stage / Lint Terraform
Commit stage  /  Count lines of code
9s
Commit stage / Count lines of code
Test stage  /  Linting
4s
Test stage / Linting
Test stage  /  Unit tests
57s
Test stage / Unit tests
Test stage  /  Perform static analysis
2m 21s
Test stage / Perform static analysis
Image build stage  /  Containers to build
6s
Image build stage / Containers to build
Matrix: Image build stage / Docker build and push
Image build stage  /  Merge commit tag all images
25s
Image build stage / Merge commit tag all images
Image build stage  /  Aggregate reports
7s
Image build stage / Aggregate reports
Acceptance stage  /  Environment set up
Acceptance stage / Environment set up
Acceptance stage  /  Accessibility test
Acceptance stage / Accessibility test
Acceptance stage  /  Contract test
Acceptance stage / Contract test
Acceptance stage  /  Integration test
Acceptance stage / Integration test
Acceptance stage  /  Load test
Acceptance stage / Load test
Acceptance stage  /  Security test
Acceptance stage / Security test
Acceptance stage  /  UI performance test
Acceptance stage / UI performance test
Acceptance stage  /  UI test
Acceptance stage / UI test
Acceptance stage  /  Environment tear down
Acceptance stage / Environment tear down
Fit to window
Zoom out
Zoom in

Annotations

1 error and 20 warnings
Image build stage / Merge commit tag all images
Process completed with exit code 5.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOperationService.cs#L73
Possible null reference return.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L195
Possible null reference return.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Clients/MeshConnectClient.cs#L99
Dereference of a possibly null reference.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Clients/MeshConnectClient.cs#L96
Dereference of a possibly null reference.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L151
Possible null reference return.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L137
Possible null reference return.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Configuration/MailboxConfigurationResolver.cs#L14
Converting null literal or possible null value to non-nullable type.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Configuration/MeshConnectConfiguration.cs#L51
Non-nullable property 'ProxyAddress' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L107
Possible null reference return.
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L67
Possible null reference return.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L23
Non-nullable property 'RecipientOdsCode' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L21
Non-nullable property 'RecipientName' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L19
Non-nullable property 'Recipient' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L17
Non-nullable property 'UploadTimestamp' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L15
Non-nullable property 'ExpiryTime' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L13
Non-nullable property 'FileName' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L7
Non-nullable property 'MessageId' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Configuration/MeshConnectConfiguration.cs#L51
Non-nullable property 'ProxyAddress' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test stage / Perform static analysis: src/ServiceLayer.Shared/Data/Migrations/20250521094738_AddNbssAppointmentEventTable.cs#L41
Define a constant instead of using this literal 'varchar(30)' 5 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
Test stage / Perform static analysis: src/ServiceLayer.Shared/Data/Migrations/20250521094738_AddNbssAppointmentEventTable.cs#L23
Define a constant instead of using this literal 'char(1)' 8 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)

Artifacts

Produced during runtime
Name Size Digest
aggregated-reports-pr48 Expired
688 KB
sha256:454a54d817d8d33aa4e4b60505a6940de57d5e950ae994c7332bb3308424cd42
lines-of-code-report.json.zip Expired
860 Bytes
sha256:3c93a5ed57817816789f27261a53f659daace706542fef29e58c15f26d0523ac
service-layer-svclyr-mesh-ingest-sbom Expired
684 KB
sha256:92d9fe694bac8b99aba034d1118946569f42e8c31ef2fbd31f8dbe9b2452cbb9
service-layer-svclyr-mesh-ingest-vulnerabilities Expired
2.94 KB
sha256:0ce76482b53f058fb654fd84555c9004ab908c07b2776dc25d522d35191cf517