Skip to content

Commit acb93ea

Browse files
committed
dialog [nfc]: Rename checkNoErrorDialog to checkNoDialog
1 parent 55d055f commit acb93ea

File tree

4 files changed

+17
-15
lines changed

4 files changed

+17
-15
lines changed

test/widgets/action_sheet_test.dart

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,7 @@ void main() {
328328
await tester.tap(findButtonForLabel('Mark channel as read'));
329329
await tester.pumpAndSettle();
330330
checkRequest(someChannel.streamId);
331-
checkNoErrorDialog(tester);
331+
checkNoDialog(tester);
332332
});
333333

334334
testWidgets('request fails', (tester) async {
@@ -742,7 +742,7 @@ void main() {
742742
await tester.tap(findButtonForLabel('Mark as resolved'));
743743
await tester.pumpAndSettle();
744744

745-
checkNoErrorDialog(tester);
745+
checkNoDialog(tester);
746746
checkRequest(message.id, '✔ zulip');
747747
});
748748

@@ -757,7 +757,7 @@ void main() {
757757
await tester.tap(findButtonForLabel('Mark as resolved'));
758758
await tester.pumpAndSettle();
759759

760-
checkNoErrorDialog(tester);
760+
checkNoDialog(tester);
761761
checkRequest(message.id, '✔ zulip');
762762
});
763763

@@ -771,7 +771,7 @@ void main() {
771771
await tester.tap(findButtonForLabel('Mark as unresolved'));
772772
await tester.pumpAndSettle();
773773

774-
checkNoErrorDialog(tester);
774+
checkNoDialog(tester);
775775
checkRequest(message.id, 'zulip');
776776
});
777777

@@ -785,7 +785,7 @@ void main() {
785785
await tester.tap(findButtonForLabel('Mark as unresolved'));
786786
await tester.pumpAndSettle();
787787

788-
checkNoErrorDialog(tester);
788+
checkNoDialog(tester);
789789
checkRequest(message.id, 'zulip');
790790
});
791791

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: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,7 @@ void main() {
395395
await prepareWithContent(tester,
396396
makeStringWithCodePoints(kMaxMessageLengthCodePoints));
397397
await tapSendButton(tester);
398-
checkNoErrorDialog(tester);
398+
checkNoDialog(tester);
399399
});
400400

401401
testWidgets('code points not counted unnecessarily', (tester) async {
@@ -434,7 +434,7 @@ void main() {
434434
await prepareWithTopic(tester,
435435
makeStringWithCodePoints(kMaxTopicLengthCodePoints));
436436
await tapSendButton(tester);
437-
checkNoErrorDialog(tester);
437+
checkNoDialog(tester);
438438
});
439439

440440
testWidgets('code points not counted unnecessarily', (tester) async {
@@ -938,7 +938,7 @@ void main() {
938938
await setupAndTapSend(tester, prepareResponse: (int messageId) {
939939
connection.prepare(json: SendMessageResult(id: messageId).toJson());
940940
});
941-
checkNoErrorDialog(tester);
941+
checkNoDialog(tester);
942942
});
943943

944944
testWidgets('ZulipApiException', (tester) async {
@@ -1078,7 +1078,7 @@ void main() {
10781078
check(call.allowMultiple).equals(true);
10791079
check(call.type).equals(FileType.media);
10801080

1081-
checkNoErrorDialog(tester);
1081+
checkNoDialog(tester);
10821082

10831083
check(controller!.content.text)
10841084
.equals('see image: [Uploading image.jpg…]()\n\n');
@@ -1137,7 +1137,7 @@ void main() {
11371137
check(call.source).equals(ImageSource.camera);
11381138
check(call.requestFullMetadata).equals(false);
11391139

1140-
checkNoErrorDialog(tester);
1140+
checkNoDialog(tester);
11411141

11421142
check(controller!.content.text)
11431143
.equals('see image: [Uploading image.jpg…]()\n\n');
@@ -1836,7 +1836,7 @@ void main() {
18361836
UploadFileResult(uri: '/path/file.jpg').toJson());
18371837
await tester.tap(find.byIcon(ZulipIcons.attach_file), warnIfMissed: false);
18381838
await tester.pump(Duration.zero);
1839-
checkNoErrorDialog(tester);
1839+
checkNoDialog(tester);
18401840
check(testBinding.takePickFilesCalls()).length.equals(1);
18411841
connection.takeRequests(); // upload request
18421842

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)