Add MenuUser entity, EF Core configuration, and migration

.github/workflows/main.yml

@@ -126,7 +126,11 @@ jobs:
         working-directory: ./backend
 
       - name: Test with the dotnet CLI
-        run: dotnet test --project MenuApi.Tests/MenuApi.Tests.csproj --configuration Release --no-build
+        run: |
+          Get-ChildItem -Recurse -Filter "*.Tests.csproj" | Where-Object { $_.FullName -notlike "*Integration*" } | ForEach-Object {
+            dotnet test $_.FullName --configuration Release --no-build
+          }
+        shell: pwsh
         working-directory: ./backend
 
 

backend/MenuApi.Integration.Tests/Factory/ApiAuthentication.cs

@@ -20,24 +20,24 @@ public ApiAuthentication()
             .AddUserSecrets<ApiTestFixture>()                     // User secrets for local development.
             .Build();
 
-        config = Configuration.GetRequiredSection("Parameters").Get<TestParameters>();
+        config = Configuration.GetRequiredSection("Parameters").Get<TestParameters>()!;
     }
 
 
     public class Auth0AuthenticationRequest
     {
-        public string client_id { get; set; }
-        public string client_secret { get; set; }
-        public string audience { get; set; }
-        public string grant_type { get; set; }
+        public string client_id { get; set; } = null!;
+        public string client_secret { get; set; } = null!;
+        public string audience { get; set; } = null!;
+        public string grant_type { get; set; } = null!;
     }
 
 
 
     public class Auth0AuthenticationResponse
     {
-        public string access_token { get; set; }
-        public string token_type { get; set; }
+        public string access_token { get; set; } = null!;
+        public string token_type { get; set; } = null!;
     }
 
 
@@ -58,6 +58,6 @@ public async Task<AuthenticationHeaderValue> GetAuthenticationHeaderValue()
         authResponse.EnsureSuccessStatusCode();
         var responseBody = await authResponse.Content.ReadFromJsonAsync<Auth0AuthenticationResponse>();
 
-        return new AuthenticationHeaderValue(responseBody.token_type, responseBody.access_token);
+        return new AuthenticationHeaderValue(responseBody!.token_type, responseBody.access_token);
     }
 }

backend/MenuApi.Integration.Tests/Factory/ApiTestFixture.cs

@@ -11,9 +11,9 @@ namespace MenuApi.Integration.Tests.Factory;
 
 public class ApiTestFixture : IAsyncLifetime
 {
-    public DistributedApplication app { get; private set; }
-    private IDistributedApplicationTestingBuilder appHost;
-    private AuthenticationHeaderValue cachedAuthHeader;
+    public DistributedApplication app { get; private set; } = null!;
+    private IDistributedApplicationTestingBuilder appHost = null!;
+    private AuthenticationHeaderValue cachedAuthHeader = null!;
 
     public async Task<HttpClient> GetHttpClient()
     {
@@ -48,7 +48,7 @@ async ValueTask IAsyncLifetime.InitializeAsync()
         // Retry app startup to handle Docker daemon health checks in CI environments
         const int maxRetries = 3;
         const int delayMs = 2000;
-        Exception lastException = null;
+        Exception? lastException = null;
         for (int attempt = 1; attempt <= maxRetries; attempt++)
         {
             try

backend/MenuApi.Integration.Tests/Factory/TestParameters.cs

@@ -2,9 +2,9 @@
 {
     internal class TestParameters
     {
-        public string Auth0TestClientId { get; set; }
-        public string Auth0TestClientSecret { get; set; }
-        public string Auth0Audience { get; set; }
-        public string Auth0Domain { get; set; }
+        public string Auth0TestClientId { get; set; } = null!;
+        public string Auth0TestClientSecret { get; set; } = null!;
+        public string Auth0Audience { get; set; } = null!;
+        public string Auth0Domain { get; set; } = null!;
     }
 }

backend/MenuApi.Integration.Tests/IngredientIntegrationTests.cs

@@ -162,24 +162,24 @@ private class Ingredient
     {
 #pragma warning disable S1144 // Unused private types or members should be removed
         public int Id { get; set; }
-        public string Name { get; set; }
-        public List<IngredientUnit> Units { get; set; }
+        public string Name { get; set; } = null!;
+        public List<IngredientUnit> Units { get; set; } = null!;
 #pragma warning restore S1144 // Unused private types or members should be removed
     }
 
     public class IngredientUnit
     {
 #pragma warning disable S1144 // Unused private types or members should be removed
-        public string Name { get; set; }
-        public string Abbreviation { get; set; }
-        public string Type { get; set; }
+        public string Name { get; set; } = null!;
+        public string? Abbreviation { get; set; }
+        public string Type { get; set; } = null!;
 #pragma warning restore S1144 // Unused private types or members should be removed
     }
 
     public class NewIngredient
     {
-        public string Name { get; set; }
-        public List<int> UnitIds { get; set; }
+        public string Name { get; set; } = null!;
+        public List<int> UnitIds { get; set; } = null!;
     }
 }
 

backend/MenuApi.Integration.Tests/MenuApi.Integration.Tests.csproj

@@ -9,6 +9,7 @@
 		<UserSecretsId>ea3aa4c7-9b32-4485-a5b2-fb1cb0def863</UserSecretsId>
 
 		<ImplicitUsings>enable</ImplicitUsings>
+		<Nullable>enable</Nullable>
 		<NoWarn>xUnit1051</NoWarn>
 	</PropertyGroup>
 

backend/MenuApi.Integration.Tests/RecipeIntegrationTests.cs

@@ -218,7 +218,7 @@ private static (int Id, string Name) GetRecipeFromJson(JsonDocument doc)
         var rootElement = doc.RootElement;
         return (
             rootElement.GetProperty("id").GetInt32(),
-            rootElement.GetProperty("name").GetString()
+            rootElement.GetProperty("name").GetString()!
         );
     }
 
@@ -227,22 +227,22 @@ private class Recipe
 #pragma warning disable S1144 // Unused private types or members should be removed
         public int Id { get; set; }
 
-        public string Name { get; set; }
+        public string Name { get; set; } = null!;
 #pragma warning restore S1144 // Unused private types or members should be removed
     }
 
     public class NewRecipe
     {
         public List<RecipeIngredient> Ingredients { get; set; } = [];
 
-        public string Name { get; set; }
+        public string Name { get; set; } = null!;
     }
 
     public class RecipeIngredient
     {
-        public string Name { get; set; }
+        public string Name { get; set; } = null!;
 
-        public string Unit { get; set; }
+        public string Unit { get; set; } = null!;
 
         public decimal Amount { get; set; }
     }

backend/MenuApi.Integration.Tests/RecipeWithIngredientsIntegrationTests.cs

@@ -351,24 +351,24 @@ private async Task PostIngredientAsync(HttpClient client, string name, List<int>
     private class NewIngredient
     {
 #pragma warning disable S1144 // Unused private types or members should be removed
-        public string Name { get; set; }
-        public List<int> UnitIds { get; set; }
+        public string Name { get; set; } = null!;
+        public List<int> UnitIds { get; set; } = null!;
 #pragma warning restore S1144 // Unused private types or members should be removed
     }
 
     public class NewRecipe
     {
 #pragma warning disable S1144 // Unused private types or members should be removed
-        public string Name { get; set; }
+        public string Name { get; set; } = null!;
         public List<RecipeIngredient> Ingredients { get; set; } = [];
 #pragma warning restore S1144 // Unused private types or members should be removed
     }
 
     public class RecipeIngredient
     {
 #pragma warning disable S1144 // Unused private types or members should be removed
-        public string Name { get; set; }
-        public string Unit { get; set; }
+        public string Name { get; set; } = null!;
+        public string Unit { get; set; } = null!;
         public decimal Amount { get; set; }
 #pragma warning restore S1144 // Unused private types or members should be removed
     }

backend/MenuApi.Integration.Tests/ValidationIntegrationTests.cs

@@ -166,19 +166,19 @@ private async Task PostIngredientAsync(HttpClient client, string name)
     public class NewRecipe
     {
         public List<RecipeIngredient> Ingredients { get; set; } = [];
-        public string Name { get; set; }
+        public string Name { get; set; } = null!;
     }
 
     public class RecipeIngredient
     {
-        public string Name { get; set; }
-        public string Unit { get; set; }
+        public string Name { get; set; } = null!;
+        public string Unit { get; set; } = null!;
         public decimal Amount { get; set; }
     }
 
     public class NewIngredient
     {
-        public string Name { get; set; }
+        public string Name { get; set; } = null!;
         public List<int> UnitIds { get; set; } = [];
     }
 }

backend/MenuApi.Tests/CustomGenerator.cs

@@ -12,7 +12,7 @@ public object Create(object request, ISpecimenContext context)
         {
             var value = context.Resolve(GetUnderlyingType(type));
 
-            return type.GetMethod("From").Invoke(null, [value]);
+            return type.GetMethod("From")!.Invoke(null, [value])!;
         }
 
         return new NoSpecimen();

backend/MenuApi.Tests/MenuApi.Tests.csproj

@@ -8,6 +8,7 @@
 		<IsPackable>false</IsPackable>
 
 		<ImplicitUsings>enable</ImplicitUsings>
+		<Nullable>enable</Nullable>
 	</PropertyGroup>
 
 	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">

backend/MenuApi.Tests/Services/RecipeServiceTests.cs

@@ -45,7 +45,7 @@ public async Task GetRecipeSuccess(DBModel.Recipe recipe, IEnumerable<DBModel.Ge
 
         var result = await sut.GetRecipeAsync(recipe.Id);
 
-        result.Name.Should().Be(recipe.Name);
+        result!.Name.Should().Be(recipe.Name);
         result.Id.Should().Be(recipe.Id);
         result.Ingredients.Should().BeEquivalentTo(expected);
     }
@@ -185,7 +185,7 @@ public async Task UpdateRecipeAsync_Deduplicates_Exact_Duplicate_Ingredients_Bef
     [Theory, CustomAutoData]
     public async Task UpdateRecipe_Should_Throw_Exception_For_null_newRecipeAsync(RecipeId recipeId)
     {
-        Func<Task> fun = () => sut.UpdateRecipeAsync(recipeId, null);
+        Func<Task> fun = () => sut.UpdateRecipeAsync(recipeId, null!);
 
         var result = await fun.Should().ThrowAsync<ArgumentNullException>();
         result.And.ParamName.Should().Be("newRecipe");

backend/MenuApi.sln

@@ -19,6 +19,8 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "MenuDB", "MenuDB\MenuDB.csp
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Menu.MigrationService", "Menu.MigrationService\Menu.MigrationService.csproj", "{678A54EA-9237-49D6-B4DC-ACEF67F6385E}"
 EndProject
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "MenuDB.Tests", "MenuDB.Tests\MenuDB.Tests.csproj", "{E780A213-994B-4759-9FC6-DB589E8C0DE2}"
+EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU
@@ -125,6 +127,18 @@ Global
 		{678A54EA-9237-49D6-B4DC-ACEF67F6385E}.Release|x64.Build.0 = Release|Any CPU
 		{678A54EA-9237-49D6-B4DC-ACEF67F6385E}.Release|x86.ActiveCfg = Release|Any CPU
 		{678A54EA-9237-49D6-B4DC-ACEF67F6385E}.Release|x86.Build.0 = Release|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Debug|x64.ActiveCfg = Debug|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Debug|x64.Build.0 = Debug|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Debug|x86.ActiveCfg = Debug|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Debug|x86.Build.0 = Debug|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Release|Any CPU.Build.0 = Release|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Release|x64.ActiveCfg = Release|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Release|x64.Build.0 = Release|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Release|x86.ActiveCfg = Release|Any CPU
+		{E780A213-994B-4759-9FC6-DB589E8C0DE2}.Release|x86.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE

backend/MenuApi/ValueObjects/MenuUserId.cs

@@ -0,0 +1,6 @@
+using Vogen;
+
+namespace MenuApi.ValueObjects;
+
+[ValueObject<int>]
+public readonly partial struct MenuUserId { }

backend/MenuDB.Tests/MenuDB.Tests.csproj

@@ -0,0 +1,38 @@
+<Project Sdk="Microsoft.NET.Sdk">
+
+	<PropertyGroup>
+		<TargetFramework>net10.0</TargetFramework>
+		<OutputType>Exe</OutputType>
+		<ImplicitUsings>enable</ImplicitUsings>
+		<Nullable>enable</Nullable>
+	</PropertyGroup>
+
+	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
+		<TreatWarningsAsErrors>True</TreatWarningsAsErrors>
+	</PropertyGroup>
+
+	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|AnyCPU'">
+		<TreatWarningsAsErrors>True</TreatWarningsAsErrors>
+	</PropertyGroup>
+
+	<ItemGroup>
+		<PackageReference Include="Microsoft.EntityFrameworkCore.InMemory" Version="10.0.8" />
+		<PackageReference Include="Microsoft.Testing.Platform" Version="2.2.3" />
+		<PackageReference Include="xunit.v3.mtp-v2" Version="3.2.2" />
+		<PackageReference Include="AwesomeAssertions" Version="9.4.0" />
+		<PackageReference Include="AwesomeAssertions.Analyzers" Version="9.0.8">
+			<PrivateAssets>all</PrivateAssets>
+			<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+		</PackageReference>
+		<PackageReference Include="GitHubActionsTestLogger" Version="3.0.4" />
+		<PackageReference Include="coverlet.collector" Version="10.0.1">
+			<PrivateAssets>all</PrivateAssets>
+			<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+		</PackageReference>
+	</ItemGroup>
+
+	<ItemGroup>
+		<ProjectReference Include="..\MenuDB\MenuDB.csproj" />
+	</ItemGroup>
+
+</Project>