Skip to content

Commit b80ec14

Browse files
test(messaging, android): reinstate subscription to topics e2e tests (#10452)
* test(messaging, android): reinstate subscription to topics e2e tests * analyse issue
1 parent d409a7b commit b80ec14

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

tests/integration_test/firebase_messaging/firebase_messaging_e2e_test.dart

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -198,10 +198,7 @@ void main() {
198198
await messaging.subscribeToTopic(topic);
199199
},
200200
// macOS skipped because it needs keychain sharing entitlement. See: https://github.com/firebase/flutterfire/issues/9538
201-
// Android skipped because it times out with a likelihood of 98%. See: https://github.com/firebase/flutterfire/issues/9651
202-
skip: kIsWeb ||
203-
defaultTargetPlatform == TargetPlatform.macOS ||
204-
defaultTargetPlatform == TargetPlatform.android,
201+
skip: kIsWeb || defaultTargetPlatform == TargetPlatform.macOS,
205202
);
206203
});
207204

@@ -213,10 +210,7 @@ void main() {
213210
await messaging.unsubscribeFromTopic(topic);
214211
},
215212
// macOS skipped because it needs keychain sharing entitlement. See: https://github.com/firebase/flutterfire/issues/9538
216-
// Android skipped because it times out with a likelihood of 98%. See: https://github.com/firebase/flutterfire/issues/9650
217-
skip: kIsWeb ||
218-
defaultTargetPlatform == TargetPlatform.macOS ||
219-
defaultTargetPlatform == TargetPlatform.android,
213+
skip: kIsWeb || defaultTargetPlatform == TargetPlatform.macOS,
220214
);
221215
});
222216

0 commit comments

Comments
 (0)