feat: new ./bin/testplane (#892)

* feat: new ./bin/testplane

* ci: add rebranding branch

.github/workflows/node.js.yml

@@ -5,9 +5,9 @@ name: Node.js CI
 
 on:
     push:
-        branches: [master]
+        branches: [master, rebranding]
     pull_request:
-        branches: [master]
+        branches: [master, rebranding]
 
 jobs:
     build:

bin/hermione

@@ -1,4 +1,4 @@
 #!/usr/bin/env node
 'use strict';
 
-var cli = require('../build/src/cli').run();
+var cli = require('../build/src/cli').run({ cliName: 'hermione' });

bin/testplane

@@ -0,0 +1,4 @@
+#!/usr/bin/env node
+'use strict';
+
+var cli = require('../build/src/cli').run();

package.json

@@ -39,7 +39,8 @@
     "test"
   ],
   "bin": {
-    "hermione": "./bin/hermione"
+    "hermione": "./bin/hermione",
+    "testplane": "./bin/testplane"
   },
   "license": "MIT",
   "dependencies": {

src/cli/index.js

@@ -39,16 +39,16 @@ process.on("unhandledRejection", (reason, p) => {
     }
 });
 
-exports.run = () => {
-    const program = new Command();
+exports.run = (opts = {}) => {
+    const program = new Command(opts.cliName || "testplane");
 
     program.version(pkg.version).allowUnknownOption().option("-c, --config <path>", "path to configuration file");
 
     const configPath = preparseOption(program, "config");
     hermione = Hermione.create(configPath);
 
     program
-        .on("--help", () => logger.log(info.configOverriding))
+        .on("--help", () => logger.log(info.configOverriding(opts)))
         .option("-b, --browser <browser>", "run tests only in specified browser", collect)
         .option("-s, --set <set>", "run tests only in the specified set", collect)
         .option("-r, --require <module>", "require module", collect)

src/cli/info.js

@@ -1,20 +1,24 @@
 "use strict";
 
-exports.configOverriding = `  Overriding config
+exports.configOverriding = (opts = {}) => {
+    const cliName = opts.cliName || "testplane";
+
+    return `  Overriding config
     To override any config option use full option path converted to --kebab-case
 
     Examples:
-      hermione --system-debug true
-      hermione --base-url http://example.com
-      hermione --browsers-firefox-sessions-per-browser 10
+      ${cliName} --system-debug true
+      ${cliName} --base-url http://example.com
+      ${cliName} --browsers-firefox-sessions-per-browser 10
 
     You can also use environment variables converted to snake_case with
-    hermione_ prefix
+    ${cliName}_ prefix
 
     Examples:
-      hermione_system_debug=true hermione
-      hermione_base_url=http://example.com hermione
-      hermione_browsers_firefox_sessions_per_browser=10 hermione
+      ${cliName}_system_debug=true ${cliName}
+      ${cliName}_base_url=http://example.com ${cliName}
+      ${cliName}_browsers_firefox_sessions_per_browser=10 ${cliName}
 
     If both cli option and environment variable are used, cli option takes precedence
 `;
+};

test/src/cli/index.js

@@ -38,11 +38,27 @@ describe("cli", () => {
 
     afterEach(() => sandbox.restore());
 
-    it('should show information about config overriding on "--help"', async () => {
-        await run_("--help");
+    describe("config overriding", () => {
+        it('should show information about config overriding on "--help"', async () => {
+            await run_("--help");
 
-        assert.calledOnce(logger.log);
-        assert.calledWith(logger.log, info.configOverriding);
+            assert.calledOnce(logger.log);
+            assert.calledWith(logger.log, info.configOverriding());
+        });
+
+        it("should show information about testplane by default", async () => {
+            const defaultResult = info.configOverriding();
+
+            assert.isTrue(defaultResult.includes("testplane"));
+            assert.isFalse(defaultResult.includes("hermione"));
+        });
+
+        it("should show information about hermione", async () => {
+            const result = info.configOverriding({ cliName: "hermione" });
+
+            assert.isTrue(result.includes("hermione"));
+            assert.isFalse(result.includes("testplane"));
+        });
     });
 
     it("should create Hermione instance", async () => {