Many more tests.

Ref #121

Author: SpaceK33z

test/Advanced.test.js renamed to test/API.test.js

@@ -67,6 +67,14 @@ describe("Advanced API", function() {
 				instance.waitUntilValid(done);
 			});
 		});
+
+		it("should work without callback", function() {
+			var instance = middleware(compiler, options);
+			plugins.done(doneStats);
+			setTimeout(function() {
+				instance.waitUntilValid();
+			});
+		});
 	});
 
 	describe("invalidate", function() {
@@ -91,6 +99,15 @@ describe("Advanced API", function() {
 				doneCalled = true;
 			});
 		});
+
+		it("should work without callback", function(done) {
+			var instance = middleware(compiler, options);
+			instance.invalidate();
+			setTimeout(function() {
+				should.strictEqual(invalidationCount, 1);
+				done();
+			});
+		});
 	});
 
 	describe("close", function() {
@@ -106,6 +123,12 @@ describe("Advanced API", function() {
 				done();
 			});
 		});
+
+		it("should call close on watcher without callback", function() {
+			var instance = middleware(compiler, options);
+			instance.close();
+			should.strictEqual(closeCount, 1);
+		});
 	});
 
 	describe("getFilenameFromUrl", function() {

test/Reporter.test.js

@@ -19,7 +19,9 @@ describe("Reporter", function() {
 	var plugins = {};
 	var compiler = {
 		watch: function() {
-			return {};
+			return {
+				invalidate: function() {}
+			};
 		},
 		plugin: function(name, callback) {
 			plugins[name] = callback;
@@ -152,6 +154,42 @@ describe("Reporter", function() {
 		});
 	});
 
+	describe("wait until bundle valid", function() {
+		it("should print message", function(done) {
+			var instance = middleware(compiler);
+
+			plugins.invalid();
+			instance.invalidate(function myInvalidateFunction() {});
+			setTimeout(function() {
+				should.strictEqual(console.log.callCount, 1);
+				should.strictEqual(console.log.calledWith("webpack: wait until bundle finished: myInvalidateFunction"), true);
+				done();
+			});
+		});
+
+		it("should not print if options.quiet is true", function(done) {
+			var instance = middleware(compiler, { quiet: true });
+
+			plugins.invalid();
+			instance.invalidate();
+			setTimeout(function() {
+				should.strictEqual(console.log.callCount, 0);
+				done();
+			});
+		});
+
+		it("should not print if options.noInfo is true", function(done) {
+			var instance = middleware(compiler, { noInfo: true });
+
+			plugins.invalid();
+			instance.invalidate();
+			setTimeout(function() {
+				should.strictEqual(console.log.callCount, 0);
+				done();
+			});
+		});
+	});
+
 	describe("custom reporter", function() {
 		it("should allow a custom reporter", function(done) {
 			middleware(compiler, {

test/Server.test.js

@@ -41,7 +41,7 @@ describe("Server", function() {
 		it("GET request to bundle file", function(done) {
 			request(app).get("/bundle.js")
 			.expect("Content-Type", "application/javascript")
-			// .expect("Content-Length", "2657")
+			.expect("Content-Length", "2780")
 			.expect("Access-Control-Allow-Origin", "*")
 			.expect(200, /console\.log\("Hey\."\)/, done);
 		});
@@ -64,6 +64,28 @@ describe("Server", function() {
 			.expect("Content-Type", "text/html; charset=utf-8")
 			.expect(404, done);
 		});
+
+		it("request to directory", function(done) {
+			request(app).get("/")
+			.expect("Content-Type", "text/html")
+			.expect("Content-Length", "10")
+			.expect("Access-Control-Allow-Origin", "*")
+			.expect(200, /My\ Index\./, done);
+		});
+
+		it("invalid range header", function(done) {
+			request(app).get("/svg.svg")
+			.set("Range", "bytes=6000-")
+			.expect(416, done);
+		});
+
+		it("valid range header", function(done) {
+			request(app).get("/svg.svg")
+			.set("Range", "bytes=3000-3500")
+			.expect("Content-Length", "501")
+			.expect("Content-Range", "bytes 3000-3500/4778")
+			.expect(206, done);
+		});
 	});
 
 	describe("custom headers", function() {

test/fixtures/server-test/index.html

@@ -0,0 +1 @@
+My Index.

test/fixtures/server-test/index.js

@@ -1,3 +1,4 @@
 require("./svg.svg");
+require("./index.html");
 
 console.log("Hey.");

test/fixtures/server-test/webpack.config.js

@@ -8,7 +8,7 @@ module.exports = {
 	module: {
 		loaders: [
 			{
-				test: /\.svg$/,
+				test: /\.(svg|html)$/,
 				loader: "file",
 				query: { name: "[name].[ext]" }
 			}