switch to new client

cmd/infracost/run.go

@@ -943,14 +943,11 @@ func checkAPIKeyIsValid(ctx *config.RunContext, next runCommandFunc) runCommandF
 				"if you continue having issues.")
 		}
 
-		pricingClient := apiclient.GetPricingAPIClient(ctx)
-		resp, err := pricingClient.DoQueries([]apiclient.GraphQLQuery{
+		pricingClient := apiclient.NewPricingAPIClient(ctx)
+		_, err := pricingClient.DoQueries([]apiclient.GraphQLQuery{
 			{},
 		})
-		if len(resp) > 0 {
-			cmd.Println(resp[0])
-		}
-		//cmd.Println(err)
+
 		var apiError *apiclient.APIError
 		if errors.As(err, &apiError) {
 			if apiError.ErrorCode == apiclient.ErrorCodeAPIKeyInvalid {

internal/apiclient/pricing.go

@@ -76,6 +76,19 @@ func GetPricingAPIClient(ctx *config.RunContext) *PricingAPIClient {
 		return pricingClient
 	}
 
+	c := NewPricingAPIClient(ctx)
+
+	initCache(ctx, c)
+	pricingClient = c
+	return c
+}
+
+// NewPricingAPIClient creates a new instance of PricingAPIClient using the given
+// RunContext configuration. Most callers should use GetPricingAPIClient instead
+// of this function to ensure that the client cache is global across the
+// application. This function is useful for creating isolated pricing clients
+// which do not share the global cache.
+func NewPricingAPIClient(ctx *config.RunContext) *PricingAPIClient {
 	if ctx == nil {
 		return nil
 	}
@@ -129,8 +142,6 @@ func GetPricingAPIClient(ctx *config.RunContext) *PricingAPIClient {
 		EventsDisabled: ctx.Config.EventsDisabled,
 	}
 
-	initCache(ctx, c)
-	pricingClient = c
 	return c
 }