refactor: Clean up some imports and miscellaneous (#2)

global.json

@@ -1,7 +1,7 @@
 {
   "sdk": {
     "version": "8.0.201",
-    "rollForward": "latestPatch",
+    "rollForward": "latestFeature",
     "allowPrerelease": true
   }
 }

src/Api/Routes/Weather/Endpoints/DeleteWeather.cs

@@ -1,8 +1,6 @@
-using Api.Routes.Weather.Models;
-using Application.Weather.AddForecast;
+using Application.Weather.DeleteForecast;
 using MediatR;
 using Microsoft.AspNetCore.Http.HttpResults;
-using Microsoft.AspNetCore.Mvc;
 
 namespace Api.Routes.Weather.Endpoints;
 

src/Api/Routes/Weather/Endpoints/GetWeather.cs

@@ -2,7 +2,6 @@
 using Application.Weather.GetForecast;
 using MediatR;
 using Microsoft.AspNetCore.Http.HttpResults;
-using Microsoft.AspNetCore.Mvc;
 
 namespace Api.Routes.Weather.Endpoints;
 

src/Api/Routes/Weather/WeatherAdminGroup.cs

@@ -1,4 +1,3 @@
-using Api.Authorization;
 using Api.Routes.Weather.Endpoints;
 
 namespace Api.Routes.Weather;

src/Api/setup-jwts.sh

@@ -1,4 +1,7 @@
-dotnet user-jwts clear
+# Warning: This will clear all JWT tokens that already exists!
+dotnet user-jwts clear --force
 
+# Create JWT token with the "write" and "read" scopes
 dotnet user-jwts create --audience weather.dev.api --scope "write" --scope "read"
+# Create JWT token with the "admin" scope
 dotnet user-jwts create --audience weather.dev.api --scope "admin"

src/Application/Weather/DeleteForecast/DeleteForecastCommand.cs

@@ -1,6 +1,6 @@
 using FluentResults;
 using MediatR;
 
-namespace Application.Weather.AddForecast;
+namespace Application.Weather.DeleteForecast;
 
 public record DeleteForecastCommand(DateOnly Date) : IRequest<Result>;

src/Application/Weather/DeleteForecast/DeleteForecastCommandHandler.cs

@@ -1,8 +1,7 @@
-using Domain;
 using FluentResults;
 using MediatR;
 
-namespace Application.Weather.AddForecast;
+namespace Application.Weather.DeleteForecast;
 
 internal class DeleteForecastCommandHandler(IWeatherRepository weatherRepository, IUnitOfWork unitOfWork)
     : IRequestHandler<DeleteForecastCommand, Result>

src/Infrastructure/Repositories/WeatherRepository.cs

@@ -15,6 +15,6 @@ public class WeatherRepository(DatabaseContext context) : IWeatherRepository
 
     public async Task AddForecastAsync(Forecast forecast)
     {
-        context.Forecasts.Add(forecast);
+        await context.Forecasts.AddAsync(forecast);
     }
 }

src/Infrastructure/TestContainers/TestContainersFactory.cs

@@ -1,6 +1,5 @@
 using Microsoft.EntityFrameworkCore;
 using Microsoft.Extensions.Logging;
-using Microsoft.Extensions.Logging.Abstractions;
 using Microsoft.Extensions.Options;
 using Testcontainers.PostgreSql;
 

tests/Api.E2E/ForecastTests.cs

@@ -163,6 +163,7 @@ public async Task DeleteForecast_AsAdmin_ShouldReturnNoContent()
         dbForecast.IsDeleted.Should().BeTrue();
     }
 
+    [Fact]
     public async Task DeleteForecast_AsUser_ShouldReturnForbidden()
     {
         // Arrange

tests/Api.E2E/Shared/ApiFactory.cs

@@ -6,7 +6,6 @@
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.AspNetCore.Mvc.Testing;
 using Microsoft.AspNetCore.TestHost;
-using Microsoft.EntityFrameworkCore;
 using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.DependencyInjection.Extensions;
@@ -17,7 +16,6 @@
 using Npgsql;
 using Respawn;
 using Respawn.Graph;
-using Testcontainers.PostgreSql;
 
 namespace Api.E2E.Shared;