notices - markAllNoticesRead - use async/await

app/scripts/metamask-controller.js

@@ -473,7 +473,7 @@ module.exports = class MetamaskController extends EventEmitter {
       // notices
       checkNotices: noticeController.updateNoticesList.bind(noticeController),
       markNoticeRead: noticeController.markNoticeRead.bind(noticeController),
-      markAllNoticesRead: noticeController.markAllNoticesRead.bind(noticeController),
+      markAllNoticesRead: nodeify(noticeController.markAllNoticesRead, noticeController),
 
       approveProviderRequest: providerApprovalController.approveProviderRequest.bind(providerApprovalController),
       clearApprovedOrigins: providerApprovalController.clearApprovedOrigins.bind(providerApprovalController),

app/scripts/notice-controller.js

@@ -58,20 +58,15 @@ module.exports = class NoticeController extends EventEmitter {
     }
   }
 
-  markAllNoticesRead (cb) {
+  async markAllNoticesRead () {
-    cb = cb || function (err) { if (err) throw err }
+    const noticeList = this.getNoticesList()
-    try {
+    noticeList.forEach(notice => {
-      const noticeList = this.getNoticesList()
+      notice.read = true
-      noticeList.forEach(notice => {
+      notice.body = ''
-        notice.read = true
+    })
-        notice.body = ''
+    this.setNoticesList(noticeList)
-      })
+    const latestNotice = this.getNextUnreadNotice()
-      this.setNoticesList(noticeList)
+    return latestNotice
-      const latestNotice = this.getNextUnreadNotice()
-      cb(null, latestNotice)
-    } catch (err) {
-      cb(err)
-    }
   }
 
 

test/unit/app/controllers/notice-controller-test.js

@@ -40,7 +40,7 @@ describe('notice-controller', function () {
     })
 
     describe('#markAllNoticesRead', () => {
-      it('marks all notices read', (done) => {
+      it('marks all notices read', async () => {
         const testList = [{
           id: 0,
           read: false,
@@ -57,11 +57,10 @@ describe('notice-controller', function () {
 
         noticeController.setNoticesList(testList)
 
-        noticeController.markAllNoticesRead()
+        await noticeController.markAllNoticesRead()
 
         const unreadNotices = noticeController.getUnreadNotices()
         assert.equal(unreadNotices.length, 0)
-        done()
       })
     })
 

test/unit/ui/app/actions.spec.js

@@ -1321,14 +1321,11 @@ describe('Actions', () => {
       completeOnboardingSpy.restore()
     })
 
-    it('', (done) => {
+    it('completing onboarding marks all notices as read', async () => {
       const store = mockStore()
-      store.dispatch(actions.setCompletedOnboarding())
+      await store.dispatch(actions.setCompletedOnboarding())
-        .then(() => {
+      assert.equal(markAllNoticesReadSpy.callCount, 1)
-          assert.equal(markAllNoticesReadSpy.callCount, 1)
+      assert.equal(completeOnboardingSpy.callCount, 1)
-          assert.equal(completeOnboardingSpy.callCount, 1)
-        })
-        done()
     })
   })