fix: ensure proper env prefix (#2260)

fixes #2259
adobe · Oct 18, 2023 · 29a42ba · 29a42ba
1 parent e274252
commit 29a42ba
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/src/cli.js b/src/cli.js
@@ -121,7 +121,7 @@ export default class CLI {
       .scriptName('aem')
       .usage('Usage: $0 <command> [options]')
       .parserConfiguration({ 'camel-case-expansion': false })
-      .env('AEM')
+      .env('AEM_')
       .check((a) => envAwareStrict(a, argv.parsed.aliases))
       .showHelpOnFail(true)
       .fail(this._failFn)

diff --git a/test/fixtures/all.env b/test/fixtures/all.env
@@ -1,6 +1,9 @@
 #
 # env file for testing that contains all possible and impossible values
 #
+# ignored
+AEMCOMPANIONPATH=foobar
+AEM_COMPANIONPATH=foobar
 
 # global
 AEM_LOG_LEVEL=debug
@@ -13,6 +16,7 @@ AEM_TARGET=foo
 AEM_FILES=*.htl,*.js
 
 # up
+AEM_PRINT_INDEX=true
 AEM_HOST=www.project-helix.io
 AEM_OPEN=false
 AEM_LIVERELOAD=false

diff --git a/test/up-cli.test.js b/test/up-cli.test.js
@@ -70,6 +70,7 @@ describe('hlx up', () => {
     sinon.assert.calledWith(mockUp.withLiveReload, false);
     sinon.assert.calledWith(mockUp.withHttpPort, 1234);
     sinon.assert.calledWith(mockUp.withBindAddr, '*');
+    sinon.assert.calledWith(mockUp.withPrintIndex, true);
     sinon.assert.calledOnce(mockUp.run);
   });