feat: Tasks have an Id

Author: Xiphereal

Backend.Tests/TestAPI/TaskFactory.cs

@@ -9,8 +9,8 @@ public static Task AnyBackendTask()
         return BackendTask(named: "Any");
     }
 
-    public static Task BackendTask(string named)
+    public static Task BackendTask(string named, string description = "Any")
     {
-        return new Task(named, "Any");
+        return new Task(named, description);
     }
 }

Backend.Tests/UnitTests.cs

@@ -0,0 +1,14 @@
+using Backend.Domain;
+using FluentAssertions;
+using NUnit.Framework;
+
+namespace Backend.Tests;
+
+public class UnitTests
+{
+    [Test]
+    public void TasksHaveAnAutogeneratedId()
+    {
+        new Task("Any", "Any").Id.Should().NotBeEmpty();
+    }
+}

Backend/Backend/Domain/Task.cs

@@ -1,3 +1,27 @@
 namespace Backend.Domain;
 
-public record Task(string Name, string Description);
+public record Task
+{
+    public Task()
+    {
+    }
+
+    public Task(string name, string description)
+    {
+        Id = Guid.NewGuid();
+        Name = name;
+        Description = description;
+    }
+
+    public Task(Guid id, string name, string description)
+    {
+        Id = id;
+        Name = name;
+        Description = description;
+    }
+
+    public Guid Id { get; init; }
+
+    public string Name { get; init; }
+    public string Description { get; init; }
+}

Backend/Backend/Persistence/Migrations/20250627173338_TasksHaveAnId.Designer.cs

@@ -0,0 +1,30 @@
+using System;
+using Microsoft.EntityFrameworkCore.Migrations;
+
+#nullable disable
+
+namespace Backend.Persistence.Migrations
+{
+    /// <inheritdoc />
+    public partial class TasksHaveAnId : Migration
+    {
+        /// <inheritdoc />
+        protected override void Up(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.AddColumn<Guid>(
+                name: "Id",
+                table: "Tasks",
+                type: "TEXT",
+                nullable: false,
+                defaultValue: new Guid("00000000-0000-0000-0000-000000000000"));
+        }
+
+        /// <inheritdoc />
+        protected override void Down(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.DropColumn(
+                name: "Id",
+                table: "Tasks");
+        }
+    }
+}

Backend/Backend/Persistence/Migrations/20250627173338_TasksHaveAnId.cs

@@ -1,4 +1,5 @@
 // <auto-generated />
+using System;
 using Backend.Persistence;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.Infrastructure;
@@ -14,7 +15,7 @@ partial class BackendContextModelSnapshot : ModelSnapshot
         protected override void BuildModel(ModelBuilder modelBuilder)
         {
 #pragma warning disable 612, 618
-            modelBuilder.HasAnnotation("ProductVersion", "9.0.4");
+            modelBuilder.HasAnnotation("ProductVersion", "9.0.6");
 
             modelBuilder.Entity("Backend.Domain.Task", b =>
                 {
@@ -25,6 +26,9 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                         .IsRequired()
                         .HasColumnType("TEXT");
 
+                    b.Property<Guid>("Id")
+                        .HasColumnType("TEXT");
+
                     b.HasKey("Name");
 
                     b.ToTable("Tasks");

Backend/Backend/Persistence/Migrations/BackendContextModelSnapshot.cs

@@ -37,8 +37,8 @@ public async Task CanLoadPersistedTasks()
     [Test]
     public async Task CanLoadSeveralPersistedTasks()
     {
-        var aTask = new Backend.Domain.Task("aTask", "Any");
-        var anotherTask = new Backend.Domain.Task("anotherTask", "Any");
+        var aTask = BackendTask("aTask", "Any");
+        var anotherTask = BackendTask("anotherTask", "Any");
         var backend = BackendBuilder.Backend()
             .With(aTask)
             .With(anotherTask)
@@ -61,13 +61,14 @@ public async Task CanPersistTasksWhenNoOtherExistedBefore()
             .Launch();
         var sut = new BackendTaskRepository(backend);
 
-        await sut.Save(DesktopTask());
+        await sut.Save(DesktopTask("A name", "A description"));
 
         var existingTasks = await sut.All();
         existingTasks.Should().BeEquivalentTo(
-        [
-            AnyBackendTask(),
-        ]);
+            [
+                BackendTask("A name", "A description"),
+            ],
+            options => options.Excluding(x => x.Id));
     }
 
     [Test]

Desktop.Tests/Integration/BackendTaskRepositoryTests.cs

@@ -8,6 +8,8 @@ public class Task : INotifyPropertyChanged
     private string name;
     private string description;
 
+    public Guid? Id { get; set; }
+
     public string Name
     {
         get => name;
@@ -38,6 +40,13 @@ public Task(string name, string description)
         Description = description;
     }
 
+    public Task(Guid id, string name, string description)
+    {
+        Id = id;
+        Name = name;
+        Description = description;
+    }
+
     public event PropertyChangedEventHandler? PropertyChanged;
 
     protected virtual void OnPropertyChanged([CallerMemberName] string? propertyName = null)

Desktop/Domain/Task.cs

@@ -51,7 +51,7 @@ public async Task<IReadOnlyList<Task>> All()
 
     private static Task ToDesktopTask(BackendTask task)
     {
-        return new Task(task.Name, task.Description);
+        return new Task(task.Id, task.Name, task.Description);
     }
 
     public async SystemTask Save(Task task)