[CLAUDE] PurchaseEvaluation: endsBeforeCeo theo RUNTIME (Mig 60 EndedByLevelFinalize) — phiếu lên CEO hết báo sai "không qua CEO" (anh Kiệt FDC UAT)
All checks were successful
Deploy SOLUTION_ERP / build-deploy (push) Successful in 5m21s
All checks were successful
Deploy SOLUTION_ERP / build-deploy (push) Successful in 5m21s
- Bug: phiếu có cấp AllowApproverFinalize nhưng approver opt-out → lên CEO thật + CEO duyệt, nhưng sơ đồ báo sai "kết thúc tại CCM, không qua CEO" + làm mờ Bước CEO (endsBeforeCeo tính theo CONFIG workflow) - Fix: +field runtime EndedByLevelFinalize (set true CHỈ ở nhánh level-finalize service) → EndsBeforeCeo cho phiếu DaDuyet đọc runtime thay config; ChoDuyet giữ heads-up config - BE: entity + EF config + service + 4 projection (detail/List/Inbox/ListApproved) runtime-aware - Mig 60 AddPeEndedByLevelFinalize + backfill (dò marker "Duyệt KẾT THÚC tại" set phiếu finalize-sớm cũ; ccm-delegation không match → giữ false) - FE: gate làm-mờ finalizeFlowStep theo endsBeforeCeo (2 app SHA-mirror b7abbc2f) - Test 440 PASS (repro peFinButWentCeo: config-finalize + EndedByLevelFinalize=false → false) + reviewer PASS 0-issue Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
@ -9,13 +9,24 @@ using SolutionErp.Infrastructure.Tests.Common;
|
||||
namespace SolutionErp.Infrastructure.Tests.Application;
|
||||
|
||||
// [S89 anh Kiệt FDC] PE finalize-note (Mig 58 AllowApproverFinalize) — TRANSLATION-SMOKE
|
||||
// cho 3 subquery mới (EndsBeforeCeo / FinalizeStepName / FinalizeLevelName) trong list
|
||||
// projection. Subquery: (from lv in db.ApprovalWorkflowLevels where lv.AllowApproverFinalize
|
||||
// cho subquery FinalizeStepName / FinalizeLevelName trong list projection. Subquery:
|
||||
// (from lv in db.ApprovalWorkflowLevels where lv.AllowApproverFinalize
|
||||
// && lv.Step!.ApprovalWorkflowId == e.ApprovalWorkflowId orderby lv.Step!.Order, lv.Order
|
||||
// select lv.Step!.Name / lv.Name).FirstOrDefault() — PHẢI translate trên EF SQLite (rủi ro
|
||||
// reviewer cảnh báo). Target = ListApprovedPurchaseEvaluationsQuery (sạch nhất: chỉ
|
||||
// Phase==DaDuyet, KHÔNG role-paging) — cùng subquery xuất hiện ở List + Inbox projection.
|
||||
// Verify: translate (không ném) + first-reachable (Step.Order↑→Level.Order↑) + V1 null→false.
|
||||
// Phase==DaDuyet, KHÔNG role-paging).
|
||||
//
|
||||
// ⚠️ [S97 2026-07-01 SPEC CHANGE — anh Kiệt FDC] EndsBeforeCeo ĐỔI CONFIG-driven → RUNTIME-driven.
|
||||
// Bug: workflow CÓ cấp AllowApproverFinalize NHƯNG người duyệt chọn trình tiếp → phiếu LÊN CEO
|
||||
// thật → sơ đồ vẫn báo sai "Duyệt là KẾT THÚC, không qua CEO" (vì suy từ config = luôn true).
|
||||
// Fix (em-main BE): field PurchaseEvaluation.EndedByLevelFinalize (bool, Mig 60) set true DUY
|
||||
// NHẤT ở nhánh level-finalize của ApproveV2Async. ListApproved (DaDuyet-only) đọc THẲNG field
|
||||
// này thay 3-subquery-Any cũ (CreateContractFromEvaluationFeatures.cs:196). FinalizeStep/LevelName
|
||||
// VẪN từ config-subquery (chỉ nhãn hiển-thị tên cấp finalize, không phải cờ true/false).
|
||||
// → test cũ (finalize-config nhưng KHÔNG set field) giờ EndsBeforeCeo=FALSE → seed field=true.
|
||||
// → repro case MỚI: config-có-finalize + field=false (đi hết lên CEO) → PHẢI false (chính bug).
|
||||
// Verify: EF translate (không ném) + config-name first-reachable (Step.Order↑→Level.Order↑) +
|
||||
// runtime-field độc-lập-config + V1 null→false.
|
||||
// IdentityFixture: ApproverUserId là Guid non-null (FK) → cần 1 User row thật.
|
||||
public class PeFinalizeProjectionTests
|
||||
{
|
||||
@ -57,7 +68,12 @@ public class PeFinalizeProjectionTests
|
||||
return wf;
|
||||
}
|
||||
|
||||
private static Guid SeedApprovedPe(TestApplicationDbContext db, Guid projectId, Guid? wfId, string code)
|
||||
// [S97] endedByLevelFinalize = RUNTIME fact phiếu ĐÃ finalize sớm tại cấp "Duyệt thay CEO"
|
||||
// (short-circuit TRƯỚC CEO). ListApproved đọc THẲNG field này (DaDuyet-only) → EndsBeforeCeo.
|
||||
// Config "workflow CÓ cấp finalize" (wfId có slot AllowApproverFinalize) KHÔNG còn quyết định.
|
||||
private static Guid SeedApprovedPe(
|
||||
TestApplicationDbContext db, Guid projectId, Guid? wfId, string code,
|
||||
bool endedByLevelFinalize = false)
|
||||
{
|
||||
var id = Guid.NewGuid();
|
||||
db.PurchaseEvaluations.Add(new PurchaseEvaluation
|
||||
@ -71,12 +87,13 @@ public class PeFinalizeProjectionTests
|
||||
ContractId = null, // chưa gen HĐ → lọt picker
|
||||
DrafterUserId = null,
|
||||
ApprovalWorkflowId = wfId,
|
||||
EndedByLevelFinalize = endedByLevelFinalize,
|
||||
});
|
||||
return id;
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public async Task ListApproved_FinalizeFields_Translate_FirstReachable_V1False()
|
||||
public async Task ListApproved_FinalizeFields_Translate_RuntimeDriven_ConfigNamesOnly_V1False()
|
||||
{
|
||||
using var fix = new IdentityFixture();
|
||||
var db = fix.Services.GetRequiredService<TestApplicationDbContext>();
|
||||
@ -86,28 +103,44 @@ public class PeFinalizeProjectionTests
|
||||
db.Projects.Add(project);
|
||||
|
||||
// Bước 1 (Cấp 1, no-flag) → Bước 2 (Cấp 1, FINALIZE). first-reachable = Bước 2 · Cấp 1.
|
||||
// 1 workflow CÓ cấp finalize dùng cho 2 phiếu (khác nhau CHỈ ở runtime field) → chứng
|
||||
// minh EndsBeforeCeo lấy từ EndedByLevelFinalize, KHÔNG suy từ config workflow.
|
||||
var wfFin = BuildWorkflow("QT-FIN", approver.Id,
|
||||
("Bước 1", new[] { ("Cấp 1", false) }),
|
||||
("Bước 2", new[] { ("Cấp 1", true) }));
|
||||
db.ApprovalWorkflows.Add(wfFin);
|
||||
// Không cấp nào finalize → endsBeforeCeo=false.
|
||||
// Không cấp nào finalize → config-name null + field mặc-định false.
|
||||
var wfPlain = BuildWorkflow("QT-PLAIN", approver.Id, ("Bước 1", new[] { ("Cấp 1", false) }));
|
||||
db.ApprovalWorkflows.Add(wfPlain);
|
||||
|
||||
var peFin = SeedApprovedPe(db, project.Id, wfFin.Id, "PE-FIN");
|
||||
// [S97] peFin: config-có-finalize + phiếu ĐÃ finalize sớm (EndedByLevelFinalize=true) → true.
|
||||
var peFin = SeedApprovedPe(db, project.Id, wfFin.Id, "PE-FIN", endedByLevelFinalize: true);
|
||||
// 🔴 [S97 REPRO — bug anh Kiệt] CÙNG workflow-có-finalize NHƯNG người duyệt trình tiếp →
|
||||
// phiếu LÊN CEO thật (EndedByLevelFinalize=false) → EndsBeforeCeo PHẢI false (không được
|
||||
// suy từ config = true như trước fix). Đây là contrast quyết định runtime-vs-config.
|
||||
var peFinButWentCeo = SeedApprovedPe(db, project.Id, wfFin.Id, "PE-FIN-CEO", endedByLevelFinalize: false);
|
||||
var pePlain = SeedApprovedPe(db, project.Id, wfPlain.Id, "PE-PLAIN");
|
||||
var peV1 = SeedApprovedPe(db, project.Id, null, "PE-V1"); // V1 legacy: no workflow
|
||||
await db.SaveChangesAsync(CancellationToken.None);
|
||||
|
||||
var handler = new ListApprovedPurchaseEvaluationsQueryHandler(db);
|
||||
// Nếu 3 subquery KHÔNG translate → ném InvalidOperationException = FAIL (blocker).
|
||||
// Nếu FinalizeStep/LevelName subquery KHÔNG translate → ném InvalidOperationException = FAIL.
|
||||
var res = await handler.Handle(new ListApprovedPurchaseEvaluationsQuery(), CancellationToken.None);
|
||||
|
||||
// Phiếu finalize sớm THẬT → true; FinalizeStep/LevelName vẫn từ config (nhãn cấp finalize).
|
||||
var fin = res.Single(x => x.Id == peFin);
|
||||
fin.EndsBeforeCeo.Should().BeTrue();
|
||||
fin.EndsBeforeCeo.Should().BeTrue("phiếu ĐÃ kết thúc sớm tại cấp finalize (runtime field true)");
|
||||
fin.FinalizeStepName.Should().Be("Bước 2"); // first-reachable bỏ qua Bước 1 no-flag
|
||||
fin.FinalizeLevelName.Should().Be("Cấp 1");
|
||||
|
||||
// 🔴 REPRO: cùng config finalize nhưng phiếu đi hết lên CEO → EndsBeforeCeo=false (bug fix).
|
||||
var finCeo = res.Single(x => x.Id == peFinButWentCeo);
|
||||
finCeo.EndsBeforeCeo.Should().BeFalse(
|
||||
"config CÓ cấp finalize nhưng phiếu lên CEO thật (EndedByLevelFinalize=false) → KHÔNG hiện 'không qua CEO'");
|
||||
// FinalizeStep/LevelName vẫn phản ánh config (workflow này có cấp finalize) — chỉ EndsBeforeCeo runtime.
|
||||
finCeo.FinalizeStepName.Should().Be("Bước 2");
|
||||
finCeo.FinalizeLevelName.Should().Be("Cấp 1");
|
||||
|
||||
res.Single(x => x.Id == pePlain).EndsBeforeCeo.Should().BeFalse();
|
||||
res.Single(x => x.Id == peV1).EndsBeforeCeo.Should().BeFalse(); // V1 null-workflow tự đúng
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user