chore: some minor updates

burythehammer · burythehammer · commit 2df372fdf97f · 2025-07-31T14:51:36.000+01:00
diff --git a/service/subscriptions/service.go b/service/subscriptions/service.go
@@ -101,7 +101,7 @@ func (a *API) UpdateCMKs(ctx context.Context, id int, subscriptionCMKs UpdateSub
 		return wrap404Error(id, err)
 	}
 
-	a.logger.Printf("Waiting for task %s to finish updating the subscription", task)
+	a.logger.Printf("Waiting for task %s to finish updating subscription %d", task, id)
 
 	err = a.taskWaiter.Wait(ctx, *task.ID)
 	if err != nil {
diff --git a/subscription_test.go b/subscription_test.go
@@ -650,6 +650,7 @@ func TestSubscription_Update_CMKs(t *testing.T) {
 		},
 	})
 
+	require.NoError(t, err)
 }
 
 func TestSubscription_Delete(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ func (a *API) UpdateCMKs(ctx context.Context, id int, subscriptionCMKs UpdateSub`
`101`	`101`	`return wrap404Error(id, err)`
`102`	`102`	`}`
`103`	`103`
`104`		`- a.logger.Printf("Waiting for task %s to finish updating the subscription", task)`
	`104`	`+ a.logger.Printf("Waiting for task %s to finish updating subscription %d", task, id)`
`105`	`105`
`106`	`106`	`err = a.taskWaiter.Wait(ctx, *task.ID)`
`107`	`107`	`if err != nil {`
Original file line number	Diff line number	Diff line change
`@@ -650,6 +650,7 @@ func TestSubscription_Update_CMKs(t *testing.T) {`
`650`	`650`	`},`
`651`	`651`	`})`
`652`	`652`
	`653`	`+ require.NoError(t, err)`
`653`	`654`	`}`
`654`	`655`
`655`	`656`	`func TestSubscription_Delete(t *testing.T) {`