Merge pull request #2 from szenti/master

feat(exponential-delay): fix exponential strategy, add tests for exponential base

Author: sonicoder86

src/strategies/delay/exponential-delay.js

@@ -9,8 +9,8 @@ class ExponentialDelay {
   }
 
   delay(attempts) {
-    const delayInSeconds = this._delayInSeconds[attempts - 1];
-    const delayInMilliSeconds = delayInSeconds * 1000 * this._multiplier;
+    const delayInSeconds = this._delayInSeconds[attempts - 1] * this._multiplier;
+    const delayInMilliSeconds = delayInSeconds * 1000;
     return Delay.wait(delayInMilliSeconds, delayInSeconds);
   }
 

src/strategies/delay/exponential-delay.spec.js

@@ -34,9 +34,27 @@ describe('Exponential Strategy', () => {
       const subject = new ExponentialDelay(3, 10);
       this.disableDelay();
 
+      yield subject.delay(1);
       yield subject.delay(2);
 
-      expect(Delay.wait).to.have.been.calledWith(20000, 2);
+      expect(Delay.wait.firstCall.args).to.eql([10000, 10]);
+      expect(Delay.wait.secondCall.args).to.eql([20000, 20]);
+    });
+
+  });
+
+
+  describe('#delay with multiplier and exponential', () => {
+
+    it('should wait on retry', function*() {
+      const subject = new ExponentialDelay(3, 2, 3);
+      this.disableDelay();
+
+      yield subject.delay(1);
+      yield subject.delay(2);
+
+      expect(Delay.wait.firstCall.args).to.eql([2000, 2]);
+      expect(Delay.wait.secondCall.args).to.eql([6000, 6]);
     });
 
   });