feat: CLI - add support for --config-dir

relative-ci · Aug 25, 2022 · 395f64f · 395f64f
1 parent 2d9dfbf
commit 395f64f
Show file tree

Hide file tree

Showing 4 changed files with 87 additions and 19 deletions.
diff --git a/bin/index.js b/bin/index.js
@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 
+const path = require('path');
 const { get } = require('lodash');
 const { readJSONSync, pathExistsSync } = require('fs-extra');
 const { cosmiconfigSync } = require('cosmiconfig');
@@ -11,7 +12,21 @@ const { agent } = require('../lib/agent');
 const { debug } = require('../lib/utils');
 const LOCALES = require('../locales/en');
 
-const searchConfig = cosmiconfigSync('relativeci').search();
+const args = yargs(hideBin(process.argv))
+  .usage('Usage: $0 OPTIONS')
+
+  .option('config-dir', { describe: 'Config directory', default: '', alias: 'c' })
+
+  .option('commit', { describe: 'Commit SHA', default: '' })
+  .option('commit-message', { describe: 'Commit message', default: '', alias: 'commitMessage' })
+  .option('branch', { describe: 'Branch name', default: '' })
+  .option('pr', { describe: 'Pull Request number', default: '' })
+  .option('slug', { describe: 'Project slug', default: '' })
+
+  .help()
+  .argv;
+
+const searchConfig = cosmiconfigSync('relativeci').search(args['config-dir']);
 debug('Config', searchConfig);
 
 if (!searchConfig) {
@@ -26,7 +41,11 @@ if (!get(config, 'webpack.stats')) {
   process.exit(0);
 }
 
-const webpackArtifactFilepath = get(config, 'webpack.stats');
+// Load webpack stats file relative to the config file
+const webpackArtifactFilepath = path.join(
+  path.dirname(searchConfig.filepath),
+  get(config, 'webpack.stats'),
+);
 
 if (!pathExistsSync(webpackArtifactFilepath)) {
   console.error(LOCALES.CLI_MISSING_STATS_FILE_ERROR);
@@ -49,16 +68,6 @@ const artifactsData = [
   },
 ];
 
-const args = yargs(hideBin(process.argv))
-  .usage('Usage: $0 OPTIONS')
-  .option('commit', { describe: 'Commit SHA', default: '' })
-  .option('commit-message', { describe: 'Commit message', default: '', alias: 'commitMessage' })
-  .option('branch', { describe: 'Branch name', default: '' })
-  .option('pr', { describe: 'Pull Request number', default: '' })
-  .option('slug', { describe: 'Project slug', default: '' })
-  .help()
-  .argv;
-
 debug('CLI arguments', args);
 
 agent(artifactsData, config, args);
diff --git a/test/cli.test.js b/test/cli.test.js
@@ -16,7 +16,22 @@ const MOCK_RESULT = {
 
 const MOCK_SERVER_PORT = 5998;
 
+const createServer = () => http.createServer((req, res) => {
+  res.write(JSON.stringify(MOCK_RESULT));
+  res.end();
+});
+
 describe('CLI', () => {
+  let server;
+
+  beforeAll(() => {
+    server = createServer().listen(MOCK_SERVER_PORT);
+  });
+
+  afterAll(() => {
+    server.close();
+  });
+
   test('should return error if config is missing', (done) => {
     exec('./bin/index.js', (error, stdout, sterr) => {
       expect(sterr).toContain('relativeci.config.js file is missing!');
@@ -39,21 +54,30 @@ describe('CLI', () => {
   });
 
   test('should run agent successfully', (done) => {
-    const server = http.createServer((req, res) => {
-      res.write(JSON.stringify(MOCK_RESULT));
-      res.end();
-    }).listen(MOCK_SERVER_PORT);
-
     exec(
       `cd test/cli/valid-data &&
         RELATIVE_CI_ENDPOINT=http://localhost:${MOCK_SERVER_PORT}/save \
         RELATIVE_CI_SLUG=org/project \
         RELATIVE_CI_KEY=abc123 \
         ../../../bin/index.js
       `,
-      (_, stdout, sterr) => {
+      (_, stdout) => {
+        expect(stdout).toContain('Job #1 done.');
+        done();
+      },
+    );
+  });
+
+  test('should run agent successfully from parent directory', (done) => {
+    exec(
+      `cd test/cli/custom-config-dir &&
+        RELATIVE_CI_ENDPOINT=http://localhost:${MOCK_SERVER_PORT}/save \
+        RELATIVE_CI_SLUG=org/project \
+        RELATIVE_CI_KEY=abc123 \
+        ../../../bin/index.js --config-dir app
+      `,
+      (_, stdout) => {
         expect(stdout).toContain('Job #1 done.');
-        server.close();
         done();
       },
     );

diff --git a/test/cli/custom-config-dir/app/relativeci.config.js b/test/cli/custom-config-dir/app/relativeci.config.js
@@ -0,0 +1,5 @@
+module.exports = {
+  webpack: {
+    stats: './webpack-stats.json',
+  },
+};
diff --git a/test/cli/custom-config-dir/app/webpack-stats.json b/test/cli/custom-config-dir/app/webpack-stats.json
@@ -0,0 +1,30 @@
+{
+  "assets": [
+    {
+      "name": "main.js",
+      "size": 28
+    }
+  ],
+  "chunks": [
+    {
+      "id": 179,
+      "entry": true,
+      "initial": true,
+      "files": [
+        "main.js"
+      ],
+      "names": [
+        "main"
+      ]
+    }
+  ],
+  "modules": [
+    {
+      "name": "./src/index.js",
+      "size": 29,
+      "chunks": [
+        179
+      ]
+    }
+  ]
+}