Attempt to fix functional tests

actualbudget · Dec 5, 2024 · 9016693 · 9016693
1 parent b913d3f
commit 9016693
Show file tree

Hide file tree

Showing 9 changed files with 39 additions and 4 deletions.
diff --git a/...cking--6dbdb-page-header-shows-the-previous-month-s-budget-1-chromium-linux.png b/...cking--6dbdb-page-header-shows-the-previous-month-s-budget-1-chromium-linux.png
diff --git a/...cking--96ebb-page-header-shows-the-previous-month-s-budget-3-chromium-linux.png b/...cking--96ebb-page-header-shows-the-previous-month-s-budget-3-chromium-linux.png
diff --git a/...cking--d7184-page-header-shows-the-previous-month-s-budget-2-chromium-linux.png b/...cking--d7184-page-header-shows-the-previous-month-s-budget-2-chromium-linux.png
diff --git a/packages/desktop-client/e2e/page-models/mobile-account-page.js b/packages/desktop-client/e2e/page-models/mobile-account-page.js
@@ -10,6 +10,8 @@ export class MobileAccountPage {
     this.searchBox = page.getByPlaceholder(/^Search/);
     this.transactionList = page.getByLabel('Transaction list');
     this.transactions = this.transactionList.getByRole('button');
+    this.maybeTransactions =
+      this.queryByLabel('Transaction list').queryByRole('button');
     this.createTransactionButton = page.getByRole('button', {
       name: 'Add Transaction',
     });

diff --git a/packages/desktop-client/e2e/page-models/mobile-navigation.js b/packages/desktop-client/e2e/page-models/mobile-navigation.js
@@ -1,3 +1,4 @@
+import { MobileAccountPage } from './mobile-account-page';
 import { MobileAccountsPage } from './mobile-accounts-page';
 import { MobileBudgetPage } from './mobile-budget-page';
 import { MobileTransactionEntryPage } from './mobile-transaction-entry-page';
@@ -22,6 +23,13 @@ export class MobileNavigation {
     return new MobileAccountsPage(this.page);
   }
 
+  async goToUncategorizedPage() {
+    const link = this.page.getByRole('link', { name: /Uncategorized/ });
+    await link.click();
+
+    return new MobileAccountPage(this.page);
+  }
+
   async goToTransactionEntryPage() {
     const link = this.page.getByRole('link', { name: 'Transaction' });
     await link.click();

diff --git a/...js-snapshots/Reports-loads-net-worth-and-cash-flow-reports-1-chromium-linux.png b/...js-snapshots/Reports-loads-net-worth-and-cash-flow-reports-1-chromium-linux.png
diff --git a/...js-snapshots/Reports-loads-net-worth-and-cash-flow-reports-2-chromium-linux.png b/...js-snapshots/Reports-loads-net-worth-and-cash-flow-reports-2-chromium-linux.png
diff --git a/...js-snapshots/Reports-loads-net-worth-and-cash-flow-reports-3-chromium-linux.png b/...js-snapshots/Reports-loads-net-worth-and-cash-flow-reports-3-chromium-linux.png
diff --git a/packages/desktop-client/e2e/transactions.mobile.test.js b/packages/desktop-client/e2e/transactions.mobile.test.js
@@ -48,11 +48,9 @@ test.describe('Mobile Transactions', () => {
     );
     await expect(page).toMatchThemeScreenshots();
 
-    const accountPage = await transactionEntryPage.createTransaction();
+    const invalidAccountPage = await transactionEntryPage.createTransaction();
 
-    await expect(accountPage.transactions.nth(0)).toHaveText(
-      'KrogerClothing-12.34',
-    );
+    await expect(invalidAccountPage.maybeTransactions).toBeNull();
     await expect(page).toMatchThemeScreenshots();
   });
 
@@ -82,4 +80,31 @@ test.describe('Mobile Transactions', () => {
       'KrogerClothing-12.34',
     );
   });
+
+  test('creates a transaction from `/accounts/uncategorized` page', async () => {
+    const uncategorizedPage = await navigation.goToUncategorizedPage();
+    const transactionEntryPage =
+      await uncategorizedPage.clickCreateTransaction();
+
+    await expect(transactionEntryPage.header).toHaveText('New Transaction');
+    await expect(page).toMatchThemeScreenshots();
+
+    await transactionEntryPage.amountField.fill('12.34');
+    // Click anywhere to cancel active edit.
+    await transactionEntryPage.header.click();
+    await transactionEntryPage.fillField(
+      page.getByTestId('payee-field'),
+      'Kroger',
+    );
+    await transactionEntryPage.fillField(
+      page.getByTestId('category-field'),
+      'Clothing',
+    );
+
+    await transactionEntryPage.createTransaction();
+
+    await expect(uncategorizedPage.transactions.nth(0)).toHaveText(
+      'KrogerClothing-12.34',
+    );
+  });
 });