fix: Return clearer error when team name empty or not set (#1354)

Right now the error users get when the team is not set is `failed to
configure quota monitor: failed to get usage: 404 Not Found`. The recent
CLI change in cloudquery/cloudquery#15119 will
ensure that the team should mostly get set, but there can still be cases
where users are in multiple teams and forget to run `cloudquery switch`.
In such a case, a friendlier error would be helpful.

Author: hermanschaaf

premium/usage.go

@@ -2,9 +2,11 @@ package premium
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"math/rand"
 	"net/http"
+	"os"
 	"sync/atomic"
 	"time"
 
@@ -166,15 +168,6 @@ func NewUsageClient(pluginTeam cqapi.PluginTeam, pluginKind cqapi.PluginKind, pl
 		op(u)
 	}
 
-	// Set team name from configuration if not provided
-	if u.teamName == "" {
-		teamName, err := config.GetValue("team")
-		if err != nil {
-			return nil, fmt.Errorf("failed to get team name from config: %w", err)
-		}
-		u.teamName = teamName
-	}
-
 	// Create a default api client if none was provided
 	if u.apiClient == nil {
 		tokenClient := auth.NewTokenClient()
@@ -192,6 +185,20 @@ func NewUsageClient(pluginTeam cqapi.PluginTeam, pluginKind cqapi.PluginKind, pl
 		u.apiClient = ac
 	}
 
+	// Set team name from configuration if not provided
+	if u.teamName == "" {
+		teamName, err := config.GetValue("team")
+		if errors.Is(err, os.ErrNotExist) {
+			return nil, fmt.Errorf("config file for reading team name not found (%w). Hint: use `cloudquery login` and/or `cloudquery switch <team>`", err)
+		} else if err != nil {
+			return nil, fmt.Errorf("failed to get team name from config: %w", err)
+		}
+		if teamName == "" {
+			return nil, fmt.Errorf("team name not set. Hint: use `cloudquery switch <team>`")
+		}
+		u.teamName = teamName
+	}
+
 	u.backgroundUpdater()
 
 	return u, nil