From 5e78fc72c54195ca37be5168cff49c363d94cb23 Mon Sep 17 00:00:00 2001
From: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Date: Thu, 3 Aug 2023 11:08:28 -0700
Subject: [PATCH] tests: add timeout to key version destruction test (#3537)

---
 internal/tests/api/credentials/credentials_test.go | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/internal/tests/api/credentials/credentials_test.go b/internal/tests/api/credentials/credentials_test.go
index 4700fc81a..c754c3f8f 100644
--- a/internal/tests/api/credentials/credentials_test.go
+++ b/internal/tests/api/credentials/credentials_test.go
@@ -4,6 +4,7 @@
 package credentials_test
 
 import (
+	"context"
 	"fmt"
 	"net/http"
 	"testing"
@@ -493,6 +494,8 @@ func TestUpdateAfterKeyRotation(t *testing.T) {
 	// Should start asynchronous rewrapping of the encrypted JSON credential
 	assert.Equal("pending", result.State)
 
+	ctx, cancel := context.WithTimeout(ctx, time.Minute)
+	defer cancel()
 	for {
 		jobs, err := scopesClient.ListKeyVersionDestructionJobs(ctx, proj.PublicId)
 		require.NoError(err)
@@ -501,6 +504,7 @@ func TestUpdateAfterKeyRotation(t *testing.T) {
 		}
 		select {
 		case <-ctx.Done():
+			t.Fatalf("Timed out waiting for key version destruction job, got jobs: %#v", jobs.Items)
 			break
 		case <-time.After(time.Millisecond * 100):
 		}