Fix flaky mobile test

actualbudget · Oct 14, 2024 · 557a4f4 · 557a4f4
1 parent a2c7273
commit 557a4f4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 8 deletions.
diff --git a/packages/desktop-client/e2e/budget.test.js b/packages/desktop-client/e2e/budget.test.js
@@ -27,9 +27,7 @@ test.describe('Budget', () => {
   test('renders the summary information: available funds, overspent, budgeted and for next month', async () => {
     const summary = budgetPage.budgetSummary.first();
 
-    await expect(summary.getByText('Available Funds')).toBeVisible({
-      timeout: 10000,
-    });
+    await expect(summary.getByText('Available Funds')).toBeVisible();
     await expect(summary.getByText(/^Overspent in /)).toBeVisible();
     await expect(summary.getByText('Budgeted')).toBeVisible();
     await expect(summary.getByText('For Next Month')).toBeVisible();

diff --git a/packages/desktop-client/e2e/page-models/mobile-navigation.js b/packages/desktop-client/e2e/page-models/mobile-navigation.js
@@ -105,31 +105,37 @@ export class MobileNavigation {
   }
 
   async goToBudgetPage() {
-    return this.navigateToPage('Budget', () => new MobileBudgetPage(this.page));
+    return await this.navigateToPage(
+      'Budget',
+      () => new MobileBudgetPage(this.page),
+    );
   }
 
   async goToAccountsPage() {
-    return this.navigateToPage(
+    return await this.navigateToPage(
       'Accounts',
       () => new MobileAccountsPage(this.page),
     );
   }
 
   async goToTransactionEntryPage() {
-    return this.navigateToPage(
+    return await this.navigateToPage(
       'Transaction',
       () => new MobileTransactionEntryPage(this.page),
     );
   }
 
   async goToReportsPage() {
-    return this.navigateToPage(
+    return await this.navigateToPage(
       'Reports',
       () => new MobileReportsPage(this.page),
     );
   }
 
   async goToSettingsPage() {
-    return this.navigateToPage('Settings', () => new SettingsPage(this.page));
+    return await this.navigateToPage(
+      'Settings',
+      () => new SettingsPage(this.page),
+    );
   }
 }