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 classes #485

Sign in to view logs

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

refactor: DTOSS-9159 - split file-level validation into four separate classes #485

Triggered via pull request June 10, 2025 10:15
@ianfnelsonianfnelson
opened #48
refactor/DTOSS-9159-refactor
Status Success
Total duration 7m 0s
Artifacts 4

cicd-1-pull-request.yaml

on: pull_request
Set CI/CD metadata
8s
Set CI/CD metadata
Validate PR title
5s
Validate PR title
Commit stage  /  Scan secrets
6s
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
8s
Commit stage / Check English usage
Commit stage  /  Lint Terraform
6s
Commit stage / Lint Terraform
Commit stage  /  Count lines of code
6s
Commit stage / Count lines of code
Test stage  /  Linting
3s
Test stage / Linting
Test stage  /  Unit tests
1m 1s
Test stage / Unit tests
Test stage  /  Perform static analysis
2m 15s
Test stage / Perform static analysis
Image build stage  /  Containers to build
5s
Image build stage / Containers to build
Matrix: Image build stage / Docker build and push
Image build stage  /  Merge commit tag all images
0s
Image build stage / Merge commit tag all images
Image build stage  /  Aggregate reports
3s
Image build stage / Aggregate reports
Acceptance stage  /  Environment set up
4s
Acceptance stage / Environment set up
Acceptance stage  /  Contract test
7s
Acceptance stage / Contract test
Acceptance stage  /  Security test
5s
Acceptance stage / Security test
Acceptance stage  /  UI test
3s
Acceptance stage / UI test
Acceptance stage  /  UI performance test
3s
Acceptance stage / UI performance test
Acceptance stage  /  Integration test
4s
Acceptance stage / Integration test
Acceptance stage  /  Accessibility test
3s
Acceptance stage / Accessibility test
Acceptance stage  /  Load test
3s
Acceptance stage / Load test
Acceptance stage  /  Environment tear down
4s
Acceptance stage / Environment tear down
Fit to window
Zoom out
Zoom in

Annotations

20 warnings
Test stage / Unit tests: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L27
Non-nullable property 'RecipientOrgName' 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/Models/Response/OutboxService/TrackOutboxResponse.cs#L25
Non-nullable property 'RecipientOrgCode' 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/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 / Unit tests: 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 / Unit tests: 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 / Unit tests: 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 / Unit tests: 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 / Unit tests: 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 / Unit tests: 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 / 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 / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshInboxService.cs#L125
Possible null reference return.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshInboxService.cs#L69
Possible null reference return.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOperationService.cs#L73
Possible null reference return.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L195
Possible null reference return.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L151
Possible null reference return.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L137
Possible null reference return.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L107
Possible null reference return.
Test stage / Perform static analysis: src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Services/MeshOutboxService.cs#L67
Possible null reference return.
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:041f6571ac30ffb0edb0218228741f50cd9ad7f702e9d0237a2c4dedd79ab350
lines-of-code-report.json.zip Expired
863 Bytes
sha256:982830d6ad80d6f2bec7f0c6bd91de3c5e9694d243de740a853432624fc570a6
service-layer-svclyr-mesh-ingest-sbom Expired
684 KB
sha256:609713b6a6a75fbb6123da4d5463826bea5e8aab6f976f75f2964d38001c6315
service-layer-svclyr-mesh-ingest-vulnerabilities Expired
2.94 KB
sha256:86842e0f114e5bac52f7906a2d39e81b6b09039baed5168fe5ec2ff1a240321f