Solves #920

nock · May 28, 2017 · a109522 · a109522
1 parent 17d7c25
commit a109522
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/lib/match_body.js b/lib/match_body.js
@@ -59,6 +59,10 @@ function deepEqualExtended(spec, body) {
   }
   if (spec && (spec.constructor === Object || spec.constructor === Array) && body) {
     var keys = Object.keys(spec);
+    var bodyKeys = Object.keys(body);
+    if (keys.length !== bodyKeys.length) {
+      return false;
+    }
     for (var i = 0; i < keys.length; i++) {
       if (!deepEqualExtended(spec[keys[i]], body[keys[i]])) {
         return false;

diff --git a/tests/test_body_match.js b/tests/test_body_match.js
@@ -67,6 +67,24 @@ test('match body with empty object inside', function (t) {
   });
 })
 
+test('doesn\'t match body with mismatching keys', function (t) {
+  nock('http://encodingsareus.com')
+    .post('/', { a: 'a' })
+    .reply(200);
+
+  mikealRequest({
+    url: 'http://encodingsareus.com/',
+    method: 'post',
+    json: {
+      a: 'a',
+      b: 'b'
+    }
+  }, function(err) {
+    assert.ok(err);
+    t.end();
+  });
+});
+
 test('match body with form multipart', function(t) {
 
   nock('http://encodingsareus.com')