Merge pull request #1044 from lightninglabs/poll-new-address

Poll new address

Author: valentinewallace

src/action/index-mobile.js

@@ -99,7 +99,6 @@ when(() => store.walletUnlocked, () => grpc.initLnd());
 when(
   () => store.lndReady,
   () => {
-    wallet.getNewAddress();
     wallet.pollBalances();
     wallet.pollExchangeRate();
     channel.pollChannels();

src/action/index.js

@@ -92,7 +92,6 @@ when(
   () => store.lndReady && store.autopilotReady,
   () => {
     autopilot.init();
-    wallet.getNewAddress();
     wallet.pollBalances();
     wallet.pollExchangeRate();
     channel.pollChannels();

src/action/wallet.js

@@ -144,7 +144,11 @@ class WalletAction {
    * @return {Promise<undefined>}
    */
   async update() {
-    await Promise.all([this.getBalance(), this.getChannelBalance()]);
+    await Promise.all([
+      this.getBalance(),
+      this.getChannelBalance(),
+      this.getNewAddress(),
+    ]);
   }
 
   /**
@@ -472,12 +476,9 @@ class WalletAction {
    * @return {Promise<undefined>}
    */
   async getNewAddress() {
-    // - `p2wkh`: Pay to witness key hash (`WITNESS_PUBKEY_HASH` = 0)
-    // - `np2wkh`: Pay to nested witness key hash (`NESTED_PUBKEY_HASH` = 1)
-    // - `p2pkh`:  Pay to public key hash (`PUBKEY_HASH` = 2)
     try {
       const { address } = await this._grpc.sendCommand('NewAddress', {
-        type: 1,
+        type: 3, // UNUSED_NESTED_PUBKEY_HASH = 3
       });
       this._store.walletAddress = address;
     } catch (err) {

test/unit/action/wallet.spec.js

@@ -138,7 +138,7 @@ describe('Action Wallet Unit Tests', () => {
     it('should refresh wallet balances', async () => {
       sandbox.stub(wallet, 'pollExchangeRate');
       await wallet.update();
-      expect(grpc.sendCommand, 'was called twice');
+      expect(grpc.sendCommand, 'was called thrice');
       expect(wallet.pollExchangeRate, 'was not called');
     });
   });