Skip to content

Commit 8adbd78

Browse files
committed
test(messaging, ios): remove state dependency on notification registration
if the app is not reinstalled or has unexpected state at startup, then the notification registration state will be unexpected, flaking a test
1 parent 361a018 commit 8adbd78

1 file changed

Lines changed: 0 additions & 24 deletions

File tree

packages/messaging/e2e/messaging.e2e.js

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -61,15 +61,6 @@ describe('messaging() modular', function () {
6161
this.skip();
6262
}
6363
});
64-
it('defaults to false on ios before registering', async function () {
65-
if (device.getPlatform() === 'ios') {
66-
should.equal(firebase.messaging().isDeviceRegisteredForRemoteMessages, false);
67-
await firebase.messaging().registerDeviceForRemoteMessages();
68-
should.equal(firebase.messaging().isDeviceRegisteredForRemoteMessages, true);
69-
} else {
70-
this.skip();
71-
}
72-
});
7364
});
7465

7566
describe('unregisterDeviceForRemoteMessages', function () {
@@ -437,21 +428,6 @@ describe('messaging() modular', function () {
437428
this.skip();
438429
}
439430
});
440-
it('defaults to false on ios before registering', async function () {
441-
const {
442-
getMessaging,
443-
isDeviceRegisteredForRemoteMessages,
444-
registerDeviceForRemoteMessages,
445-
} = messagingModular;
446-
447-
if (device.getPlatform() === 'ios') {
448-
should.equal(isDeviceRegisteredForRemoteMessages(getMessaging()), false);
449-
await registerDeviceForRemoteMessages(getMessaging());
450-
should.equal(isDeviceRegisteredForRemoteMessages(getMessaging()), true);
451-
} else {
452-
this.skip();
453-
}
454-
});
455431
});
456432

457433
describe('unregisterDeviceForRemoteMessages', function () {

0 commit comments

Comments
 (0)