Skip to content

Bump Microsoft.AspNetCore.Mvc.Testing from 6.0.22 to 6.0.25 #3296

Bump Microsoft.AspNetCore.Mvc.Testing from 6.0.22 to 6.0.25

Bump Microsoft.AspNetCore.Mvc.Testing from 6.0.22 to 6.0.25 #3296

Triggered via pull request December 15, 2023 10:16
Status Failure
Total duration 5m 28s
Artifacts 3

test.yml

on: pull_request
unit-tests-and-codecov
3m 32s
unit-tests-and-codecov
workflow-executor-integration-tests
5m 19s
workflow-executor-integration-tests
task-manager-integration-tests
2m 5s
task-manager-integration-tests
Fit to window
Zoom out
Zoom in

Annotations

1 error and 52 warnings
sonarscanner
Process completed with exit code 1.
task-manager-integration-tests: src/WorkflowManager/Logging/Log.200000.Workflow.cs#L112
Multiple logging methods are using event id 200018 in class Log
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
unit-tests-and-codecov: src/WorkflowManager/Logging/Log.200000.Workflow.cs#L112
Multiple logging methods are using event id 200018 in class Log
unit-tests-and-codecov: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L193
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/Logging/Log.200000.Workflow.cs#L112
Multiple logging methods are using event id 200018 in class Log
unit-tests-and-codecov: src/WorkflowManager/Database/Repositories/ArtifactsRepository.cs#L193
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
sonarscanner: src/TaskManager/API/Migrations/M001_TaskDispatchEventInfo_addVerion.cs#L22
Rename class 'M001_TaskDispatchEventInfo_addVerion' to match pascal case naming rules, consider using 'M001TaskDispatchEventInfoaddVerion'. (https://rules.sonarsource.com/csharp/RSPEC-101)
sonarscanner: src/TaskManager/API/Migrations/M001_TaskDispatchEventInfo_addVerion.cs#L36
Handle the exception or explain in a comment why it can be ignored. (https://rules.sonarsource.com/csharp/RSPEC-2486)
sonarscanner: src/TaskManager/API/Migrations/M001_TaskDispatchEventInfo_addVerion.cs#L37
Either remove or fill this block of code. (https://rules.sonarsource.com/csharp/RSPEC-108)
sonarscanner: src/TaskManager/API/Migrations/DocumentVersionConvert.cs#L32
Remove this array creation and simply pass the elements. (https://rules.sonarsource.com/csharp/RSPEC-3878)
sonarscanner: src/TaskManager/API/Extensions/TypeExtensions.cs#L58
Replace this call to 'Assembly.LoadFile' with 'Assembly.Load'. (https://rules.sonarsource.com/csharp/RSPEC-3885)
sonarscanner: src/TaskManager/API/Extensions/TypeExtensions.cs#L54
"Find" method should be used instead of the "FirstOrDefault" extension method. (https://rules.sonarsource.com/csharp/RSPEC-6602)
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/Models/ExecutionStatDTO.cs#L22
Rename class 'ExecutionStatDTO' to match pascal case naming rules, consider using 'ExecutionStatDto'. (https://rules.sonarsource.com/csharp/RSPEC-101)
workflow-executor-integration-tests: src/WorkflowManager/Logging/Log.200000.Workflow.cs#L112
Multiple logging methods are using event id 200018 in class Log
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/Logging/Log.200000.Workflow.cs#L112
Multiple logging methods are using event id 200018 in class Log
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: tests/UnitTests/WorkflowManager.Tests/DummyMessagingService.cs#L53
The event 'DummyMessagingService.OnConnectionError' is never used

Artifacts

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