Skip to content

Merge pull request #894 from Project-MONAI/AI-230 #3200

Merge pull request #894 from Project-MONAI/AI-230

Merge pull request #894 from Project-MONAI/AI-230 #3200

Triggered via push October 20, 2023 14:00
Status Success
Total duration 7m 52s
Artifacts 3
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

test.yml

on: push
unit-tests-and-codecov
4m 9s
unit-tests-and-codecov
workflow-executor-integration-tests
6m 11s
workflow-executor-integration-tests
task-manager-integration-tests
3m 8s
task-manager-integration-tests
Fit to window
Zoom out
Zoom in

Annotations

51 warnings
task-manager-integration-tests: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L49
Non-nullable property 'Id' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
task-manager-integration-tests: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/Support/MinioDataSeeding.cs#L40
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
task-manager-integration-tests: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/ArtifactReceivedEventStepDefinitions.cs#L44
Non-nullable property 'WorkflowPublisher' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
task-manager-integration-tests: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/ArtifactReceivedEventStepDefinitions.cs#L59
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
task-manager-integration-tests: tests/UnitTests/WorkflowManager.Tests/DummyMessagingService.cs#L53
The event 'DummyMessagingService.OnConnectionError' is never used
task-manager-integration-tests: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L49
Non-nullable property 'Id' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
unit-tests-and-codecov: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L49
Non-nullable property 'Id' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
unit-tests-and-codecov: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L194
The variable 'ex' is declared but never used
unit-tests-and-codecov: tests/UnitTests/WorkflowManager.Tests/DummyMessagingService.cs#L53
The event 'DummyMessagingService.OnConnectionError' is never used
unit-tests-and-codecov: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L49
Non-nullable property 'Id' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
unit-tests-and-codecov: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L194
The variable 'ex' is declared but never used
unit-tests-and-codecov: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/Support/MinioDataSeeding.cs#L40
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
unit-tests-and-codecov: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/ArtifactReceivedEventStepDefinitions.cs#L44
Non-nullable property 'WorkflowPublisher' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
unit-tests-and-codecov: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/ArtifactReceivedEventStepDefinitions.cs#L59
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
unit-tests-and-codecov: tests/UnitTests/WorkflowManager.Tests/DummyMessagingService.cs#L53
The event 'DummyMessagingService.OnConnectionError' is never used
unit-tests-and-codecov: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L49
Non-nullable property 'Id' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
workflow-executor-integration-tests: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L49
Non-nullable property 'Id' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
workflow-executor-integration-tests: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/Support/MinioDataSeeding.cs#L40
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
workflow-executor-integration-tests: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/ArtifactReceivedEventStepDefinitions.cs#L44
Non-nullable property 'WorkflowPublisher' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
workflow-executor-integration-tests: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/ArtifactReceivedEventStepDefinitions.cs#L59
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
workflow-executor-integration-tests: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L49
Non-nullable property 'Id' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
workflow-executor-integration-tests: tests/UnitTests/WorkflowManager.Tests/DummyMessagingService.cs#L53
The event 'DummyMessagingService.OnConnectionError' is never used
workflow-executor-integration-tests: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/ArtifactReceivedEventStepDefinitions.cs#L44
Non-nullable property 'WorkflowPublisher' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
workflow-executor-integration-tests: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/ArtifactReceivedEventStepDefinitions.cs#L59
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
workflow-executor-integration-tests: tests/IntegrationTests/WorkflowExecutor.IntegrationTests/Support/MinioDataSeeding.cs#L40
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
workflow-executor-integration-tests: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L49
Non-nullable property 'Id' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
sonarscanner: src/WorkflowManager/Contracts/Models/ExecutionStatDTO.cs#L22
Rename class 'ExecutionStatDTO' to match pascal case naming rules, consider using 'ExecutionStatDto'. (https://rules.sonarsource.com/csharp/RSPEC-101)
sonarscanner: src/WorkflowManager/Contracts/Models/ExecutionStatDTO.cs#L33
Remove the member initializer, all constructors set an initial value for the member. (https://rules.sonarsource.com/csharp/RSPEC-3604)
sonarscanner: src/WorkflowManager/Contracts/Models/ExecutionStatDTO.cs#L37
Remove the member initializer, all constructors set an initial value for the member. (https://rules.sonarsource.com/csharp/RSPEC-3604)
sonarscanner: src/WorkflowManager/Contracts/Migrations/M001_ExecutionStats_addVersion.cs#L23
Rename class 'M001_ExecutionStats_addVersion' to match pascal case naming rules, consider using 'M001ExecutionStatsaddVersion'. (https://rules.sonarsource.com/csharp/RSPEC-101)
sonarscanner: src/WorkflowManager/Contracts/Migrations/M001_Payload_addVerion.cs#L22
Rename class 'M001_Payload_addVerion' to match pascal case naming rules, consider using 'M001PayloadaddVerion'. (https://rules.sonarsource.com/csharp/RSPEC-101)
sonarscanner: src/WorkflowManager/Contracts/Migrations/M001_WorkflowInstance_addVerion.cs#L22
Rename class 'M001_WorkflowInstance_addVerion' to match pascal case naming rules, consider using 'M001WorkflowInstanceaddVerion'. (https://rules.sonarsource.com/csharp/RSPEC-101)
sonarscanner: src/WorkflowManager/Contracts/Migrations/M001_WorkflowRevision_addVerion.cs#L22
Rename class 'M001_WorkflowRevision_addVerion' to match pascal case naming rules, consider using 'M001WorkflowRevisionaddVerion'. (https://rules.sonarsource.com/csharp/RSPEC-101)
sonarscanner: src/WorkflowManager/Contracts/Migrations/M002_ExecutionStats_addWorkflowId.cs#L23
Rename class 'M002_ExecutionStats_addWorkflowId' to match pascal case naming rules, consider using 'M002ExecutionStatsaddWorkflowId'. (https://rules.sonarsource.com/csharp/RSPEC-101)
sonarscanner: src/WorkflowManager/Contracts/Migrations/M002_Payload_addPayloadDeleted.cs#L22
Rename class 'M002_Payload_addPayloadDeleted' to match pascal case naming rules, consider using 'M002PayloadaddPayloadDeleted'. (https://rules.sonarsource.com/csharp/RSPEC-101)
sonarscanner: src/WorkflowManager/Contracts/Migrations/M003_Payload_addDataTrigger.cs#L24
Rename class 'M003_Payload_addDataTrigger' to match pascal case naming rules, consider using 'M003PayloadaddDataTrigger'. (https://rules.sonarsource.com/csharp/RSPEC-101)

Artifacts

Produced during runtime
Name Size
TaskManagerIntegrationTestReport Expired
1.69 MB
WorkflowExecutorIntegrationTestReport Expired
2.65 MB
code-coverage-reports Expired
17.7 MB