fix(messaging, ios): native.getInitialNotification can be undefined (#5926)

chuganzy · web-flow · commit f0318d2465f2 · 2021-12-14T09:33:01.000-05:00
diff --git a/packages/messaging/e2e/messaging.e2e.js b/packages/messaging/e2e/messaging.e2e.js
@@ -142,7 +142,7 @@ describe('messaging()', function () {
 
   describe('getInitialNotification', function () {
     it('returns null when no initial notification', async function () {
-      should.equal(await firebase.messaging().getInitialNotification(), null);
+      should.strictEqual(await firebase.messaging().getInitialNotification(), null);
     });
   });
 
diff --git a/packages/messaging/lib/index.js b/packages/messaging/lib/index.js
@@ -122,7 +122,12 @@ class FirebaseMessagingModule extends FirebaseModule {
   }
 
   getInitialNotification() {
-    return this.native.getInitialNotification();
+    return this.native.getInitialNotification().then(value => {
+      if (value) {
+        return value;
+      }
+      return null;
+    });
   }
 
   getIsHeadless() {

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,12 @@ class FirebaseMessagingModule extends FirebaseModule {`
`122`	`122`	`}`
`123`	`123`
`124`	`124`	`getInitialNotification() {`
`125`		`- return this.native.getInitialNotification();`
	`125`	`+ return this.native.getInitialNotification().then(value => {`
	`126`	`+ if (value) {`
	`127`	`+ return value;`
	`128`	`+ }`
	`129`	`+ return null;`
	`130`	`+ });`
`126`	`131`	`}`
`127`	`132`
`128`	`133`	`getIsHeadless() {`