Fix test op for whole document comparison. Handle null in deepEquals

Author: briancavalier

lib/deepEquals.js

@@ -37,6 +37,10 @@ function compareArrays(a, b) {
 }
 
 function compareObjects(a, b) {
+	if((a === null && b !== null) || (a !== null && b === null)) {
+		return false;
+	}
+
 	var akeys = Object.keys(a);
 	var bkeys = Object.keys(b);
 

lib/patches.js

@@ -55,8 +55,9 @@ exports.copy = {
 
 function applyTest(x, test) {
 	var pointer = find(x, test.path);
+	var target = pointer.key === void 0 ? pointer.target : pointer.target[pointer.key];
 
-	if(!deepEquals(pointer.target[pointer.key], test.value)) {
+	if(!deepEquals(target, test.value)) {
 		throw new TestFailedError('test failed ' + JSON.stringify(test));
 	}
 

test/jsonPatch-test.js

@@ -5,6 +5,7 @@ var refute = buster.referee.refute;
 var patches = require('../lib/patches');
 var jsonPatch = require('../lib/jsonPatch');
 var InvalidPatchOperationError = require('../lib/InvalidPatchOperationError');
+var TestFailedError = require('../lib/TestFailedError');
 
 buster.testCase('jsonPatch', {
 	'add': {
@@ -161,6 +162,40 @@ buster.testCase('jsonPatch', {
 			assert.exception(function() {
 				jsonPatch.apply([{ op: 'test', path: '/x', value: y }], a);
 			}, 'TestFailedError');
+		},
+
+		'should test whole document': {
+			'when document and value are not null': function() {
+				var doc = { a: { b: 123 } };
+				refute.exception(function() {
+					jsonPatch.apply([{ op: 'test', path: '', value: doc }], doc);
+				});
+			},
+
+			'when document and value are null': function() {
+				refute.exception(function() {
+					jsonPatch.apply([{ op: 'test', path: '', value: null }], null);
+				});
+			},
+
+			'when value is null': function() {
+				var doc = { a: { b: 123 } };
+				assert.exception(function() {
+					jsonPatch.apply([{ op: 'test', path: '', value: doc }], null);
+				}, function(e) {
+					return e instanceof TestFailedError;
+				});
+			},
+
+			'when document is null': function() {
+				var doc = { a: { b: 123 } };
+				assert.exception(function() {
+					jsonPatch.apply([{ op: 'test', path: '', value: null }], doc);
+				}, function(e) {
+					return e instanceof TestFailedError;
+				});
+			}
 		}
+
 	}
 });