toki/nomadcode
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

test: improve RoadmapRevision verification to use dynamic after value

Browse source 
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.
This commit is contained in:
toki 2026-06-19 16:37:50 +09:00
parent d0d4c810ae
commit 83036e366b
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
services/core/cmd/server/main_test.go
View file

@ -232,8 +232,9 @@ func TestGitoHTTPWebhookDeliveryEnqueuesCreationSyncOnce(t *testing.T) {
job := enq.jobs[0] job := enq.jobs[0]
// 10. Verify job content. // 10. Verify job content.
if job.RoadmapRevision != "abc123def456" { // RoadmapRevision should match the HTTP payload's `after` value (passed through scanner).
t.Fatalf("RoadmapRevision: got %q, want %q", job.RoadmapRevision, "abc123def456") if job.RoadmapRevision != after {
t.Fatalf("RoadmapRevision: got %q, want %q (HTTP payload `after`)", job.RoadmapRevision, after)
} }
if job.Expected.WorkItemID != "wi-123" { if job.Expected.WorkItemID != "wi-123" {
t.Fatalf("Expected.WorkItemID: got %q, want %q", job.Expected.WorkItemID, "wi-123") t.Fatalf("Expected.WorkItemID: got %q, want %q", job.Expected.WorkItemID, "wi-123")