Use setTimeout in polling exchange rate and test poll in test

Author: tanx

src/action/wallet.js

@@ -5,7 +5,7 @@
 
 import { observe } from 'mobx';
 import { toBuffer, parseSat, checkHttpStatus, nap } from '../helper';
-import { MIN_PASSWORD_LENGTH, NOTIFICATION_DELAY } from '../config';
+import { MIN_PASSWORD_LENGTH, NOTIFICATION_DELAY, RATE_DELAY } from '../config';
 import * as log from './log';
 
 class WalletAction {
@@ -285,10 +285,9 @@ class WalletAction {
    * @return {Promise<undefined>}
    */
   async pollExchangeRate() {
-    // Poll every 15 minutes, starting now.
     await this.getExchangeRate();
-    let interval = 15 * 60 * 1000;
-    setInterval(() => this.getExchangeRate(), interval);
+    clearTimeout(this.tPollRate);
+    this.tPollRate = setTimeout(() => this.pollExchangeRate(), RATE_DELAY);
   }
 
   /**

src/config.js

@@ -1,6 +1,7 @@
 module.exports.RETRY_DELAY = 3000;
 module.exports.LND_INIT_DELAY = 5000;
 module.exports.NOTIFICATION_DELAY = 5000;
+module.exports.RATE_DELAY = 15 * 60 * 1000;
 
 module.exports.LND_PORT = 10009;
 module.exports.LND_PEER_PORT = 10019;

test/unit/action/wallet.spec.js

@@ -5,6 +5,7 @@ import WalletAction from '../../../src/action/wallet';
 import NavAction from '../../../src/action/nav';
 import NotificationAction from '../../../src/action/notification';
 import * as logger from '../../../src/action/log';
+import { nap } from '../../../src/helper';
 import nock from 'nock';
 import 'isomorphic-fetch';
 
@@ -23,6 +24,7 @@ describe('Action Wallet Unit Tests', () => {
     store = new Store();
     require('../../../src/config').RETRY_DELAY = 1;
     require('../../../src/config').NOTIFICATION_DELAY = 1;
+    require('../../../src/config').RATE_DELAY = 1;
     grpc = sinon.createStubInstance(GrpcAction);
     db = sinon.createStubInstance(AppStorage);
     notification = sinon.createStubInstance(NotificationAction);
@@ -31,6 +33,7 @@ describe('Action Wallet Unit Tests', () => {
   });
 
   afterEach(() => {
+    clearTimeout(wallet.tPollRate);
     sandbox.restore();
   });
 
@@ -328,7 +331,8 @@ describe('Action Wallet Unit Tests', () => {
     it('should call getExchangeRate', async () => {
       sandbox.stub(wallet, 'getExchangeRate');
       await wallet.pollExchangeRate();
-      expect(wallet.getExchangeRate, 'was called once');
+      await nap(30);
+      expect(wallet.getExchangeRate.callCount, 'to be greater than', 1);
     });
   });