feat: reset flags

Author: snitin315

bin/cli-flags.js

@@ -1,6 +1,7 @@
 'use strict';
 
-const normalizeOption = (option) => (typeof option === 'object' ? option : {});
+const normalizeOption = (option) =>
+  typeof option === 'object' && !Array.isArray(option) ? option : {};
 
 module.exports = {
   bonjour: {
@@ -344,6 +345,23 @@ module.exports = {
     simpleType: 'string',
     multiple: true,
   },
+  'firewall-reset': {
+    configs: [
+      {
+        type: 'reset',
+        multiple: false,
+        description: 'Clear all items provided in firewall configuration.',
+        path: 'firewall',
+      },
+    ],
+    description: 'Clear all items provided in firewall configuration.',
+    simpleType: 'boolean',
+    multiple: false,
+    processor(opts) {
+      opts.firewall = opts.firewall || [];
+      delete opts.firewallReset;
+    },
+  },
   'history-api-fallback': {
     configs: [
       {
@@ -647,6 +665,60 @@ module.exports = {
       delete opts.openApp;
     },
   },
+  'open-reset': {
+    configs: [
+      {
+        type: 'reset',
+        multiple: false,
+        description: 'Clear all items provided in open configuration.',
+        path: 'open',
+      },
+    ],
+    description: 'Clear all items provided in open configuration.',
+    simpleType: 'boolean',
+    multiple: false,
+    processor(opts) {
+      opts.open = opts.open || [];
+      delete opts.openReset;
+    },
+  },
+  'open-target-reset': {
+    configs: [
+      {
+        type: 'reset',
+        multiple: false,
+        description: 'Clear all items provided in open.target configuration.',
+        path: 'open.target',
+      },
+    ],
+    description: 'Clear all items provided in open.target configuration.',
+    simpleType: 'boolean',
+    multiple: false,
+    processor(opts) {
+      opts.open = normalizeOption(opts.open);
+      opts.open.target = opts.openTarget || [];
+      delete opts.openTargetReset;
+    },
+  },
+  'open-app-name-reset': {
+    configs: [
+      {
+        type: 'reset',
+        multiple: false,
+        description: 'Clear all items provided in open.app.name configuration.',
+        path: 'open.app.name',
+      },
+    ],
+    description: 'Clear all items provided in open.app.name configuration.',
+    simpleType: 'boolean',
+    multiple: false,
+    processor(opts) {
+      opts.open = normalizeOption(opts.open);
+      opts.open.app = normalizeOption(opts.open.app);
+      opts.open.app.name = opts.openAppName || [];
+      delete opts.openAppNameReset;
+    },
+  },
   port: {
     configs: [
       {
@@ -794,6 +866,42 @@ module.exports = {
       delete opts.staticWatch;
     },
   },
+  'static-reset': {
+    configs: [
+      {
+        type: 'reset',
+        multiple: false,
+        description: 'Clear all items provided in static configuration.',
+        path: 'static',
+      },
+    ],
+    description: 'Clear all items provided in static configuration.',
+    simpleType: 'boolean',
+    multiple: false,
+    processor(opts) {
+      opts.static = opts.static || [];
+      delete opts.staticReset;
+    },
+  },
+  'static-public-path-reset': {
+    configs: [
+      {
+        type: 'reset',
+        multiple: false,
+        description:
+          'Clear all items provided in static.publicPath configuration.',
+        path: 'static.publicPath',
+      },
+    ],
+    description: 'Clear all items provided in static.publicPath configuration.',
+    simpleType: 'boolean',
+    multiple: false,
+    processor(opts) {
+      opts.static = normalizeOption(opts.static);
+      opts.static.publicPath = opts.staticPublicPath || [];
+      delete opts.staticPublicPathReset;
+    },
+  },
   'watch-files': {
     configs: [
       {
@@ -831,4 +939,40 @@ module.exports = {
       delete opts.watchFilesPaths;
     },
   },
+  'watch-files-reset': {
+    configs: [
+      {
+        type: 'reset',
+        multiple: false,
+        description: 'Clear all items provided in watchFiles configuration.',
+        path: 'watchFiles',
+      },
+    ],
+    description: 'Clear all items provided in watchFiles configuration.',
+    simpleType: 'boolean',
+    multiple: false,
+    processor(opts) {
+      opts.watchFiles = opts.watchFiles || [];
+      delete opts.watchFilesReset;
+    },
+  },
+  'watch-files-paths-reset': {
+    configs: [
+      {
+        type: 'reset',
+        multiple: false,
+        description:
+          'Clear all items provided in watchFiles.paths configuration.',
+        path: 'watchFiles.paths',
+      },
+    ],
+    description: 'Clear all items provided in watchFiles.paths configuration.',
+    simpleType: 'boolean',
+    multiple: false,
+    processor(opts) {
+      opts.watchFiles = normalizeOption(opts.watchFiles);
+      opts.watchFiles.paths = opts.watchFilesPaths || [];
+      delete opts.watchFilesPathsReset;
+    },
+  },
 };

test/cli/cli.test.js

@@ -342,6 +342,15 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--firewall-reset', (done) => {
+    testBin('--firewall-reset --firewall hello.com')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it('--http2', (done) => {
     testBin('--http2')
       .then((output) => {
@@ -679,6 +688,33 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--open-reset', (done) => {
+    testBin('--open-reset --open /third.html')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
+  it('--open-reset --open-target', (done) => {
+    testBin('--open-reset --open-target')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
+  it('--open-reset --open-target <value>', (done) => {
+    testBin('--open-reset --open-target /third.html')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it('--open-app google-chrome', (done) => {
     testBin('--open-app google-chrome')
       .then((output) => {
@@ -697,6 +733,15 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--open-app-name-reset', (done) => {
+    testBin('--open-app-name-reset --open-app-name firefox')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it('--open-target', (done) => {
     testBin('--open-target')
       .then((output) => {
@@ -724,6 +769,15 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--open-target-reset', (done) => {
+    testBin('--open-target-reset --open-target first.html')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it(' --open --open-target index.html', (done) => {
     testBin('--open --open-target index.html')
       .then((output) => {
@@ -830,6 +884,34 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--static-reset', (done) => {
+    testBin(
+      `--static-reset --static ${path.resolve(
+        __dirname,
+        '../fixtures/static/webpack.config.js'
+      )}`
+    )
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
+  it('--static-reset --static-directory <value>', (done) => {
+    testBin(
+      `--static-reset --static-directory ${path.resolve(
+        __dirname,
+        '../fixtures/static/webpack.config.js'
+      )}`
+    )
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it('--static-directory', (done) => {
     testBin(
       `--static-directory ${path.resolve(
@@ -858,6 +940,24 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--static-public-path', (done) => {
+    testBin('--static-public-path /public')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
+  it('--static-public-path-reset', (done) => {
+    testBin('--static-public-path-reset --static-public-path /new-public')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it('--static-serve-index', (done) => {
     testBin('--static-serve-index')
       .then((output) => {
@@ -929,6 +1029,28 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--watch-files-reset', (done) => {
+    const watchDirectory = path.resolve(__dirname, '../fixtures/static/static');
+
+    testBin(`--watch-files-reset --watch-files ${watchDirectory}`)
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
+  it('--watch-files-reset --watch-files-paths <value>', (done) => {
+    const watchDirectory = path.resolve(__dirname, '../fixtures/static/static');
+
+    testBin(`--watch-files-reset --watch-files-paths ${watchDirectory}`)
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it('--watch-files-paths', (done) => {
     const watchDirectory = path.resolve(__dirname, '../fixtures/static/static');
 
@@ -940,6 +1062,17 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--watch-files-paths-reset', (done) => {
+    const watchDirectory = path.resolve(__dirname, '../fixtures/static/static');
+
+    testBin(`--watch-files-paths-reset --watch-files-paths ${watchDirectory}`)
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it('should accept the promise function of webpack.config.js', (done) => {
     testBin(
       false,