From 6955331e067d191dc0aace21abe6b7d2abd056fd Mon Sep 17 00:00:00 2001 From: Cristian Magherusan-Stanciu Date: Sun, 8 Nov 2026 08:46:01 +0100 Subject: [PATCH] fix(purchase): pass the platform AttemptJournal in the finalize test helper #1846 made purchase.AttemptJournal also require ClaimedByOthers, and #1832 merged after it with a test helper still typed as common.AttemptJournal, so internal/purchase tests no longer compile on main. The helper now takes the platform interface; its callers already pass a fake that implements ClaimedByOthers. Co-Authored-By: Claude Opus 5.5 --- internal/purchase/requested_at_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/purchase/requested_at_test.go b/internal/purchase/requested_at_test.go index 6b2a9a179..c3f8c40b5 100644 --- a/internal/purchase/requested_at_test.go +++ b/internal/purchase/requested_at_test.go @@ -156,7 +156,7 @@ func runRedshiftFinalize(t *testing.T, exec *config.PurchaseExecution, accounts // runRedshiftFinalizeWithJournal is runRedshiftFinalize on a manager whose attempt journal is journal. func runRedshiftFinalizeWithJournal(t *testing.T, exec *config.PurchaseExecution, accounts []config.CloudAccount, service common.ServiceType, - stamp func() (config.StampResult, error), journal common.AttemptJournal) *redshiftFinalizeRun { + stamp func() (config.StampResult, error), journal AttemptJournal) *redshiftFinalizeRun { t.Helper() ctx := context.Background() mockStore := new(MockConfigStore)