feat: allow multiple targets (#1799)

* feat: allow multiple targets Co-authored-by: Anshuman Verma <anshu.av97@gmail.com> Co-authored-by: Rishabh Chawla <rishabh31121999@gmail.com>
webpack · Sep 17, 2020 · 1724ddb · 1724ddb
1 parent 2473978
commit 1724ddb
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 6 deletions.
diff --git a/packages/serve/__tests__/__snapshots__/parseArgs.test.ts.snap b/packages/serve/__tests__/__snapshots__/parseArgs.test.ts.snap
@@ -42,8 +42,7 @@ Object {
   "webpackArgs": Object {
     "color": true,
     "config": Array [],
-    "mode": "production",
-    "target": "node",
+    "mode": "development",
   },
 }
 `;
diff --git a/packages/serve/__tests__/parseArgs.test.ts b/packages/serve/__tests__/parseArgs.test.ts
@@ -27,7 +27,7 @@ describe('parseArgs', () => {
     });
 
     it('parses webpack and dev server args', () => {
-        const args = parseArgs(cli, ['--bonjour', '--target=node', '--port', '8080']);
+        const args = parseArgs(cli, ['--bonjour', '--mode=development', '--port', '8080']);
         expect(args).toMatchSnapshot();
         expect(errorMock.mock.calls.length).toEqual(0);
         expect(processExitSpy.mock.calls.length).toEqual(0);

diff --git a/packages/webpack-cli/lib/utils/cli-flags.js b/packages/webpack-cli/lib/utils/cli-flags.js
@@ -144,6 +144,7 @@ const core = [
         alias: 't',
         type: String,
         group: ADVANCED_GROUP,
+        multiple: cli !== undefined,
         description: 'Sets the build target e.g. node',
         link: 'https://webpack.js.org/configuration/target/#target',
     },
@@ -259,7 +260,7 @@ const core = [
 // Extract all the flags being exported from core. A list of cli flags generated by core
 // can be found here https://github.com/webpack/webpack/blob/master/test/__snapshots__/Cli.test.js.snap
 let flagsFromCore =
-    typeof cli !== 'undefined'
+    cli !== undefined
         ? Object.entries(cli.getArguments()).map(([flag, meta]) => {
               if (meta.simpleType === 'string') {
                   meta.type = String;

diff --git a/test/target/flag-test/target-flag.test.js b/test/target/flag-test/target-flag.test.js
@@ -10,7 +10,11 @@ describe('--target flag', () => {
         it(`should accept ${val} with --target flag`, (done) => {
             const { stdout, stderr } = run(__dirname, ['--target', `${val}`]);
             expect(stderr).toBeFalsy();
-            expect(stdout).toContain(`target: '${val}'`);
+            if (isWebpack5) {
+                expect(stdout).toContain(`target: [ '${val}' ]`);
+            } else {
+                expect(stdout).toContain(`target: '${val}'`);
+            }
 
             stat(resolve(__dirname, 'bin/main.js'), (err, stats) => {
                 expect(err).toBe(null);
@@ -22,7 +26,11 @@ describe('--target flag', () => {
         it(`should accept ${val} with -t alias`, (done) => {
             const { stdout, stderr } = run(__dirname, ['-t', `${val}`]);
             expect(stderr).toBeFalsy();
-            expect(stdout).toContain(`target: '${val}'`);
+            if (isWebpack5) {
+                expect(stdout).toContain(`target: [ '${val}' ]`);
+            } else {
+                expect(stdout).toContain(`target: '${val}'`);
+            }
 
             stat(resolve(__dirname, 'bin/main.js'), (err, stats) => {
                 expect(err).toBe(null);
@@ -40,4 +48,12 @@ describe('--target flag', () => {
             expect(stderr).toContain('Invalid configuration object');
         }
     });
+
+    if (isWebpack5) {
+        it('should allow multiple targets', () => {
+            const { stderr, stdout } = run(__dirname, ['--target', 'node', '--target', 'async-node']);
+            expect(stderr).toBeFalsy();
+            expect(stdout).toContain(`target: [ 'node', 'async-node' ]`);
+        });
+    }
 });