Skip to content

Commit 37d7466

Browse files
committed
msglist test [nfc]: Shorten "controller" names in some tests
These tests are about scrolling, and there's no other sort of controller in sight. So just `controller` is a good name for the scroll controller.
1 parent 9cc8105 commit 37d7466

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

test/widgets/message_list_test.dart

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -462,14 +462,14 @@ void main() {
462462

463463
testWidgets('scrolling changes visibility', (tester) async {
464464
await setupMessageListPage(tester, messageCount: 10);
465-
final scrollController = findMessageListScrollController(tester)!;
465+
final controller = findMessageListScrollController(tester)!;
466466
check(isButtonVisible(tester)).equals(false);
467467

468-
scrollController.jumpTo(-600);
468+
controller.jumpTo(-600);
469469
await tester.pump();
470470
check(isButtonVisible(tester)).equals(true);
471471

472-
scrollController.jumpTo(0);
472+
controller.jumpTo(0);
473473
await tester.pump();
474474
check(isButtonVisible(tester)).equals(false);
475475
});
@@ -478,8 +478,8 @@ void main() {
478478
await setupMessageListPage(tester, messageCount: 100);
479479

480480
// Scroll up, to hide the button.
481-
final scrollController = findMessageListScrollController(tester)!;
482-
scrollController.jumpTo(-600);
481+
final controller = findMessageListScrollController(tester)!;
482+
controller.jumpTo(-600);
483483
await tester.pump();
484484
check(isButtonVisible(tester)).equals(true);
485485

@@ -497,16 +497,16 @@ void main() {
497497

498498
testWidgets('button works', (tester) async {
499499
await setupMessageListPage(tester, messageCount: 10);
500-
final scrollController = findMessageListScrollController(tester)!;
501-
scrollController.jumpTo(-600);
500+
final controller = findMessageListScrollController(tester)!;
501+
controller.jumpTo(-600);
502502
await tester.pump();
503-
check(scrollController.position.pixels).equals(-600);
503+
check(controller.position.pixels).equals(-600);
504504

505505
// Tap button.
506506
await tester.tap(find.byType(ScrollToBottomButton));
507507
// The list scrolls to the end…
508508
await tester.pumpAndSettle();
509-
check(scrollController.position.pixels).equals(0);
509+
check(controller.position.pixels).equals(0);
510510
// … and for good measure confirm the button disappeared.
511511
check(isButtonVisible(tester)).equals(false);
512512
});

0 commit comments

Comments
 (0)