Skip to content

Commit 7fdd4a9

Browse files
committed
dialog [nfc]: Rename checkNoErrorDialog to checkNoDialog
1 parent 47f3e6a commit 7fdd4a9

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 {
@@ -675,7 +675,7 @@ void main() {
675675
await tester.tap(findButtonForLabel('Mark as resolved'));
676676
await tester.pumpAndSettle();
677677

678-
checkNoErrorDialog(tester);
678+
checkNoDialog(tester);
679679
checkRequest(message.id, '✔ zulip');
680680
});
681681

@@ -690,7 +690,7 @@ void main() {
690690
await tester.tap(findButtonForLabel('Mark as resolved'));
691691
await tester.pumpAndSettle();
692692

693-
checkNoErrorDialog(tester);
693+
checkNoDialog(tester);
694694
checkRequest(message.id, '✔ zulip');
695695
});
696696

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

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

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

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

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
@@ -273,7 +273,7 @@ void main() {
273273
await prepareWithContent(tester,
274274
makeStringWithCodePoints(kMaxMessageLengthCodePoints));
275275
await tapSendButton(tester);
276-
checkNoErrorDialog(tester);
276+
checkNoDialog(tester);
277277
});
278278

279279
testWidgets('code points not counted unnecessarily', (tester) async {
@@ -310,7 +310,7 @@ void main() {
310310
await prepareWithTopic(tester,
311311
makeStringWithCodePoints(kMaxTopicLengthCodePoints));
312312
await tapSendButton(tester);
313-
checkNoErrorDialog(tester);
313+
checkNoDialog(tester);
314314
});
315315

316316
testWidgets('code points not counted unnecessarily', (tester) async {
@@ -630,7 +630,7 @@ void main() {
630630
await setupAndTapSend(tester, prepareResponse: (int messageId) {
631631
connection.prepare(json: SendMessageResult(id: messageId).toJson());
632632
});
633-
checkNoErrorDialog(tester);
633+
checkNoDialog(tester);
634634
});
635635

636636
testWidgets('ZulipApiException', (tester) async {
@@ -748,7 +748,7 @@ void main() {
748748
check(call.allowMultiple).equals(true);
749749
check(call.type).equals(FileType.media);
750750

751-
checkNoErrorDialog(tester);
751+
checkNoDialog(tester);
752752

753753
check(controller!.content.text)
754754
.equals('see image: [Uploading image.jpg…]()\n\n');
@@ -807,7 +807,7 @@ void main() {
807807
check(call.source).equals(ImageSource.camera);
808808
check(call.requestFullMetadata).equals(false);
809809

810-
checkNoErrorDialog(tester);
810+
checkNoDialog(tester);
811811

812812
check(controller!.content.text)
813813
.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
@@ -29,7 +29,9 @@ Widget checkErrorDialog(WidgetTester tester, {
2929
}
3030

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

0 commit comments

Comments
 (0)