From 83036e366bfeedb027c2da3905e1e6038a03eda9 Mon Sep 17 00:00:00 2001 From: toki Date: Fri, 19 Jun 2026 16:37:50 +0900 Subject: [PATCH] test: improve RoadmapRevision verification to use dynamic after value MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change RoadmapRevision check from hardcoded string to comparing against the HTTP payload's `after` variable directly, proving the full chain: HTTP payload `after` → scanner → bridge → job.RoadmapRevision. --- services/core/cmd/server/main_test.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/services/core/cmd/server/main_test.go b/services/core/cmd/server/main_test.go index f92feb0..0b8930d 100644 --- a/services/core/cmd/server/main_test.go +++ b/services/core/cmd/server/main_test.go @@ -232,8 +232,9 @@ func TestGitoHTTPWebhookDeliveryEnqueuesCreationSyncOnce(t *testing.T) { job := enq.jobs[0] // 10. Verify job content. - if job.RoadmapRevision != "abc123def456" { - t.Fatalf("RoadmapRevision: got %q, want %q", job.RoadmapRevision, "abc123def456") + // RoadmapRevision should match the HTTP payload's `after` value (passed through scanner). + if job.RoadmapRevision != after { + t.Fatalf("RoadmapRevision: got %q, want %q (HTTP payload `after`)", job.RoadmapRevision, after) } if job.Expected.WorkItemID != "wi-123" { t.Fatalf("Expected.WorkItemID: got %q, want %q", job.Expected.WorkItemID, "wi-123")