Merge pull request #8 from arb/fix-bug

Fixed bug where this basically didn't work at all.

Author: cjihrig

.eslintignore

@@ -10,7 +10,7 @@ var internals = {
 internals.store.lookUp = function (obj, method) {
   var i = this.length;
   var lookup;
-  while (i-- && !lookup) {
+  while (!lookup && i--) {
     var item = this[i];
 
     if (item.obj === obj && item.method === method) {
@@ -40,8 +40,9 @@ exports.replace = function (obj, method, fn) {
   var result = {
     original: obj[method].bind(obj),
     restore: function () {
-      var find = internals.store.lookUp(this._stand.obj, this._stand.method);
+      var find = internals.store.lookUp(obj, method);
       internals.store.splice(find.index, 1);
+      obj[method] = this.original;
     },
     _stand: stand
   };

lib/index.js

@@ -25,7 +25,7 @@ describe('stand-in', function () {
     it('correctly restores a replaced method', function (done) {
       var foo = {
         bar: function (valueone, valuetwo) {
-          console.log('%s, %s');
+          throw new Error('this is a test error');
         }
       };
 
@@ -35,7 +35,7 @@ describe('stand-in', function () {
 
         replace.restore();
 
-        expect(foo.bar).to.deep.equal(foo.bar);
+        expect(foo.bar).to.throw(Error, 'this is a test error');
         done();
       });