fix data race for the API key acc. tests

Signed-off-by: Dmitry Kisler <admin@dkisler.com>

Author: kislerdm

provider/acc-orgapikey_test.go

@@ -6,6 +6,7 @@ import (
 	"slices"
 	"testing"
 
+	"github.com/google/uuid"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/terraform"
@@ -27,7 +28,7 @@ func TestAccOrgAPIKeys(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	wantKeyName := projectNamePrefix + "orgApikey"
+	wantKeyName := projectNamePrefix + "orgApikey" + uuid.NewString()
 
 	t.Cleanup(func() {
 		keys, _ := client.ListOrgApiKeys(orgID)

provider/acc_api_key_test.go

@@ -3,8 +3,10 @@ package provider
 import (
 	"fmt"
 	"os"
+	"strings"
 	"testing"
 
+	"github.com/google/uuid"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/terraform"
@@ -41,8 +43,24 @@ func TestRecreateAPIKeyIfNotFound(t *testing.T) {
 
 	}
 
+	t.Cleanup(func() {
+		ref, err := client.ListApiKeys()
+		if err != nil {
+			panic(err)
+		}
+
+		for _, key := range ref {
+			if strings.HasPrefix(key.Name, "test") {
+				_, err = client.RevokeApiKey(key.ID)
+				if err != nil {
+					panic(err)
+				}
+			}
+		}
+	})
+
 	t.Run("shall indicate non empty plan if the API key was deleted outside of terraform", func(t *testing.T) {
-		keyName := "test"
+		keyName := "test" + uuid.NewString()
 		resource.Test(
 			t, resource.TestCase{
 				ProviderFactories: map[string]func() (*schema.Provider, error){
@@ -72,7 +90,7 @@ func TestRecreateAPIKeyIfNotFound(t *testing.T) {
 	})
 
 	t.Run("shall destroy even if the API key was deleted outside of terraform,", func(t *testing.T) {
-		keyName := "test"
+		keyName := "test" + uuid.NewString()
 		config := fmt.Sprintf(`resource "neon_api_key" "this" {name = "%s"}`, keyName)
 		resource.Test(
 			t, resource.TestCase{