fix more tests

sokra · sokra · commit 6d183d404468 · 2015-03-05T22:58:03.000+01:00
diff --git a/test/Errors.test.js b/test/Errors.test.js
@@ -70,6 +70,9 @@ describe("Errors", function() {
 				a: "./require-entry-point",
 				b: "./entry-point",
 				c: ["./entry-point2"]
+			},
+			output: {
+				filename: "[name].js"
 			}
 		}, function(errors, warnings) {
 			errors.length.should.be.eql(2);
diff --git a/test/HotModuleReplacementPlugin.test.js b/test/HotModuleReplacementPlugin.test.js
@@ -19,7 +19,7 @@ describe("HotModuleReplacementPlugin", function() {
 			},
 			plugins: [
 				new webpack.HotModuleReplacementPlugin(),
-				new webpack.optimize.OccurenceOrderPlugin()
+				new webpack.optimize.OccurrenceOrderPlugin()
 			]
 		});
 		fs.writeFileSync(entryFile, "1", "utf-8");
diff --git a/test/cases/parsing/context/index.js b/test/cases/parsing/context/index.js
@@ -13,8 +13,6 @@ it("should automatically create contexts", function() {
 	require("./templates/" + tmp + "l").should.be.eql("test template");
 	require("./templates/t" + mpl).should.be.eql("test template");
 	require("./templates/t" + mp + "l").should.be.eql("test template");
-	require("./templates/templateLoader")(templateFull).should.be.eql("test template");
-	require("./templates/templateLoaderIndirect")(templateFull).should.be.eql("test template");
 });
 
 it("should be able to require.resolve with automatical context", function() {
diff --git a/test/cases/parsing/context/warnings.js b/test/cases/parsing/context/warnings.js
@@ -2,5 +2,4 @@ module.exports = [
 	[/Module parse failed/, /dump-file\.txt/, /templates \^\\\.\\\/\.\*\$/],
 	[/Critical dependencies/, /templateLoader\.js/],
 	[/Critical dependencies/, /templateLoaderIndirect\.js/],
-	[/Critical dependencies/, /templateLoaderIndirect\.js/],
 ];
diff --git a/test/cases/parsing/evaluate/index.js b/test/cases/parsing/evaluate/index.js
@@ -1,10 +1,3 @@
-it("should define DEBUG", function() {
-	DEBUG.should.be.eql(false);
-	(typeof DEBUG).should.be.eql("boolean");
-	var x = require(DEBUG ? "fail" : "./a");
-	var y = DEBUG ? require("fail") : require("./a");
-});
-
 it("should evaluate null", function() {
 	var y = null ? require("fail") : require("./a");
 	if(null)
@@ -13,8 +6,8 @@ it("should evaluate null", function() {
 
 it("should short-circut evaluating", function() {
 	var expr;
-	var a = DEBUG && expr ? require("fail") : require("./a");
-	var b = !DEBUG || expr ? require("./a") : require("fail");
+	var a = false && expr ? require("fail") : require("./a");
+	var b = true || expr ? require("./a") : require("fail");
 });
 
 it("should evaluate __dirname and __resourceQuery with replace and substr", function() {
diff --git a/test/configCases/parsing/context/index.js b/test/configCases/parsing/context/index.js
@@ -0,0 +1,5 @@
+it("should automatically create contexts", function() {
+	var template = "tmpl", templateFull = "./tmpl.js";
+	require("../../../cases/parsing/context/templates/templateLoader")(templateFull).should.be.eql("test template");
+	require("../../../cases/parsing/context/templates/templateLoaderIndirect")(templateFull).should.be.eql("test template");
+});
diff --git a/test/configCases/parsing/context/warnings.js b/test/configCases/parsing/context/warnings.js
@@ -0,0 +1,3 @@
+module.exports = [
+	[/Module parse failed/, /dump-file\.txt/, /templates \^\\\.\\\//]
+];
diff --git a/test/configCases/parsing/context/webpack.config.js b/test/configCases/parsing/context/webpack.config.js
@@ -0,0 +1,8 @@
+module.exports = {
+	module: {
+		unknownContextRegExp: /^\.\//,
+		unknownContextCritical: false,
+		exprContextRegExp: /^\.\//,
+		exprContextCritical: false
+	}
+};
diff --git a/test/configCases/plugins/define-plugin/a.js b/test/configCases/plugins/define-plugin/a.js
diff --git a/test/configCases/plugins/define-plugin/index.js b/test/configCases/plugins/define-plugin/index.js
@@ -1,3 +1,10 @@
+it("should define FALSE", function() {
+	FALSE.should.be.eql(false);
+	(typeof TRUE).should.be.eql("boolean");
+	var x = require(FALSE ? "fail" : "./a");
+	var y = FALSE ? require("fail") : require("./a");
+});
+
 it("should define CODE", function() {
 	CODE.should.be.eql(3);
 	(typeof CODE).should.be.eql("number");
diff --git a/test/configCases/plugins/define-plugin/webpack.config.js b/test/configCases/plugins/define-plugin/webpack.config.js
@@ -2,6 +2,8 @@ var DefinePlugin = require("../../../../lib/DefinePlugin");
 module.exports = {
 	plugins: [
 		new DefinePlugin({
+			TRUE: true,
+			FALSE: false,
 			UNDEFINED: undefined,
 			FUNCTION: function(a) { return a + 1; },
 			CODE: "(1+2)",

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ describe("HotModuleReplacementPlugin", function() {`
`19`	`19`	`},`
`20`	`20`	`plugins: [`
`21`	`21`	`new webpack.HotModuleReplacementPlugin(),`
`22`		`- new webpack.optimize.OccurenceOrderPlugin()`
	`22`	`+ new webpack.optimize.OccurrenceOrderPlugin()`
`23`	`23`	`]`
`24`	`24`	`});`
`25`	`25`	`fs.writeFileSync(entryFile, "1", "utf-8");`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+module.exports = [`
	`2`	`+ [/Module parse failed/, /dump-file\.txt/, /templates \^\\\.\\\//]`
	`3`	`+];`