chore: auth login --skip-verify (#30)

Author: notnmeyer

cmd/auth_login.go

@@ -13,6 +13,7 @@ import (
 )
 
 var loginName string
+var loginSkipVerify bool
 
 var loginCmd = &cobra.Command{
 	Use:   "login",
@@ -34,23 +35,37 @@ var loginCmd = &cobra.Command{
 			return fmt.Errorf("API key cannot be empty")
 		}
 
-		result, err := runAuthLogin(apiKey, loginName)
+		result, err := runAuthLogin(apiKey, loginName, loginSkipVerify)
 		if err != nil {
 			return err
 		}
 
 		if isJSONOutput() {
-			return printJSON(cmd.OutOrStdout(), Result{Success: true, Message: fmt.Sprintf("Authenticated as team: %s", result.TeamName)})
+			msg := fmt.Sprintf("API key saved as %q", loginName)
+			if result != nil {
+				msg = fmt.Sprintf("Authenticated as team: %s", result.TeamName)
+			}
+			return printJSON(cmd.OutOrStdout(), Result{Success: true, Message: msg})
+		}
+		if result != nil {
+			fmt.Fprintf(cmd.OutOrStdout(), "API key saved as %q. Authenticated as team: %s\n", loginName, result.TeamName)
+		} else {
+			fmt.Fprintf(cmd.OutOrStdout(), "API key saved as %q.\n", loginName)
 		}
-		fmt.Fprintf(cmd.OutOrStdout(), "API key saved as %q. Authenticated as team: %s\n", loginName, result.TeamName)
 		return nil
 	},
 }
 
-func runAuthLogin(apiKey, name string) (*api.APIKeyResponse, error) {
+func runAuthLogin(apiKey, name string, skipVerify bool) (*api.APIKeyResponse, error) {
 	if name == "" {
 		return nil, errors.New("use --name to give this key a name")
 	}
+	if skipVerify {
+		if err := config.Save(apiKey, name); err != nil {
+			return nil, err
+		}
+		return nil, nil
+	}
 	result, err := api.NewClient(config.EndpointURL(), apiKey, debugFlag).GetAPIKey()
 	if err != nil {
 		return nil, fmt.Errorf("API key verification failed: %w", err)
@@ -63,5 +78,6 @@ func runAuthLogin(apiKey, name string) (*api.APIKeyResponse, error) {
 
 func init() {
 	loginCmd.Flags().StringVarP(&loginName, "name", "n", "", "Name for this API key (e.g. my-team)")
+	loginCmd.Flags().BoolVar(&loginSkipVerify, "skip-verify", false, "Save the API key without verifying it")
 	authCmd.AddCommand(loginCmd)
 }

cmd/auth_login_test.go

@@ -11,7 +11,7 @@ import (
 func TestRunAuthLogin(t *testing.T) {
 	t.Run("saves key and returns team name", func(t *testing.T) {
 		serveJSON(t, http.StatusOK, `{"teamName":"Acme"}`)
-		result, err := runAuthLogin("test-key", "acme")
+		result, err := runAuthLogin("test-key", "acme", false)
 		if err != nil {
 			t.Fatalf("unexpected error: %v", err)
 		}
@@ -23,17 +23,27 @@ func TestRunAuthLogin(t *testing.T) {
 
 	t.Run("returns error on api failure", func(t *testing.T) {
 		serveJSON(t, http.StatusUnauthorized, `{"error":"Invalid API key"}`)
-		_, err := runAuthLogin("bad-key", "acme")
+		_, err := runAuthLogin("bad-key", "acme", false)
 		if err == nil {
 			t.Fatal("expected error, got nil")
 		}
 	})
 
 	t.Run("returns error when name is empty", func(t *testing.T) {
 		serveJSON(t, http.StatusOK, `{"teamName":"Acme"}`)
-		_, err := runAuthLogin("test-key", "")
+		_, err := runAuthLogin("test-key", "", false)
 		if err == nil {
 			t.Fatal("expected error, got nil")
 		}
 	})
+
+	t.Run("skip-verify saves key without calling api", func(t *testing.T) {
+		result, err := runAuthLogin("test-key", "acme", true)
+		if err != nil {
+			t.Fatalf("unexpected error: %v", err)
+		}
+		if result != nil {
+			t.Errorf("expected nil result, got %+v", result)
+		}
+	})
 }

cmd/auth_logout_test.go

@@ -8,7 +8,7 @@ import (
 func TestRunAuthLogout(t *testing.T) {
 	t.Run("succeeds", func(t *testing.T) {
 		serveJSON(t, http.StatusOK, `{}`)
-		runAuthLogin("test-key", "acme")
+		runAuthLogin("test-key", "acme", true)
 		if err := runAuthLogout("acme"); err != nil {
 			t.Fatalf("unexpected error: %v", err)
 		}