Skip to content

Commit

Permalink
pr comments
Browse files Browse the repository at this point in the history
  • Loading branch information
oldGreg5 committed Jan 16, 2025
1 parent cbdaa7c commit eb9732d
Show file tree
Hide file tree
Showing 4 changed files with 71 additions and 41 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ import 'package:patrol_finders/patrol_finders.dart';
import '../pageobject/account_dropdown_page.dart';
import '../pageobject/app_bar_page.dart';
import '../pageobject/overall_spaces_page.dart';
import '../utils/constants.dart';

void main() async {
late final GoRouter router;
Expand All @@ -34,7 +35,7 @@ void main() async {
(PatrolTester $) async {
await $.pumpWidgetAndSettle(App(routerConfig: router));
await $(OverallSpacesPage.userShortcutBtn)
.tap(settleTimeout: const Duration(seconds: 10));
.tap(settleTimeout: TimeConstants.long.duration);;
await $(AppBarPage.accountPopupBtn).tap();
await AccountDropdownPage.accountDropdownLooksAsExpected($);
await AccountDropdownPage.accountDropdownContainsSpecificData($);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import 'package:patrol_finders/patrol_finders.dart';
import '../pageobject/app_bar_page.dart';
import '../pageobject/overall_spaces_page.dart';
import '../pageobject/spaces_drawer_page.dart';
import '../utils/constants.dart';
import '../utils/selector_utils.dart';

void main() async {
Expand All @@ -34,7 +35,7 @@ void main() async {
(PatrolTester $) async {
await $.pumpWidgetAndSettle(App(routerConfig: router));
await $(OverallSpacesPage.visitorShortcutBtn)
.tap(settleTimeout: const Duration(seconds: 10));
.tap(settleTimeout: TimeConstants.long.duration);
expect($(AppBarPage.spacesDrawerButton).exists, false);
},
);
Expand All @@ -46,7 +47,7 @@ void main() async {
(PatrolTester $) async {
await $.pumpWidgetAndSettle(App(routerConfig: router));
await $(OverallSpacesPage.guestShortcutBtn)
.tap(settleTimeout: const Duration(seconds: 10));
.tap(settleTimeout: TimeConstants.long.duration);
await $(AppBarPage.spacesDrawerButton).waitUntilVisible().tap();
SpacesDrawerPage.commonElementsLookAsExpected($);

Expand All @@ -66,7 +67,7 @@ void main() async {
(PatrolTester $) async {
await $.pumpWidgetAndSettle(App(routerConfig: router));
await $(OverallSpacesPage.guestShortcutBtn)
.tap(settleTimeout: const Duration(seconds: 10));
.tap(settleTimeout: TimeConstants.long.duration);
await $(AppBarPage.spacesDrawerButton).waitUntilVisible().tap();

// iterate thru spaces by clicking next
Expand All @@ -92,7 +93,7 @@ void main() async {
(PatrolTester $) async {
await $.pumpWidgetAndSettle(App(routerConfig: router));
await $(OverallSpacesPage.userShortcutBtn)
.tap(settleTimeout: const Duration(seconds: 10));
.tap(settleTimeout: TimeConstants.long.duration);
await $(AppBarPage.spacesDrawerButton).waitUntilVisible().tap();
SpacesDrawerPage.commonElementsLookAsExpected($);
for (final space in Space.values) {
Expand All @@ -107,7 +108,7 @@ void main() async {
(PatrolTester $) async {
await $.pumpWidgetAndSettle(App(routerConfig: router));
await $(OverallSpacesPage.guestShortcutBtn)
.tap(settleTimeout: const Duration(seconds: 10));
.tap(settleTimeout: TimeConstants.long.duration);
await $(AppBarPage.spacesDrawerButton).waitUntilVisible().tap();
await $(SpacesDrawerPage.allSpacesBtn).tap();
expect($(OverallSpacesPage.spacesListView), findsOneWidget);
Expand All @@ -119,7 +120,7 @@ void main() async {
(PatrolTester $) async {
await $.pumpWidgetAndSettle(App(routerConfig: router));
await $(OverallSpacesPage.userShortcutBtn)
.tap(settleTimeout: const Duration(seconds: 10));
.tap(settleTimeout: TimeConstants.long.duration);
await $(AppBarPage.spacesDrawerButton).waitUntilVisible().tap();
await $(SpacesDrawerPage.allSpacesBtn).tap();
expect($(OverallSpacesPage.spacesListView), findsOneWidget);
Expand All @@ -138,7 +139,7 @@ void main() async {
};
await $.pumpWidgetAndSettle(App(routerConfig: router));
await $(OverallSpacesPage.userShortcutBtn)
.tap(settleTimeout: const Duration(seconds: 10));
.tap(settleTimeout: TimeConstants.long.duration);
await $(AppBarPage.spacesDrawerButton).waitUntilVisible().tap();
for (final space in Space.values) {
await $(SpacesDrawerPage.chooserItem(space)).tap();
Expand Down
Loading

0 comments on commit eb9732d

Please sign in to comment.