Skip to content

Commit 40165ef

Browse files
committed
dialog [nfc]: Rename checkNoErrorDialog to checkNoDialog
1 parent d224a5b commit 40165ef

File tree

4 files changed

+16
-14
lines changed

4 files changed

+16
-14
lines changed

test/widgets/action_sheet_test.dart

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -273,7 +273,7 @@ void main() {
273273
await tester.tap(findButtonForLabel('Mark channel as read'));
274274
await tester.pumpAndSettle();
275275
checkRequest(someChannel.streamId);
276-
checkNoErrorDialog(tester);
276+
checkNoDialog(tester);
277277
});
278278

279279
testWidgets('request fails', (tester) async {
@@ -688,7 +688,7 @@ void main() {
688688
await tester.tap(findButtonForLabel('Mark as resolved'));
689689
await tester.pumpAndSettle();
690690

691-
checkNoErrorDialog(tester);
691+
checkNoDialog(tester);
692692
checkRequest(message.id, '✔ zulip');
693693
});
694694

@@ -703,7 +703,7 @@ void main() {
703703
await tester.tap(findButtonForLabel('Mark as resolved'));
704704
await tester.pumpAndSettle();
705705

706-
checkNoErrorDialog(tester);
706+
checkNoDialog(tester);
707707
checkRequest(message.id, '✔ zulip');
708708
});
709709

@@ -717,7 +717,7 @@ void main() {
717717
await tester.tap(findButtonForLabel('Mark as unresolved'));
718718
await tester.pumpAndSettle();
719719

720-
checkNoErrorDialog(tester);
720+
checkNoDialog(tester);
721721
checkRequest(message.id, 'zulip');
722722
});
723723

@@ -731,7 +731,7 @@ void main() {
731731
await tester.tap(findButtonForLabel('Mark as unresolved'));
732732
await tester.pumpAndSettle();
733733

734-
checkNoErrorDialog(tester);
734+
checkNoDialog(tester);
735735
checkRequest(message.id, 'zulip');
736736
});
737737

test/widgets/app_test.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -401,14 +401,14 @@ void main() {
401401
check(ZulipApp.ready).value.isFalse();
402402
await tester.pump();
403403
check(findSnackBarByText(message).evaluate()).isEmpty();
404-
checkNoErrorDialog(tester);
404+
checkNoDialog(tester);
405405

406406
check(ZulipApp.ready).value.isTrue();
407407
// After app startup, reportErrorToUserBriefly displays a SnackBar.
408408
reportErrorToUserBriefly(message, details: details);
409409
await tester.pumpAndSettle();
410410
check(findSnackBarByText(message).evaluate()).single;
411-
checkNoErrorDialog(tester);
411+
checkNoDialog(tester);
412412

413413
// Open the error details dialog.
414414
await tester.tap(find.text('Details'));
@@ -493,7 +493,7 @@ void main() {
493493
reportErrorToUserModally(title, message: message);
494494
check(ZulipApp.ready).value.isFalse();
495495
await tester.pump();
496-
checkNoErrorDialog(tester);
496+
checkNoDialog(tester);
497497

498498
check(ZulipApp.ready).value.isTrue();
499499
// After app startup, reportErrorToUserModally displays an [AlertDialog].

test/widgets/compose_box_test.dart

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ void main() {
276276
await prepareWithContent(tester,
277277
makeStringWithCodePoints(kMaxMessageLengthCodePoints));
278278
await tapSendButton(tester);
279-
checkNoErrorDialog(tester);
279+
checkNoDialog(tester);
280280
});
281281

282282
testWidgets('code points not counted unnecessarily', (tester) async {
@@ -313,7 +313,7 @@ void main() {
313313
await prepareWithTopic(tester,
314314
makeStringWithCodePoints(kMaxTopicLengthCodePoints));
315315
await tapSendButton(tester);
316-
checkNoErrorDialog(tester);
316+
checkNoDialog(tester);
317317
});
318318

319319
testWidgets('code points not counted unnecessarily', (tester) async {
@@ -739,7 +739,7 @@ void main() {
739739
await setupAndTapSend(tester, prepareResponse: (int messageId) {
740740
connection.prepare(json: SendMessageResult(id: messageId).toJson());
741741
});
742-
checkNoErrorDialog(tester);
742+
checkNoDialog(tester);
743743
});
744744

745745
testWidgets('ZulipApiException', (tester) async {
@@ -877,7 +877,7 @@ void main() {
877877
check(call.allowMultiple).equals(true);
878878
check(call.type).equals(FileType.media);
879879

880-
checkNoErrorDialog(tester);
880+
checkNoDialog(tester);
881881

882882
check(controller!.content.text)
883883
.equals('see image: [Uploading image.jpg…]()\n\n');
@@ -936,7 +936,7 @@ void main() {
936936
check(call.source).equals(ImageSource.camera);
937937
check(call.requestFullMetadata).equals(false);
938938

939-
checkNoErrorDialog(tester);
939+
checkNoDialog(tester);
940940

941941
check(controller!.content.text)
942942
.equals('see image: [Uploading image.jpg…]()\n\n');

test/widgets/dialog_checks.dart

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,9 @@ Widget checkErrorDialog(WidgetTester tester, {
3131
}
3232

3333
// TODO(#996) update this to check for per-platform flavors of alert dialog
34-
void checkNoErrorDialog(WidgetTester tester) {
34+
/// Checks that there is no dialog.
35+
/// Fails if one is found.
36+
void checkNoDialog(WidgetTester tester) {
3537
check(find.byType(AlertDialog)).findsNothing();
3638
}
3739

0 commit comments

Comments
 (0)