Wondering if this captures the intent of what we'd like this function…

… to be.

helper_test.go

@@ -1235,23 +1235,22 @@ func waitForRunLock(t *testing.T, client *Client, workspaceID string) {
 
 func retry(f retryableFn) (interface{}, error) { //nolint
 	tick := time.NewTicker(tickDuration * time.Second)
-
 	retries := 0
 	maxRetries := 5
 
-	for {
-		if tick.C != nil {
-			res, err := f()
-			if err == nil {
-				return res, nil
-			}
+	defer tick.Stop()
 
-			if retries >= maxRetries {
-				return nil, err
-			}
+	for {
+		res, err := f()
+		if err == nil {
+			return res, nil
+		}
 
-			retries += 1
+		if retries >= maxRetries {
+			return nil, err
 		}
+
+		retries += 1
 	}
 }