Added color for warning/error in cli

Author: akclace

cmd/clace/app_cmds.go

@@ -21,7 +21,7 @@ import (
 const (
 	DRY_RUN_FLAG    = "dry-run"
 	DRY_RUN_ARG     = "dryRun"
-	DRY_RUN_MESSAGE = "\n*** dry-run mode, changes have NOT been committed. ***\n"
+	DRY_RUN_MESSAGE = "\n" + YELLOW + "*** dry-run mode, changes have NOT been committed. ***" + RESET + "\n"
 	PATH_SPEC_HELP  = `The (optional) domain and path are separated by a ":". appPathGlob supports a glob pattern.
 In the glob, * matches any number of characters, ** matches any number of characters including /.
 all is a shortcut for "*:**", which matches all apps across all domains, including no domain.

cmd/clace/flags.go

@@ -22,6 +22,13 @@ const (
 	FORMAT_JSONL_PRETTY = "jsonl_pretty"
 	FORMAT_CSV          = "csv"
 )
+const (
+	//Terminal colors
+	RESET  = "\033[0m"
+	RED    = "\033[31m"
+	GREEN  = "\033[32m"
+	YELLOW = "\033[33m"
+)
 
 func envString(name string) string {
 	return fmt.Sprintf("CL_%s", strings.ToUpper(strings.ReplaceAll(name, ".", "_")))

cmd/clace/main.go

@@ -198,7 +198,7 @@ func main() {
 		},
 		ExitErrHandler: func(c *cli.Context, err error) {
 			if err != nil {
-				fmt.Fprintf(cli.ErrWriter, "error: %s\n", err)
+				fmt.Fprintf(cli.ErrWriter, RED+"error: %s\n"+RESET, err)
 				os.Exit(1)
 			}
 		},

tests/commander/test_load_app.yaml

@@ -23,8 +23,7 @@ tests:
     stdout: "app /disk_usage_dev is not permitted to load plugin fs.in. Audit the app and approve permissions"
   load0041: # check audit output
     command: '../clace app approve --dry-run /disk_usage_dev | sed "s/app_dev_.*/YYY/g"'
-    stdout:
-      exactly: "App permissions have been approved /disk_usage_dev - YYY\n  Plugins :\n    fs.in\n  Permissions:\n    fs.in.abs [] \n    fs.in.list [] \n    fs.in.find [] \n1 app(s) audited, 1 app(s) approved, 0 app(s) promoted.\n\n*** dry-run mode, changes have NOT been committed. ***\n"
+    stdout: "dry-run mode, changes have NOT been committed."
     exit-code: 0
   load0041a: # test dev app, make sure dry run did not update the in memory cache
     command: curl -su "admin:qwerty" localhost:25222/disk_usage_dev

tests/test_basics.yaml

@@ -12,16 +12,14 @@ tests:
 
   basic020:
     command: CL_CONFIG_FILE=config_basic_client_np.toml ../clace app list /invalid
-    stderr:
-      exactly: "error: 404 page not found" # admin api's are disabled over http
+    stderr: "error: 404 page not found" # admin api's are disabled over http
     stdout:
       line-count: 0
     exit-code: 1
 
   basic030:
     command: CL_CONFIG_FILE=config_basic_client.toml ../clace app list /invalid # No app
-    stderr:
-      exactly: "error: 404 page not found" # admin api's are disabled over http
+    stderr: "error: 404 page not found" # admin api's are disabled over http
     stdout:
       line-count: 0
     exit-code: 1