Fix lint issues

Author: andersnm

lib/validator.js

@@ -33,19 +33,19 @@ function escapeEvalString(str) {
 	// Based on https://github.com/joliss/js-string-escape
 	return str.replace(escapeEvalRegex, function (character) {
 		switch (character) {
-		  case '"':
-		  case "'":
-		  case '\\':
-			return '\\' + character
-		  // Four possible LineTerminator characters need to be escaped:
-		  case '\n':
-			return '\\n'
-		  case '\r':
-			return '\\r'
-		  case '\u2028':
-			return '\\u2028'
-		  case '\u2029':
-			return '\\u2029'
+			case "\"":
+			case "'":
+			case "\\":
+				return "\\" + character;
+			// Four possible LineTerminator characters need to be escaped:
+			case "\n":
+				return "\\n";
+			case "\r":
+				return "\\r";
+			case "\u2028":
+				return "\\u2028";
+			case "\u2029":
+				return "\\u2029";
 		}
 	});
 }
@@ -135,7 +135,7 @@ Validator.prototype.compileSchemaObject = function(schemaObject) {
 	for (let i = 0; i < compiledObject.properties.length; i++) {
 		const property = compiledObject.properties[i];
 		const name = escapeEvalString(property.name);
-		const propertyValueExpr = identifierRegex.test(name) ? `value.${name}` : `value[\"${name}\"]`;
+		const propertyValueExpr = identifierRegex.test(name) ? `value.${name}` : `value["${name}"]`;
 
 		sourceCode.push(`propertyPath = (path !== undefined ? path + ".${name}" : "${name}");`);
 		if (Array.isArray(property.compiledType)) {

test/validator.spec.js

@@ -1114,12 +1114,11 @@ describe("Test irregular object property names", () => {
 
 	it("should compile schema with reserved keyword", () => {
 		// Reserved keywords are permitted as unquoted property names in ES5+. There is no special support for these
-		const o = {};
 		const schema = {
-			'for': { type: "string" },
-			'goto': { type: "string" },
-			'var': { type: "string" },
-			'try': { type: "string" },
+			for: { type: "string" },
+			goto: { type: "string" },
+			var: { type: "string" },
+			try: { type: "string" },
 		};
 
 		const res = v.validate({