fix(cli): ignore --env flag for commands that don't use a project config

garden-service/src/cli/cli.ts

@@ -317,7 +317,7 @@ export class GardenCli {
       do {
         try {
           if (command.noProject) {
-            garden = await makeDummyGarden(root, contextOpts)
+            garden = await makeDummyGarden(root, { ...contextOpts, environmentName: undefined })
           } else {
             garden = await Garden.factory(root, contextOpts)
           }

garden-service/test/unit/src/cli/cli.ts

@@ -33,6 +33,26 @@ describe("cli", () => {
       const { result } = await cli.parse(["test-command", "some", "args"])
       expect(result).to.eql({ args: { _: ["some", "args"] } })
     })
+
+    it(`ignore the env flag when a command has noProject=true`, async () => {
+      class TestCommand extends Command {
+        name = "test-command"
+        help = "halp!"
+        noProject = true
+
+        async action({ args }) {
+          return { result: { args } }
+        }
+      }
+
+      const command = new TestCommand()
+      const cli = new GardenCli()
+      cli.addCommand(command, cli["program"])
+
+      const { result, errors } = await cli.parse(["test-command", "--env", "invalid-env"])
+      expect(errors).to.eql([])
+      expect(result).to.eql({ args: { _: [] } })
+    })
   })
 
   describe("makeDummyGarden", () => {

garden-service/test/unit/src/commands/plugins.ts

@@ -123,4 +123,20 @@ describe("PluginsCommand", () => {
 
     expect(result).to.eql({ args: ["foo"] })
   })
+
+  it(`ignore the env flag when printing help text`, async () => {
+    const garden = await TestGarden.factory(tmpDir.path, { plugins: [testPluginA, testPluginB] })
+    const log = garden.log
+
+    const result = await command.action({
+      garden,
+      log,
+      headerLog: log,
+      footerLog: log,
+      args: { plugin: undefined, command: undefined },
+      opts: withDefaultGlobalOpts({ env: "invalid-env" }),
+    })
+
+    expect(result.errors).to.be.undefined
+  })
 })