Skip to content

Fixed #371 and simplified cognitive complexity of DragItemToUnplaced … #1095

Fixed #371 and simplified cognitive complexity of DragItemToUnplaced …

Fixed #371 and simplified cognitive complexity of DragItemToUnplaced … #1095

Triggered via push September 8, 2023 15:14
Status Failure
Total duration 5m 32s
Artifacts

coveralls.yaml

on: push
Matrix: build
Matrix: test-platforms
send-summary
4s
send-summary
Fit to window
Zoom out
Zoom in

Annotations

8 errors and 52 warnings
build (SharedTest)
Process completed with exit code 1.
build (DataAccessTest)
The operation was canceled.
build (GeneratorTest)
The operation was canceled.
build (IntegrationTest)
The operation was canceled.
build (BusinessLogicTest)
The operation was canceled.
build (PresentationTest)
The operation was canceled.
build (AuthoringToolTest)
The operation was canceled.
build (BackendAccessTest)
The operation was canceled.
test-platforms (ubuntu-20.04)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test-platforms (ubuntu-20.04): TestHelpers/PersistEntityProvider.cs#L35
Possible null reference argument for parameter 'learningContent' in 'LearningElementPe.LearningElementPe(string name, ILearningContentPe learningContent, string description, string goals, LearningElementDifficultyEnum difficulty, ElementModel elementModel, int workload = 0, int points = 1, double positionX = 0, double positionY = 0)'.
test-platforms (ubuntu-20.04): BackendAccessTest/BackendServices/UserBackendServicesUt.cs#L233
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
test-platforms (ubuntu-20.04): BackendAccessTest/BackendServices/UserBackendServicesUt.cs#L263
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
test-platforms (ubuntu-20.04): BusinessLogicTest/API/BusinessLogicUt.cs#L818
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
test-platforms (ubuntu-20.04): DataAccessTest/Persistence/ContentFileHandlerUt.cs#L45
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.
test-platforms (ubuntu-20.04): DataAccessTest/Persistence/ContentFileHandlerUt.cs#L140
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.
test-platforms (ubuntu-20.04): PresentationTest/TestExtensions.cs#L189
Possible null reference return.
test-platforms (ubuntu-20.04): PresentationTest/Components/Forms/BaseFormUt.cs#L170
Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
test-platforms (ubuntu-20.04): PresentationTest/Components/Forms/BaseFormUt.cs#L165
Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
test-platforms (ubuntu-20.04): PresentationTest/PresentationLogic/AuthoringToolWorkspace/AuthoringToolWorkspacePresenterUt.cs#L20
Non-nullable field '_authoringToolWorkspaceViewModel' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
build (SharedTest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test-platforms (ubuntu-22.04)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test-platforms (ubuntu-22.04): TestHelpers/PersistEntityProvider.cs#L35
Possible null reference argument for parameter 'learningContent' in 'LearningElementPe.LearningElementPe(string name, ILearningContentPe learningContent, string description, string goals, LearningElementDifficultyEnum difficulty, ElementModel elementModel, int workload = 0, int points = 1, double positionX = 0, double positionY = 0)'.
test-platforms (ubuntu-22.04): BackendAccessTest/BackendServices/UserBackendServicesUt.cs#L233
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
test-platforms (ubuntu-22.04): BackendAccessTest/BackendServices/UserBackendServicesUt.cs#L263
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
test-platforms (ubuntu-22.04): DataAccessTest/Persistence/ContentFileHandlerUt.cs#L45
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.
test-platforms (ubuntu-22.04): DataAccessTest/Persistence/ContentFileHandlerUt.cs#L140
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.
test-platforms (ubuntu-22.04): BusinessLogicTest/API/BusinessLogicUt.cs#L818
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
test-platforms (ubuntu-22.04): PresentationTest/Components/Forms/BaseFormUt.cs#L170
Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
test-platforms (ubuntu-22.04): PresentationTest/Components/Forms/BaseFormUt.cs#L165
Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
test-platforms (ubuntu-22.04): PresentationTest/TestExtensions.cs#L189
Possible null reference return.
build (DataAccessTest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (GeneratorTest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (IntegrationTest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (BusinessLogicTest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (PresentationTest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (AuthoringToolTest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (BackendAccessTest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test-platforms (macos-latest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test-platforms (macos-latest): BackendAccessTest/BackendServices/UserBackendServicesUt.cs#L233
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
test-platforms (macos-latest): BackendAccessTest/BackendServices/UserBackendServicesUt.cs#L263
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
test-platforms (macos-latest): TestHelpers/PersistEntityProvider.cs#L35
Possible null reference argument for parameter 'learningContent' in 'LearningElementPe.LearningElementPe(string name, ILearningContentPe learningContent, string description, string goals, LearningElementDifficultyEnum difficulty, ElementModel elementModel, int workload = 0, int points = 1, double positionX = 0, double positionY = 0)'.
test-platforms (macos-latest): DataAccessTest/Persistence/ContentFileHandlerUt.cs#L45
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.
test-platforms (macos-latest): DataAccessTest/Persistence/ContentFileHandlerUt.cs#L140
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.
test-platforms (macos-latest): BusinessLogicTest/API/BusinessLogicUt.cs#L818
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
test-platforms (macos-latest): PresentationTest/TestExtensions.cs#L189
Possible null reference return.
test-platforms (macos-latest): PresentationTest/PresentationLogic/AuthoringToolWorkspace/AuthoringToolWorkspacePresenterUt.cs#L20
Non-nullable field '_authoringToolWorkspaceViewModel' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
test-platforms (macos-latest): PresentationTest/Components/Forms/BaseFormUt.cs#L170
Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
test-platforms (windows-latest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test-platforms (windows-latest): BackendAccessTest/BackendServices/UserBackendServicesUt.cs#L233
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
test-platforms (windows-latest): BackendAccessTest/BackendServices/UserBackendServicesUt.cs#L263
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
test-platforms (windows-latest): TestHelpers/PersistEntityProvider.cs#L35
Possible null reference argument for parameter 'learningContent' in 'LearningElementPe.LearningElementPe(string name, ILearningContentPe learningContent, string description, string goals, LearningElementDifficultyEnum difficulty, ElementModel elementModel, int workload = 0, int points = 1, double positionX = 0, double positionY = 0)'.
test-platforms (windows-latest): DataAccessTest/Persistence/ContentFileHandlerUt.cs#L45
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.
test-platforms (windows-latest): DataAccessTest/Persistence/ContentFileHandlerUt.cs#L140
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.
test-platforms (windows-latest): BusinessLogicTest/API/BusinessLogicUt.cs#L818
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
test-platforms (windows-latest): PresentationTest/Components/Forms/BaseFormUt.cs#L170
Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
test-platforms (windows-latest): PresentationTest/Components/Forms/BaseFormUt.cs#L165
Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
test-platforms (windows-latest): PresentationTest/TestExtensions.cs#L189
Possible null reference return.
test-platforms (windows-latest): PresentationTest/PresentationLogic/LearningWorld/LearningWorldPresenterUt.cs#L26
Non-nullable field '_authoringToolWorkspaceViewModel' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.