Nds improve logging #3344
neildsouth
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
test.yml
on: pull_request
unit-tests-and-codecov
3m 36s
workflow-executor-integration-tests
5m 2s
task-manager-integration-tests
1m 59s
sonarscanner
2m 10s
Annotations
35 warnings
|
task-manager-integration-tests
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-dotnet@v3, actions/[email protected], actions/[email protected]. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
task-manager-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
task-manager-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
task-manager-integration-tests:
tests/UnitTests/WorkflowExecuter.Tests/Services/WorkflowExecuterServiceTests.cs#L3707
Public method 'ProcessPayload_With_Multiple_Taskdestinations_One_Has_Inputs' on test class 'WorkflowExecuterServiceTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
|
task-manager-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
task-manager-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
task-manager-integration-tests:
tests/UnitTests/WorkflowExecuter.Tests/Services/WorkflowExecuterServiceTests.cs#L3707
Public method 'ProcessPayload_With_Multiple_Taskdestinations_One_Has_Inputs' on test class 'WorkflowExecuterServiceTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
|
unit-tests-and-codecov
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-dotnet@v3, actions/[email protected], actions/[email protected]. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
unit-tests-and-codecov:
tests/UnitTests/WorkflowExecuter.Tests/Services/WorkflowExecuterServiceTests.cs#L3707
Public method 'ProcessPayload_With_Multiple_Taskdestinations_One_Has_Inputs' on test class 'WorkflowExecuterServiceTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
|
unit-tests-and-codecov:
tests/UnitTests/WorkflowExecuter.Tests/Services/WorkflowExecuterServiceTests.cs#L3707
Public method 'ProcessPayload_With_Multiple_Taskdestinations_One_Has_Inputs' on test class 'WorkflowExecuterServiceTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
|
unit-tests-and-codecov:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
unit-tests-and-codecov:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
unit-tests-and-codecov:
tests/UnitTests/WorkflowExecuter.Tests/Services/WorkflowExecuterServiceTests.cs#L3707
Public method 'ProcessPayload_With_Multiple_Taskdestinations_One_Has_Inputs' on test class 'WorkflowExecuterServiceTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
|
unit-tests-and-codecov:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
unit-tests-and-codecov:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
workflow-executor-integration-tests
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-dotnet@v3, actions/[email protected], actions/[email protected]. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
workflow-executor-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
workflow-executor-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
workflow-executor-integration-tests:
tests/UnitTests/WorkflowExecuter.Tests/Services/WorkflowExecuterServiceTests.cs#L3707
Public method 'ProcessPayload_With_Multiple_Taskdestinations_One_Has_Inputs' on test class 'WorkflowExecuterServiceTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
|
workflow-executor-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
workflow-executor-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
workflow-executor-integration-tests:
tests/UnitTests/WorkflowExecuter.Tests/Services/WorkflowExecuterServiceTests.cs#L3707
Public method 'ProcessPayload_With_Multiple_Taskdestinations_One_Has_Inputs' on test class 'WorkflowExecuterServiceTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
|
workflow-executor-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
workflow-executor-integration-tests:
tests/IntegrationTests/WorkflowExecutor.IntegrationTests/StepDefinitions/TasksApiStepDefinitions.cs#L52
Dereference of a possibly null reference.
|
|
sonarscanner
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-java@v3, actions/checkout@v3, actions/setup-dotnet@v3, actions/[email protected], actions/download-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
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/Extensions/TypeExtensions.cs#L26
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
|
sonarscanner:
src/TaskManager/API/Extensions/TypeExtensions.cs#L27
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
|
sonarscanner:
src/TaskManager/API/Extensions/TypeExtensions.cs#L34
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
|
sonarscanner:
src/TaskManager/API/Extensions/TypeExtensions.cs#L35
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
|
sonarscanner:
src/TaskManager/API/Extensions/TypeExtensions.cs#L36
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
|
sonarscanner:
src/TaskManager/API/Models/TaskDispatchEventInfo.cs#L73
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
|
sonarscanner:
src/TaskManager/API/ServiceNotFoundException.cs#L26
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
TaskManagerIntegrationTestReport
Expired
|
1.69 MB |
|
|
WorkflowExecutorIntegrationTestReport
Expired
|
2.65 MB |
|
|
code-coverage-reports
Expired
|
17.6 MB |
|