[CLAUDE] App: chuẩn hóa mã dự án — trim + kiểm-trùng bỏ dấu-cách chống trùng do khoảng trắng

Project Create/Update: LƯU chỉ Trim (giữ mã như người dùng gõ — owner chốt), nhưng kiểm-trùng so trên dạng bỏ-dấu-cách (Code.Replace(' ','')) để chặn tạo/sửa thành bản trùng chỉ khác khoảng trắng ('FLOCK 01' vs 'FLOCK01'). +4 test guard ProjectCodeNormalizationTests. Đây là vá gốc cho vụ dự án trùng vừa gom trên prod (5 phiếu + 5 ngân sách trỏ lại, 3 bản dấu-cách ẩn mềm). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-07-23 17:11:36 +07:00
parent 7d39d8d61b
commit 4510cea1e3
2 changed files with 98 additions and 6 deletions
--- a/tests/SolutionErp.Infrastructure.Tests/Application/ProjectCodeNormalizationTests.cs
+++ b/tests/SolutionErp.Infrastructure.Tests/Application/ProjectCodeNormalizationTests.cs
@ -0,0 +1,83 @@
+using Microsoft.EntityFrameworkCore;
+using SolutionErp.Application.Common.Exceptions;
+using SolutionErp.Application.Master.Projects;
+using SolutionErp.Domain.Master;
+using SolutionErp.Infrastructure.Tests.Common;
+
+namespace SolutionErp.Infrastructure.Tests.Application;
+
+// [Fix UAT 2026-07-23 — dự án trùng do khác dấu-cách trong mã: "FLOCK 01" vs "FLOCK01"]
+// Owner (chốt): LƯU chỉ cắt đầu/cuối (Trim) — giữ mã như người dùng gõ; nhưng KIỂM-TRÙNG
+// bỏ-qua-dấu-cách để chặn tạo/sửa thành bản trùng chỉ khác khoảng trắng (chống tái phát
+// sau khi gom dữ liệu). Dupe-check là bug-class guard → test-before (docs/rules.md §7).
+// Handlers chỉ cần IApplicationDbContext → new trực tiếp với fix.Db.
+public class ProjectCodeNormalizationTests
+{
+    private static CreateProjectCommand NewProject(string code, string name = "Dự án") =>
+        new(code, name, null, null, null, null, null, null, null, null, null);
+
+    [Fact]
+    public async Task CreateProject_SpaceVariantOfExistingCode_ThrowsConflict()
+    {
+        using var fix = new SqliteDbFixture();
+        var db = fix.Db;
+        db.Projects.Add(new Project { Id = Guid.NewGuid(), Code = "FLOCK01", Name = "FLOCK01" });
+        await db.SaveChangesAsync(CancellationToken.None);
+
+        // "FLOCK 01" (dấu cách giữa) == "FLOCK01" sau khi bỏ dấu cách → phải chặn.
+        var act = async () => await new CreateProjectCommandHandler(db)
+            .Handle(NewProject("FLOCK 01"), CancellationToken.None);
+
+        await act.Should().ThrowAsync<ConflictException>(
+            "kiểm-trùng bỏ-qua-dấu-cách phải bắt bản trùng chỉ khác khoảng trắng");
+        (await db.Projects.CountAsync()).Should().Be(1, "không tạo thêm bản trùng dấu-cách");
+    }
+
+    [Fact]
+    public async Task CreateProject_TrimsLeadingTrailingWhitespace_ButKeepsInternalSpace()
+    {
+        using var fix = new SqliteDbFixture();
+        var db = fix.Db;
+
+        var id = await new CreateProjectCommandHandler(db)
+            .Handle(NewProject("  CAL 02  "), CancellationToken.None);
+
+        var saved = await db.Projects.FirstAsync(x => x.Id == id);
+        // Owner chọn "chỉ cắt đầu/cuối": Trim bỏ khoảng trắng đầu/cuối, GIỮ dấu cách giữa.
+        saved.Code.Should().Be("CAL 02");
+    }
+
+    [Fact]
+    public async Task CreateProject_GenuinelyNewCode_Succeeds()
+    {
+        using var fix = new SqliteDbFixture();
+        var db = fix.Db;
+        db.Projects.Add(new Project { Id = Guid.NewGuid(), Code = "FLOCK01", Name = "FLOCK01" });
+        await db.SaveChangesAsync(CancellationToken.None);
+
+        var act = async () => await new CreateProjectCommandHandler(db)
+            .Handle(NewProject("SOV01"), CancellationToken.None);
+
+        await act.Should().NotThrowAsync("mã mới hoàn toàn không đụng bản nào");
+        (await db.Projects.CountAsync()).Should().Be(2);
+    }
+
+    [Fact]
+    public async Task UpdateProject_ToSpaceVariantOfAnotherCode_ThrowsConflict()
+    {
+        using var fix = new SqliteDbFixture();
+        var db = fix.Db;
+        db.Projects.Add(new Project { Id = Guid.NewGuid(), Code = "FLOCK01", Name = "FLOCK01" });
+        var otherId = Guid.NewGuid();
+        db.Projects.Add(new Project { Id = otherId, Code = "TARGET", Name = "Khác" });
+        await db.SaveChangesAsync(CancellationToken.None);
+
+        // Đổi mã "TARGET" → "FLOCK 01" (trùng "FLOCK01" bỏ dấu cách) → phải chặn.
+        var act = async () => await new UpdateProjectCommandHandler(db)
+            .Handle(new UpdateProjectCommand(otherId, "FLOCK 01", "Khác",
+                null, null, null, null, null, null, null, null, null), CancellationToken.None);
+
+        await act.Should().ThrowAsync<ConflictException>(
+            "sửa sang biến thể dấu-cách của mã đã tồn tại cũng phải bị chặn");
+    }
+}