Test fix #3

ethereum-alarm-clock · Nov 20, 2018 · c51fe39 · c51fe39
1 parent e5c98cb
commit c51fe39
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/src/Util.ts b/src/Util.ts
@@ -156,7 +156,7 @@ export default class W3Util {
     const currentBlockNumber: number = await this.getBlockNumber();
     const firstBlock: IBlock = await this.getBlock(currentBlockNumber - numLookbackBlocks);
 
-    for (let i = currentBlockNumber - numLookbackBlocks; i < currentBlockNumber; i++) {
+    for (let i = firstBlock.number; i < currentBlockNumber; i++) {
       blockPromises.push(this.getBlock(i));
     }
 

diff --git a/test/unit/UnitTestUtil.ts b/test/unit/UnitTestUtil.ts
@@ -7,6 +7,14 @@ describe('Util Unit Tests', async () => {
   const config: Config = await mockConfig();
   const util: W3Util = new W3Util(config.web3);
 
+  describe('getAverageBlockTime()', () => {
+    it('returns the average blocktime of last 100 blocks', async () => {
+      const avgBlockTime = await util.getAverageBlockTime();
+      assert.isNumber(avgBlockTime);
+      assert.isAbove(avgBlockTime, 0);
+    });
+  });
+
   describe('estimateGas()', () => {
     it('returns a number', async () => {
       const gas = await util.estimateGas({});
@@ -65,12 +73,4 @@ describe('Util Unit Tests', async () => {
       assert.isTrue(watching);
     });
   });
-
-  describe('getAverageBlockTime()', () => {
-    it('returns the average blocktime of last 100 blocks', async () => {
-      const avgBlockTime = await util.getAverageBlockTime();
-      assert.isNumber(avgBlockTime);
-      assert.isAbove(avgBlockTime, 0);
-    });
-  });
 });