feat: Rewrite use of TestContainers #16
Triggered via pull request
September 27, 2024 08:56
Status
Success
Total duration
1m 14s
Artifacts
–
build-and-test.yaml
on: pull_request
build-and-test
1m 6s
Annotations
11 warnings
build-and-test
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
build-and-test:
src/Application/Weather/GetForecast/GetForecastQueryHandler.cs#L15
Dereference of a possibly null reference.
|
build-and-test:
src/Infrastructure/Repositories/WeatherRepository.cs#L16
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-and-test:
src/Api/Routes/Weather/Models/GetWeatherResponse.cs#L5
Non-nullable property 'Forecast' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build-and-test:
tests/Api.E2E/ForecastTests.cs#L163
Dereference of a possibly null reference.
|
build-and-test:
tests/Api.E2E/ForecastTests.cs#L189
Dereference of a possibly null reference.
|
build-and-test:
tests/Api.E2E/ForecastTests.cs#L166
Public method 'DeleteForecast_AsUser_ShouldReturnForbidden' on test class 'ForecastTests' 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)
|
build-and-test:
src/Application/Weather/GetForecast/GetForecastQueryHandler.cs#L15
Dereference of a possibly null reference.
|
build-and-test:
src/Infrastructure/Repositories/WeatherRepository.cs#L16
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-and-test:
src/Api/Routes/Weather/Models/GetWeatherResponse.cs#L5
Non-nullable property 'Forecast' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build-and-test:
tests/Api.E2E/ForecastTests.cs#L163
Dereference of a possibly null reference.
|