List only 1 'syncing to chain' ntfn

Author: valentinewallace

src/action/notification.js

@@ -9,15 +9,21 @@ class NotificationAction {
 
   display({ type, msg, wait, err, handler, handlerLbl }) {
     if (err) log.error(msg, err);
-    this._store.notifications.push({
-      type: type || (err ? 'error' : 'info'),
-      message: msg,
-      waiting: wait,
-      date: new Date(),
-      handler: handler || (err ? () => this._nav.goCLI() : null),
-      handlerLbl: handlerLbl || (err ? 'Show error logs' : null),
-      display: true,
-    });
+    const ntfnCount = this._store.notifications.length;
+    let prevNtfn = ntfnCount ? this._store.notifications[ntfnCount - 1] : null;
+    if (prevNtfn && prevNtfn.message === msg) {
+      prevNtfn.date = new Date();
+    } else {
+      this._store.notifications.push({
+        type: type || (err ? 'error' : 'info'),
+        message: msg,
+        waiting: wait,
+        date: new Date(),
+        handler: handler || (err ? () => this._nav.goCLI() : null),
+        handlerLbl: handlerLbl || (err ? 'Show error logs' : null),
+        display: true,
+      });
+    }
     clearTimeout(this.tdisplay);
     this.tdisplay = setTimeout(() => this.close(), NOTIFICATION_DELAY);
   }

test/unit/action/notification.spec.js

@@ -84,6 +84,15 @@ describe('Action Notification Unit Tests', () => {
       store.notifications[0].handler();
       expect(nav.goCLI, 'was called once');
     });
+
+    it('avoid redundant notifications but update date', async () => {
+      notification.display({ msg: 'hello' });
+      const date = store.notifications[0].date;
+      await nap(10);
+      notification.display({ msg: 'hello' });
+      expect(store.notifications.length, 'to equal', 1);
+      expect(store.notifications[0].date, 'not to equal', date);
+    });
   });
 
   describe('close()', () => {