- Update inspect_args to also print managed environment_variables

DannyBen · Dec 22, 2023 · 4fe17b7 · 4fe17b7
1 parent 6efc553
commit 4fe17b7
Show file tree

Hide file tree

Showing 7 changed files with 36 additions and 0 deletions.
diff --git a/lib/bashly/views/command/inspect_args.gtx b/lib/bashly/views/command/inspect_args.gtx
@@ -25,5 +25,13 @@
 >     for k in "${sorted_keys[@]}"; do echo "- \${deps[$k]} = ${deps[$k]}"; done
 >   fi
 >
+if environment_variables.any?
+  >   echo
+  >   echo environment variables:
+  environment_variables.each do |env_var|
+    >   echo "- \${{ env_var.name.upcase }} = ${<%= env_var.name.upcase %>:-unset}"
+  end
+  >
+end
 > }
 > 
diff --git a/spec/approvals/examples/command-aliases b/spec/approvals/examples/command-aliases
@@ -69,6 +69,9 @@ Examples:
 # you can edit it freely and regenerate (it will not be overwritten)
 args:
 - ${args[source]} = somefile
+
+environment variables:
+- $API_KEY = unset
 + ./cli upload --help
 cli upload - Upload a file
 
@@ -119,3 +122,6 @@ Examples:
 # you can edit it freely and regenerate (it will not be overwritten)
 args:
 - ${args[source]} = somefile
+
+environment variables:
+- $API_KEY = unset
diff --git a/spec/approvals/examples/commands b/spec/approvals/examples/commands
@@ -83,6 +83,9 @@ args:
 - ${args[--force]} = 1
 - ${args[source]} = sourcefile
 - ${args[target]} = targetfile
+
+environment variables:
+- $API_KEY = unset
 + ./cli upload --help
 cli upload - Upload a file
 
@@ -115,3 +118,6 @@ missing required flag: --user, -u USER
 args:
 - ${args[source]} = sourcefile
 - ${args[--user]} = username
+
+environment variables:
+- $API_KEY = unset
diff --git a/spec/approvals/examples/environment-variables b/spec/approvals/examples/environment-variables
@@ -65,6 +65,9 @@ missing required environment variable: MY_SECRET
 # code for 'cli verify' goes here
 # you can edit it freely and regenerate (it will not be overwritten)
 args: none
+
+environment variables:
+- $API_KEY = unset
 environment:
 - API_KEY=
 - ENVIRONMENT=development
@@ -76,6 +79,9 @@ environment:
 # code for 'cli verify' goes here
 # you can edit it freely and regenerate (it will not be overwritten)
 args: none
+
+environment variables:
+- $API_KEY = unset
 environment:
 - API_KEY=
 - ENVIRONMENT=production

diff --git a/spec/approvals/examples/multiline b/spec/approvals/examples/multiline
@@ -57,6 +57,9 @@ Examples:
 # code for 'multi multiline' goes here
 # you can edit it freely and regenerate (it will not be overwritten)
 args: none
+
+environment variables:
+- $MULTI_VITAMIN = unset
 + ./multi multiline -h
 multi multiline
 

diff --git a/spec/approvals/fixtures/environment-variables-initialize b/spec/approvals/fixtures/environment-variables-initialize
@@ -16,6 +16,10 @@ PASS: NESTED_VAR is empty
 # you can edit it freely and regenerate (it will not be overwritten)
 args:
 - ${args[source]} = source
+
+environment variables:
+- $API_KEY = must-be-available-in-initialize
+- $CONFIG_PATH = somepath
 + export API_KEY=override-value
 + API_KEY=override-value
 + ./cli

diff --git a/spec/approvals/fixtures/partials-extension b/spec/approvals/fixtures/partials-extension
@@ -102,3 +102,6 @@ Environment Variables:
 # you can edit it freely and regenerate (it will not be overwritten)
 args:
 - ${args[source]} = something
+
+environment variables:
+- $API_KEY = unset