test: fixed api precheck not being configured

Author: burythehammer

provider/client/client.go

@@ -1,9 +1,31 @@
 package client
 
 import (
+	"os"
+
 	rediscloudApi "github.com/RedisLabs/rediscloud-go-api"
 )
 
 type ApiClient struct {
 	Client *rediscloudApi.Client
 }
+
+// NewClient creates a new ApiClient using environment variables for configuration.
+// This is useful for tests that need to create a client before the provider is configured.
+func NewClient() (*ApiClient, error) {
+	var config []rediscloudApi.Option
+
+	url := os.Getenv("REDISCLOUD_URL")
+	if url != "" {
+		config = append(config, rediscloudApi.BaseURL(url))
+	}
+
+	client, err := rediscloudApi.NewClient(config...)
+	if err != nil {
+		return nil, err
+	}
+
+	return &ApiClient{
+		Client: client,
+	}, nil
+}

provider/rediscloud_essentials_subscription_test.go

@@ -23,7 +23,11 @@ var essentialsMarketplaceFlag = flag.Bool("essentialsMarketplace", false,
 func testAccPreCheckEssentialsSubscription(t *testing.T) {
 	testAccPreCheck(t)
 
-	apiClient := testProvider.Meta().(*client.ApiClient)
+	apiClient, err := client.NewClient()
+	if err != nil {
+		t.Fatalf("Failed to create API client: %v", err)
+	}
+
 	subs, err := apiClient.Client.FixedSubscriptions.List(context.TODO())
 	if err != nil {
 		t.Fatalf("Failed to list essentials subscriptions: %v", err)