#12 Tests + small fixes

Author: pkafel

json-diff.js

@@ -175,7 +175,8 @@ function getDiffRepresentation(left, right) {
   var leftParseResult = _parseJson(left);
   var rightParseResult = _parseJson(right);
 
-  if(leftParseResult["exception"] !== null || rightParseResult["exception"] !== null) throw new ValidationException(leftErrors, rightErrors);
+  if(leftParseResult["exception"] !== null || rightParseResult["exception"] !== null)
+    throw new ValidationException(leftParseResult["exception"], rightParseResult["exception"]);
 
   var leftJson = leftParseResult["json"]; var rightJson = rightParseResult["json"];
   var leftJsonType = _getType(leftParseResult["json"]); var rightJsonType = _getType(rightParseResult["json"]);

test/spec/json-diff-spec.js

@@ -193,5 +193,33 @@ describe("Get Json diff representation", function() {
       expect(result.diff[1].key).toEqual("a");
       expect(result.diff[1].op).toEqual(REMOVE);
       expect(result.diff[1].valueType).toEqual(SCALAR);
-  })
+  });
+
+  it("Diff should throw exception with correct error message when left json is invalid", function() {
+    var call = function() {
+      getDiffRepresentation("Oh, how I wish you were here now...", "{\"a\":\"1\"}");
+    };
+    expect(call).toThrow(new ValidationException("Input is not a valid JSON", null));
+  });
+
+  it("Diff should throw exception with correct error message when right json is invalid", function() {
+    var call = function() {
+      getDiffRepresentation("{\"a\":\"1\"}", "... we were just two lost souls");
+    };
+    expect(call).toThrow(new ValidationException(null, "Input is not a valid JSON"));
+  });
+
+  it("Diff should throw exception with correct error message when left and right json is invalid", function() {
+    var call = function() {
+      getDiffRepresentation("... swimming in a fish bowl...", "... year after year.");
+    };
+    expect(call).toThrow(new ValidationException("Input is not a valid JSON", "Input is not a valid JSON"));
+  });
+
+  it("Diff should throw exception with correct error message when one of jsons is scalar", function() {
+    var call = function() {
+      getDiffRepresentation("11", "{}");
+    };
+    expect(call).toThrow(new ValidationException("Input is not a valid JSON", null));
+  });
 });