Merge pull request #709 from lightninglabs/syncing-complete-notify

Display syncing complete notification

Author: tanx

src/action/info.js

@@ -36,8 +36,12 @@ class InfoAction {
           msg: `Syncing to chain (block: ${response.block_height})`,
           wait: true,
         });
-        log.info(`Syncing to chain ... block height: ${response.block_height}`);
         this._store.percentSynced = this.calcPercentSynced(response);
+      } else {
+        this._notification.display({
+          type: 'success',
+          msg: 'Syncing complete',
+        });
       }
       return response.synced_to_chain;
     } catch (err) {

test/unit/action/info.spec.js

@@ -55,13 +55,16 @@ describe('Action Info Unit Tests', () => {
       });
     });
 
-    it('should not show notification if synced', async () => {
+    it('should show completed notification if synced', async () => {
       grpc.sendCommand.withArgs('getInfo').resolves({
         synced_to_chain: true,
-        block_height: 1234,
       });
       await info.getInfo();
-      expect(notification.display, 'was not called');
+      expect(notification.display, 'was called once');
+      expect(notification.display, 'was called with', {
+        type: 'success',
+        msg: 'Syncing complete',
+      });
     });
 
     it('should return true if chain is synced', async () => {