Add extra eslint rules

Author: SpaceK33z

.eslintrc

@@ -6,6 +6,7 @@
     "rules": {
         "indent": [2, "tab", { "SwitchCase": 1 }],
         "brace-style": ["error", "1tbs"],
+        "quotes": ["error", "double"],
         "semi": "error",
         "no-eval": "error",
         "eol-last": "error",
@@ -23,6 +24,8 @@
         "no-unused-vars": "error",
         "no-dupe-keys": "error",
         "valid-typeof": "error",
+        "object-curly-spacing": ["error", "always"],
+        "key-spacing": "error",
         "space-infix-ops": "error",
         "no-negated-in-lhs": "error",
         "no-octal": "error",

middleware.js

@@ -56,7 +56,7 @@ module.exports = function(compiler, options) {
 	if(typeof options.log !== "function") options.log = console.log.bind(console);
 	if(typeof options.warn !== "function") options.warn = console.warn.bind(console);
 	if(typeof compiler.outputPath === "string" && compiler.outputPath.lastIndexOf("/", 0) !== 0) {
-		throw new Error('`output.path` needs to be an absolute path or "/".');
+		throw new Error("`output.path` needs to be an absolute path or `/`.");
 	}
 
 	// store our files in memory
@@ -160,13 +160,13 @@ module.exports = function(compiler, options) {
 	}
 
 	function handleRangeHeaders(content, req, res) {
-		res.setHeader('Accept-Ranges', 'bytes');
+		res.setHeader("Accept-Ranges", "bytes");
 		if(req.headers.range) {
 			var ranges = parseRange(content.length, req.headers.range);
 
 			// unsatisfiable
 			if(-1 == ranges) {
-				res.setHeader('Content-Range', 'bytes */' + content.length);
+				res.setHeader("Content-Range", "bytes */" + content.length);
 				res.statusCode = 416;
 			}
 
@@ -176,8 +176,8 @@ module.exports = function(compiler, options) {
 				res.statusCode = 206;
 				var length = content.length;
 				res.setHeader(
-					'Content-Range',
-					'bytes ' + ranges[0].start + '-' + ranges[0].end + '/' + length
+					"Content-Range",
+					"bytes " + ranges[0].start + "-" + ranges[0].end + "/" + length
 				);
 
 				content = content.slice(ranges[0].start, ranges[0].end + 1);
@@ -196,7 +196,7 @@ module.exports = function(compiler, options) {
 			}, req);
 		}
 
-		if(req.method !== 'GET') {
+		if(req.method !== "GET") {
 			return goNext();
 		}
 

test/API.test.js

@@ -51,7 +51,7 @@ describe("Advanced API", function() {
 				if(doneCalled) {
 					done();
 				} else {
-					done(new Error('`waitUntilValid` called before bundle was done'));
+					done(new Error("`waitUntilValid` called before bundle was done"));
 				}
 			});
 			setTimeout(function() {
@@ -83,7 +83,7 @@ describe("Advanced API", function() {
 			plugins.done(doneStats);
 			setTimeout(function() {
 				instance.waitUntilValid(function(stats) {
-					should(stats).have.keys('hasErrors', 'hasWarnings');
+					should(stats).have.keys("hasErrors", "hasWarnings");
 					done();
 				});
 			});
@@ -104,7 +104,7 @@ describe("Advanced API", function() {
 					should.strictEqual(invalidationCount, 1);
 					done();
 				} else {
-					done(new Error('`invalid` called before bundle was done'));
+					done(new Error("`invalid` called before bundle was done"));
 				}
 			});
 			setTimeout(function() {

test/Reporter.test.js

@@ -4,7 +4,7 @@ var fs = require("fs");
 var path = require("path");
 require("mocha-sinon");
 
-var extendedStats = fs.readFileSync(path.join(__dirname, 'fixtures', 'stats.txt'), 'utf8');
+var extendedStats = fs.readFileSync(path.join(__dirname, "fixtures", "stats.txt"), "utf8");
 
 var simpleStats = {
 	hasErrors: function() {
@@ -29,7 +29,7 @@ describe("Reporter", function() {
 	};
 	beforeEach(function() {
 		plugins = {};
-		this.sinon.stub(console, 'log');
+		this.sinon.stub(console, "log");
 	});
 
 	describe("valid/invalid messages", function() {

test/Server.test.js

@@ -12,7 +12,7 @@ describe("Server", function() {
 	var app;
 
 	function listenShorthand(done) {
-		return app.listen(8000, '127.0.0.1', function(err) {
+		return app.listen(8000, "127.0.0.1", function(err) {
 			if(err) done(err);
 			done();
 		});