feat: use env var UNS_NETWORK to select network

src/baseCommand.ts

@@ -1,4 +1,5 @@
 import { Command, flags as oFlags } from "@oclif/command";
+import { FlagInvalidOptionError } from "@oclif/parser/lib/errors";
 import { Client, configManager } from "@uns/crypto";
 import { cli } from "cli-ux";
 import { UNSCLIAPI } from "./api";
@@ -14,6 +15,7 @@ export abstract class BaseCommand extends Command {
             description: "Network used to create UNIK nft token",
             required: true,
             options: UTILS.getNetworksList(),
+            env: "UNS_NETWORK",
         }),
         verbose: oFlags.boolean({
             char: "v",
@@ -49,6 +51,14 @@ export abstract class BaseCommand extends Command {
         /**
          * Configuration
          */
+
+        // This happen when providing network through env var
+        // see Oclif PR: https://github.com/oclif/parser/pull/64
+        // will be removed if PR is accepted
+        if (!UTILS.getNetworksList().includes(flags.network)) {
+            throw new FlagInvalidOptionError(BaseCommand.baseFlags.network, flags.network);
+        }
+
         const networkName = flags.network === "local" ? "testnet" : flags.network;
 
         const networkPreset = configManager.getPreset(networkName);

test/__fixtures__/commands/status.ts

@@ -54,4 +54,16 @@ export const outputCases = [
         args: ["status", "--network", "devnet", "--format", "table", "--verbose"],
         expected: infoNode + statusResultTable,
     },
+    {
+        description: "Should use env var and return devnet status json",
+        args: ["status"],
+        expected: statusResultJson,
+        UNS_NETWORK: "devnet",
+    },
+    {
+        description: "Should not use env var and return devnet status json",
+        args: ["status", "--network", "devnet"],
+        expected: statusResultJson,
+        UNS_NETWORK: "customnetwork",
+    },
 ];

test/commands/status.test.ts

@@ -31,6 +31,7 @@ const applyTestCase = (testCase: any) => {
                 },
             }),
         )
+        .env({ UNS_NETWORK: testCase.UNS_NETWORK })
         .stdout()
         .command(testCase.args)
         .it(testCase.description, ctx => {
@@ -49,5 +50,12 @@ describe("status command", () => {
         // tslint:disable-next-line:no-empty
         .it("Should exit with code 2 if network is not known", _ => {});
 
+    const network = "customNetwork";
+    test.env({ UNS_NETWORK: network })
+        .command(["status"])
+        .catch(err => expect(err.message.match(`Expected --network=${network} to be one of`)))
+        // tslint:disable-next-line:no-empty
+        .it("Should use UNS_NETWORK env var then throw error", _ => {});
+
     outputCases.forEach(testCase => applyTestCase(testCase));
 });