tests(binTestCases): refactor to make tests pass (#472)

Author: evenstensberg

.eslintignore

@@ -1,4 +1,5 @@
 **/__testfixtures__/*
 coverage
 test
-docs
+docs
+node_modules

test/BinTestCases.test.js

@@ -167,8 +167,8 @@ describe("BinTestCases", function() {
 					try {
 						testAssertions(env.code, stdout, stderr);
 					} catch(e) {
-						console.log(`### stderr ###\n${env.stderr.join("")}`);
-						console.log(`### stdout ###\n${env.stdout.join("")}`);
+					//	console.log(`### stderr ###\n${env.stderr.join("")}`);
+					//	console.log(`### stdout ###\n${env.stdout.join("")}`);
 						throw e;
 					}
 				});

test/binCases/config-name/found-many/stdin.js

@@ -3,7 +3,7 @@
 module.exports = function testAssertions(code, stdout, stderr) {
 	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
-	expect(stdout[9]).toContain("./index2.js");
-	expect(stdout[17]).toContain("./index3.js");
+	expect(stdout[8]).toContain("./index2.js");
+	expect(stdout[15]).toContain("./index3.js");
 	expect(stderr).toHaveLength(0);
 };

test/binCases/config-name/found-one/stdin.js

@@ -3,7 +3,6 @@
 module.exports = function testAssertions(code, stdout, stderr) {
 	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
-
-	expect(stdout[7]).toContain("./index2.js");
+	expect(stdout[6]).toContain("./index2.js");
 	expect(stderr).toHaveLength(0);
 };

test/binCases/config-type/array/stdin.js

@@ -7,7 +7,7 @@ module.exports = function testAssertions(code, stdout, stderr) {
 	expect(stdout[1]).toContain("Version: ");
 	expect(stdout[2]).toContain("Child");
 	expect(stdout[7]).toContain("entry-a.bundle.js");
-	expect(stdout[10]).toContain("Child");
-	expect(stdout[15]).toContain("entry-b.bundle.js");
+	expect(stdout[9]).toContain("Child");
+	expect(stdout[14]).toContain("entry-b.bundle.js");
 	expect(stderr).toHaveLength(0);
 };

test/binCases/configFile/plugins-precedence/stdin.js

@@ -3,6 +3,6 @@
 module.exports = function testAssertions(code, stdout, stderr) {
 	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
-	expect(stdout[7]).toContain("ok.js");
+	expect(stdout[6]).toContain("ok.js");
 	expect(stderr).toHaveLength(0);
 };

test/binCases/configFile/profile/stdin.js

@@ -4,8 +4,8 @@ module.exports = function testAssertions(code, stdout, stderr) {
 	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
 
-	expect(stdout[8]).toContain("factory:");
-	expect(stdout[10]).toContain("factory:");
-	expect(stdout[12]).toContain("factory:");
+	expect(stdout[7]).toContain("factory:");
+	expect(stdout[9]).toContain("factory:");
+	expect(stdout[11]).toContain("factory:");
 	expect(stderr).toHaveLength(0);
 };

test/binCases/entry/cli-override/stdin.js

@@ -4,6 +4,6 @@ module.exports = function testAssertions(code, stdout, stderr) {
 	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
 	expect(stdout[5]).toContain("cliEntry.js");
-	expect(stdout[7]).toContain("index.js");
+	expect(stdout[6]).toContain("index.js");
 	expect(stderr).toHaveLength(0);
 };

test/binCases/entry/multi-file/stdin.js

@@ -4,8 +4,8 @@ module.exports = function testAssertions(code, stdout, stderr) {
 	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
 	expect(stdout[5]).toContain("null.js");
-	expect(stdout[7]).toMatch(/a\.js.*\{0\}/);
-	expect(stdout[8]).toMatch(/index\.js.*\{0\}/);
-	expect(stdout[9]).toMatch(/multi.*index\.js.*a\.js/); // should have multi-file entry
+	expect(stdout[6]).toMatch(/a\.js.*\{0\}/);
+	expect(stdout[7]).toMatch(/index\.js.*\{0\}/);
+	expect(stdout[8]).toMatch(/multi.*index\.js.*a\.js/); // should have multi-file entry
 	expect(stderr).toHaveLength(0);
 };

test/binCases/entry/named-entry/stdin.js

@@ -4,8 +4,8 @@ module.exports = function testAssertions(code, stdout, stderr) {
 	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
 	expect(stdout[5]).toContain("null.js");
-	expect(stdout[7]).toContain("foo.js"); // named entry from --entry foo=./a.js
-	expect(stdout[9]).toMatch(/index\.js.*\{0\}/);
-	expect(stdout[10]).toMatch(/a\.js.*\{1\}/);
+	expect(stdout[6]).toContain("foo.js"); // named entry from --entry foo=./a.js
+	expect(stdout[7]).toMatch(/index\.js.*\{0\}/);
+	expect(stdout[8]).toMatch(/a\.js.*\{1\}/);
 	expect(stderr).toHaveLength(0);
 };