Skip to content

Commit 361ee16

Browse files
u7088495gnprice
authored andcommitted
dialog [nfc]: Rename checkNoErrorDialog to checkNoDialog
1 parent 98b94bd commit 361ee16

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
@@ -335,7 +335,7 @@ void main() {
335335
await tester.tap(findButtonForLabel('Mark channel as read'));
336336
await tester.pumpAndSettle();
337337
checkRequest(someChannel.streamId);
338-
checkNoErrorDialog(tester);
338+
checkNoDialog(tester);
339339
});
340340

341341
testWidgets('request fails', (tester) async {
@@ -776,7 +776,7 @@ void main() {
776776
await tester.tap(findButtonForLabel('Mark as resolved'));
777777
await tester.pumpAndSettle();
778778

779-
checkNoErrorDialog(tester);
779+
checkNoDialog(tester);
780780
checkRequest(message.id, '✔ zulip');
781781
});
782782

@@ -791,7 +791,7 @@ void main() {
791791
await tester.tap(findButtonForLabel('Mark as resolved'));
792792
await tester.pumpAndSettle();
793793

794-
checkNoErrorDialog(tester);
794+
checkNoDialog(tester);
795795
checkRequest(message.id, '✔ zulip');
796796
});
797797

@@ -805,7 +805,7 @@ void main() {
805805
await tester.tap(findButtonForLabel('Mark as unresolved'));
806806
await tester.pumpAndSettle();
807807

808-
checkNoErrorDialog(tester);
808+
checkNoDialog(tester);
809809
checkRequest(message.id, 'zulip');
810810
});
811811

@@ -819,7 +819,7 @@ void main() {
819819
await tester.tap(findButtonForLabel('Mark as unresolved'));
820820
await tester.pumpAndSettle();
821821

822-
checkNoErrorDialog(tester);
822+
checkNoDialog(tester);
823823
checkRequest(message.id, 'zulip');
824824
});
825825

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');
@@ -1864,7 +1864,7 @@ void main() {
18641864
UploadFileResult(url: '/path/file.jpg').toJson());
18651865
await tester.tap(find.byIcon(ZulipIcons.attach_file), warnIfMissed: false);
18661866
await tester.pump(Duration.zero);
1867-
checkNoErrorDialog(tester);
1867+
checkNoDialog(tester);
18681868
check(testBinding.takePickFilesCalls()).length.equals(1);
18691869
connection.takeRequests(); // upload request
18701870

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)