cli: allow array value for --ouput-library. Fixes #557

webpack · Aug 9, 2018 · 0fb0daa · 0fb0daa
1 parent 72c70f2
commit 0fb0daa
Show file tree

Hide file tree

Showing 9 changed files with 43 additions and 2 deletions.
diff --git a/bin/config-yargs.js b/bin/config-yargs.js
@@ -137,7 +137,7 @@ module.exports = function(yargs) {
 				group: OUTPUT_GROUP
 			},
 			"output-library": {
-				type: "string",
+				type: "array",
 				describe: "Expose the exports of the entry point as library",
 				group: OUTPUT_GROUP,
 				requiresArg: true

diff --git a/bin/convert-argv.js b/bin/convert-argv.js
@@ -441,7 +441,8 @@ module.exports = function(...args) {
 
 		ifArg("output-library", function(value) {
 			ensureObject(options, "output");
-			options.output.library = value;
+			ensureArray(options.output, "library");
+			options.output.library.push(value);
 		});
 
 		ifArg("output-library-target", function(value) {

diff --git a/test/BinTestCases.test.js b/test/BinTestCases.test.js
@@ -68,9 +68,11 @@ const defaultArgs = loadOptsFile(path.join(casesPath, "test.opts"));
 
 describe("BinTestCases", function() {
 	const tests = findTestsRecursive(casesPath);
+	const patternFilter = process.env.BIN_TEST_CASES_GREP ? new RegExp(process.env.BIN_TEST_CASES_GREP) : null;
 
 	tests.forEach(testDirectory => {
 		const testName = testDirectory.replace(casesPath, "");
+		if (patternFilter && !testName.match(patternFilter)) return;
 		const testArgs = getTestSpecificArguments(testDirectory) || defaultArgs;
 		const testAssertions = require(path.join(testDirectory, "stdin.js"));
 		const outputPath = path.join(

diff --git a/test/binCases/output/output-library-many/index.js b/test/binCases/output/output-library-many/index.js
@@ -0,0 +1 @@
+module.exports = "index";
diff --git a/test/binCases/output/output-library-many/stdin.js b/test/binCases/output/output-library-many/stdin.js
@@ -0,0 +1,14 @@
+"use strict";
+
+const fs = require("fs");
+const path = require("path");
+
+module.exports = function testAssertions(code, stdout, stderr) {
+	expect(code).toBe(0);
+	expect(stdout).toEqual(expect.anything());
+	expect(stdout[7]).toMatch(/index\.js.*\{0\}/);
+	expect(stderr).toHaveLength(0);
+
+	const output = fs.readFileSync(path.join(__dirname, "../../../js/bin/output/output-library-many/main.js"), "utf-8");
+	expect(output).toContain("window.key1=window.key1||{},window.key1.key2=function");
+};
diff --git a/test/binCases/output/output-library-many/test.opts b/test/binCases/output/output-library-many/test.opts
@@ -0,0 +1,4 @@
+./index.js
+--target async-node
+--output-library-target window
+--output-library key1 --output-library key2
diff --git a/test/binCases/output/output-library-single/index.js b/test/binCases/output/output-library-single/index.js
@@ -0,0 +1 @@
+module.exports = "index";
diff --git a/test/binCases/output/output-library-single/stdin.js b/test/binCases/output/output-library-single/stdin.js
@@ -0,0 +1,14 @@
+"use strict";
+
+const fs = require("fs");
+const path = require("path");
+
+module.exports = function testAssertions(code, stdout, stderr) {
+	expect(code).toBe(0);
+	expect(stdout).toEqual(expect.anything());
+	expect(stdout[7]).toMatch(/index\.js.*\{0\}/);
+	expect(stderr).toHaveLength(0);
+
+	const output = fs.readFileSync(path.join(__dirname, "../../../js/bin/output/output-library-single/main.js"), "utf-8");
+	expect(output).toContain("window.key1=function");
+};
diff --git a/test/binCases/output/output-library-single/test.opts b/test/binCases/output/output-library-single/test.opts
@@ -0,0 +1,4 @@
+./index.js
+--target async-node
+--output-library-target window
+--output-library key1