remove some unused code #1729
Annotations
95 warnings
test-platforms (macos-latest):
SharedTest/ThemeHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
test-platforms (macos-latest):
SharedTest/LearningElementDifficultyHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
test-platforms (macos-latest):
BackendAccessTest/BackendServices/UserWebApiServicesUt.cs#L298
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
|
test-platforms (macos-latest):
BackendAccessTest/BackendServices/UserWebApiServicesUt.cs#L373
'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#L65
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/Extensions/ZipExtensionsUt.cs#L17
Non-nullable field '_fileSystem' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|
test-platforms (macos-latest):
DataAccessTest/Persistence/ContentFileHandlerUt.cs#L48
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#L147
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/PersistenceCt.cs#L466
Dereference of a possibly null reference.
|
test-platforms (macos-latest):
DataAccessTest/Persistence/PersistenceCt.cs#L467
Dereference of a possibly null reference.
|
test-platforms (ubuntu-20.04):
SharedTest/ThemeHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
test-platforms (ubuntu-20.04):
SharedTest/LearningElementDifficultyHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
test-platforms (ubuntu-20.04):
TestHelpers/PersistEntityProvider.cs#L65
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/UserWebApiServicesUt.cs#L298
'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/UserWebApiServicesUt.cs#L373
'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#L644
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/Extensions/ZipExtensionsUt.cs#L17
Non-nullable field '_fileSystem' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|
test-platforms (ubuntu-20.04):
DataAccessTest/Persistence/ContentFileHandlerUt.cs#L48
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#L147
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/PersistenceCt.cs#L466
Dereference of a possibly null reference.
|
test-platforms (ubuntu-22.04):
SharedTest/ThemeHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
test-platforms (ubuntu-22.04):
SharedTest/LearningElementDifficultyHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
test-platforms (ubuntu-22.04):
BackendAccessTest/BackendServices/UserWebApiServicesUt.cs#L298
'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/UserWebApiServicesUt.cs#L373
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
|
test-platforms (ubuntu-22.04):
TestHelpers/PersistEntityProvider.cs#L65
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):
DataAccessTest/Extensions/ZipExtensionsUt.cs#L17
Non-nullable field '_fileSystem' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|
test-platforms (ubuntu-22.04):
DataAccessTest/Persistence/ContentFileHandlerUt.cs#L48
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#L147
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/PersistenceCt.cs#L466
Dereference of a possibly null reference.
|
test-platforms (ubuntu-22.04):
DataAccessTest/Persistence/PersistenceCt.cs#L467
Dereference of a possibly null reference.
|
build (DataAccessTest):
TestHelpers/PersistEntityProvider.cs#L65
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)'.
|
build (DataAccessTest):
DataAccessTest/Extensions/ZipExtensionsUt.cs#L17
Non-nullable field '_fileSystem' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|
build (DataAccessTest):
DataAccessTest/Persistence/ContentFileHandlerUt.cs#L48
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.
|
build (DataAccessTest):
DataAccessTest/Persistence/ContentFileHandlerUt.cs#L147
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.
|
build (DataAccessTest):
DataAccessTest/Persistence/PersistenceCt.cs#L466
Dereference of a possibly null reference.
|
build (DataAccessTest):
DataAccessTest/Persistence/PersistenceCt.cs#L467
Dereference of a possibly null reference.
|
build (DataAccessTest):
TestHelpers/PersistEntityProvider.cs#L65
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)'.
|
build (DataAccessTest):
DataAccessTest/Extensions/ZipExtensionsUt.cs#L17
Non-nullable field '_fileSystem' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|
build (DataAccessTest):
DataAccessTest/Persistence/ContentFileHandlerUt.cs#L48
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.
|
build (DataAccessTest):
DataAccessTest/Persistence/ContentFileHandlerUt.cs#L147
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.
|
build (SharedTest):
SharedTest/ThemeHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
build (SharedTest):
SharedTest/LearningElementDifficultyHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
build (SharedTest):
SharedTest/ThemeHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
build (SharedTest):
SharedTest/LearningElementDifficultyHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
build (PresentationTest):
TestHelpers/PersistEntityProvider.cs#L65
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)'.
|
build (PresentationTest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L45
Non-nullable property 'StoryEle1' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (PresentationTest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L46
Non-nullable property 'StoryEle2' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (PresentationTest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L47
Non-nullable property 'Ele1' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (PresentationTest):
PresentationTest/Components/ContentFiles/ContentFilesViewUt.cs#L115
Dereference of a possibly null reference.
|
build (PresentationTest):
PresentationTest/PresentationLogic/AuthoringToolWorkspace/AuthoringToolWorkspacePresenterUt.cs#L152
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.
|
build (PresentationTest):
PresentationTest/PresentationLogic/AuthoringToolWorkspace/AuthoringToolWorkspacePresenterUt.cs#L167
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.
|
build (PresentationTest):
PresentationTest/PresentationLogic/AuthoringToolWorkspace/AuthoringToolWorkspacePresenterUt.cs#L185
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.
|
build (PresentationTest):
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 (PresentationTest):
PresentationTest/PresentationLogic/LearningWorld/LearningWorldPresenterUt.cs#L61
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.
|
build (BackendAccessTest):
BackendAccessTest/BackendServices/UserWebApiServicesUt.cs#L298
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
|
build (BackendAccessTest):
BackendAccessTest/BackendServices/UserWebApiServicesUt.cs#L373
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
|
build (BackendAccessTest):
BackendAccessTest/BackendServices/UserWebApiServicesUt.cs#L298
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
|
build (BackendAccessTest):
BackendAccessTest/BackendServices/UserWebApiServicesUt.cs#L373
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
|
build (GeneratorTest):
TestHelpers/PersistEntityProvider.cs#L65
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)'.
|
build (GeneratorTest):
TestHelpers/PersistEntityProvider.cs#L65
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)'.
|
build (BusinessLogicTest):
TestHelpers/PersistEntityProvider.cs#L65
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)'.
|
build (BusinessLogicTest):
BusinessLogicTest/API/BusinessLogicUt.cs#L644
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.
|
build (BusinessLogicTest):
TestHelpers/PersistEntityProvider.cs#L65
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)'.
|
build (BusinessLogicTest):
BusinessLogicTest/API/BusinessLogicUt.cs#L644
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.
|
build (AuthoringToolTest):
TestHelpers/PersistEntityProvider.cs#L65
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)'.
|
build (AuthoringToolTest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L45
Non-nullable property 'StoryEle1' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (AuthoringToolTest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L46
Non-nullable property 'StoryEle2' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (AuthoringToolTest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L47
Non-nullable property 'Ele1' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (AuthoringToolTest):
PresentationTest/Components/ContentFiles/ContentFilesViewUt.cs#L115
Dereference of a possibly null reference.
|
build (AuthoringToolTest):
PresentationTest/Components/Forms/BaseFormUt.cs#L180
Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (AuthoringToolTest):
PresentationTest/Components/Forms/BaseFormUt.cs#L175
Non-nullable property 'Foo' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (AuthoringToolTest):
PresentationTest/Components/Forms/Element/ElementModelHandlerUt.cs#L68
Possible null reference argument for parameter 'path1' in 'string Path.Combine(string path1, string path2, string path3)'.
|
build (AuthoringToolTest):
PresentationTest/Components/Forms/Element/UnplacedElementsDropZoneUt.cs#L78
Non-nullable field '_itemList' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|
build (AuthoringToolTest):
PresentationTest/PresentationLogic/AuthoringToolWorkspace/AuthoringToolWorkspacePresenterUt.cs#L152
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.
|
build (IntegrationTest):
TestHelpers/PersistEntityProvider.cs#L65
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)'.
|
build (IntegrationTest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L45
Non-nullable property 'StoryEle1' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (IntegrationTest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L46
Non-nullable property 'StoryEle2' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (IntegrationTest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L47
Non-nullable property 'Ele1' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build (IntegrationTest):
PresentationTest/TestExtensions.cs#L189
Possible null reference return.
|
build (IntegrationTest):
PresentationTest/PresentationLogic/LearningWorld/LearningWorldPresenterUt.cs#L61
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.
|
build (IntegrationTest):
PresentationTest/PresentationLogic/LearningWorld/LearningWorldPresenterUt.cs#L85
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.
|
build (IntegrationTest):
PresentationTest/PresentationLogic/AuthoringToolWorkspace/AuthoringToolWorkspacePresenterUt.cs#L152
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.
|
build (IntegrationTest):
PresentationTest/PresentationLogic/AuthoringToolWorkspace/AuthoringToolWorkspacePresenterUt.cs#L167
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.
|
build (IntegrationTest):
PresentationTest/PresentationLogic/AuthoringToolWorkspace/AuthoringToolWorkspacePresenterUt.cs#L185
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):
SharedTest/ThemeHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
test-platforms (windows-latest):
SharedTest/LearningElementDifficultyHelperUt.cs#L24
Cannot convert null literal to non-nullable reference type.
|
test-platforms (windows-latest):
BackendAccessTest/BackendServices/UserWebApiServicesUt.cs#L298
'MockedRequestExtensions.Respond(MockedRequest, HttpResponseMessage)' is obsolete: 'Using this overload is not recommended. Instead, use Respond(req => new HttpResponseMessage(...))'
|
test-platforms (windows-latest):
BackendAccessTest/BackendServices/UserWebApiServicesUt.cs#L373
'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#L65
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):
BusinessLogicTest/API/BusinessLogicUt.cs#L644
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/View/LearningWorld/LearningWorldTreeViewUt.cs#L45
Non-nullable property 'StoryEle1' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
test-platforms (windows-latest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L46
Non-nullable property 'StoryEle2' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
test-platforms (windows-latest):
PresentationTest/View/LearningWorld/LearningWorldTreeViewUt.cs#L47
Non-nullable property 'Ele1' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
test-platforms (windows-latest):
PresentationTest/PresentationLogic/LearningWorld/LearningWorldPresenterUt.cs#L61
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.
|
send-summary
The following actions uses Node.js version which is deprecated and will be forced to run on node20: CalmDownVal/webhook-summary@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|