feat: improve error message in autocomplete install (scaleway#1102)

remyleone · Jun 16, 2020 · c649abd · c649abd
1 parent ac2741a
commit c649abd
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/internal/namespaces/autocomplete/autocomplete.go b/internal/namespaces/autocomplete/autocomplete.go
@@ -182,7 +182,7 @@ func InstallCommandRun(ctx context.Context, argsI interface{}) (i interface{}, e
 		defer f.Close()
 	}
 	if err != nil {
-		return nil, err
+		return nil, installationNotFound(shellName, shellConfigurationFilePath, script.CompleteScript)
 	}
 
 	// Early exit if eval line is already present in the shell configuration.

diff --git a/internal/namespaces/autocomplete/errors.go b/internal/namespaces/autocomplete/errors.go
@@ -24,3 +24,10 @@ func installationCancelledError(shellName string, script string) *core.CliError
 		Hint: fmt.Sprintf("To manually enable autocomplete for %v, run: %v", shellName, script),
 	}
 }
+
+func installationNotFound(shellName string, location string, script string) *core.CliError {
+	return &core.CliError{
+		Err:  fmt.Errorf("cannot find where to install autocomplete script (tried %s)", location),
+		Hint: fmt.Sprintf("You can add this line: `%s` in your %s configuration file", script, shellName),
+	}
+}