Existing mock should be overwritten when new mock is defined (#109)

* Added test to check if we can overwrite existing mocks

* Added functionality to allow for overwriting of handlers instead of just adding to end of array

* removed arrow function and code cleanup

Author: davidlewallen

src/index.js

@@ -102,7 +102,16 @@ function addHandler(method, handlers, handler) {
       handlers[verb].push(handler);
     });
   } else {
-    handlers[method].push(handler);
+    if (handlers[method].length) {
+      handlers[method].forEach(function(item, index) {
+        if (item[0] === handler[0]) {
+          handlers[method].splice(index, 1, handler);
+        }
+        handlers[method].push(handler);
+      });
+    } else {
+      handlers[method].push(handler);
+    }
   }
 }
 

test/basics.spec.js

@@ -532,4 +532,27 @@ describe('MockAdapter basics', function() {
         expect(data[0].bar).to.equal(123);
       });
   });
+
+  it('should overwrite existing mock', function() {
+    var data = [
+      {
+        bar: 123
+      }
+    ];
+
+    var data2 = [
+      {
+        bar: 321
+      }
+    ];
+
+    mock.onGet('/').reply(200, data);
+    mock.onGet('/').reply(200, data2);
+
+    return instance
+      .get('/')
+      .then(function(response) {
+        expect(response.data).to.deep.equal(data2);
+      });
+  });
 });