fix(data): exclude Lo Celso from seed data, keep as Create/Delete fixture (#435)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: nanotaboada

src/Dotnet.Samples.AspNetCore.WebApi/Migrations/20260329000000_NormalizePlayerDataset.cs

@@ -93,25 +93,11 @@ protected override void Up(MigrationBuilder migrationBuilder)
                 "UPDATE Players SET Id = '79c96f29-c59f-5f98-96b8-3a5946246624' WHERE SquadNumber = 22"
             );
 
-            // Insert Lo Celso (squad 27) — OR IGNORE is safe on a fresh DB where SeedSubstitutes
-            // already seeded him (because GetSubstitutesWithId() now includes him), and required
-            // for an existing DB that was seeded before this migration was added.
-            migrationBuilder.Sql(
-                "INSERT OR IGNORE INTO \"Players\" (\"Id\", \"FirstName\", \"MiddleName\", \"LastName\", \"DateOfBirth\", \"SquadNumber\", \"Position\", \"AbbrPosition\", \"Team\", \"League\", \"Starting11\") "
-                    + "VALUES ('F8D13028-0D22-5513-8774-08A2332B5814', 'Giovani', NULL, 'Lo Celso', '1996-04-09 00:00:00', 27, 'Central Midfield', 'CM', 'Real Betis Balompié', 'La Liga', 0)"
-            );
         }
 
         /// <inheritdoc />
         protected override void Down(MigrationBuilder migrationBuilder)
         {
-            // Remove Lo Celso (squad 27)
-            migrationBuilder.DeleteData(
-                table: "Players",
-                keyColumn: "Id",
-                keyValue: Guid.Parse("f8d13028-0d22-5513-8774-08a2332b5814")
-            );
-
             // Revert team/league corrections
             migrationBuilder.Sql(
                 "UPDATE Players SET Team = 'Chelsea FC', League = 'Premier League' WHERE SquadNumber = 24"

src/Dotnet.Samples.AspNetCore.WebApi/Utilities/PlayerData.cs

@@ -540,9 +540,9 @@ public static List<Player> GetSubstitutes()
     }
 
     /// <summary>
-    /// Create a predefined list of 16 substitute players where each player has a fixed GUID identifier and full profile data.
+    /// Create a predefined list of 15 substitute players where each player has a fixed GUID identifier and full profile data.
     /// </summary>
-    /// <returns>A list of 16 Player instances representing substitute players; each entry includes a predefined Id (Guid) and populated fields such as name, date of birth, squad number, position (and abbreviation), team, league, and Starting11 set to false.</returns>
+    /// <returns>A list of 15 Player instances representing substitute players (squads 1–26); each entry includes a predefined Id (Guid) and populated fields such as name, date of birth, squad number, position (and abbreviation), team, league, and Starting11 set to false. Lo Celso (squad 27) is intentionally excluded — his squad number falls outside the seeded range so he can serve as the canonical Create/Delete fixture without conflicting with seeded data.</returns>
     public static List<Player> GetSubstitutesWithId()
     {
         return
@@ -754,19 +754,6 @@ public static List<Player> GetSubstitutesWithId()
                 League = "Serie A",
                 Starting11 = false,
             },
-            new()
-            {
-                Id = Guid.Parse("f8d13028-0d22-5513-8774-08a2332b5814"),
-                FirstName = "Giovani",
-                LastName = "Lo Celso",
-                DateOfBirth = new DateTime(1996, 4, 9, 0, 0, 0, DateTimeKind.Utc),
-                SquadNumber = 27,
-                Position = Position.CentralMidfield.Text,
-                AbbrPosition = Position.CentralMidfield.Abbr,
-                Team = "Real Betis Balompié",
-                League = "La Liga",
-                Starting11 = false,
-            },
         ];
     }