Merge pull request #65 from openboxes/OBPIH-7533

OBPIH-7533: add create putaway test for shipment with more than 1 item

Author: alannadolny

src/components/NewAlertPopup.ts

@@ -0,0 +1,23 @@
+import { Page } from '@playwright/test';
+
+import BasePageModel from '@/pages/BasePageModel';
+
+class NewAlertPopup extends BasePageModel {
+  constructor(page: Page) {
+    super(page);
+  }
+
+  get tableDialog() {
+    return this.page.getByTestId('modal-with-table');
+  }
+
+  get yesButton() {
+    return this.tableDialog.getByRole('button', { name: 'Yes' });
+  }
+
+  get noButton() {
+    return this.tableDialog.getByRole('button', { name: 'No' });
+  }
+}
+
+export default NewAlertPopup;

src/pages/putaway/CreatePutawayPage.ts

@@ -30,9 +30,32 @@ class CreatePutawayPage extends BasePageModel {
     ).toBeVisible();
   }
 
+  async goToPage() {
+    await this.page.goto('./putAway/create');
+  }
+
   get startPutawayButton() {
     return this.page.getByTestId('start-putaway').nth(0);
   }
+
+  get showByStockMovementFilter() {
+    return this.page.getByTestId('show-by-button');
+  }
+
+  get linesInPendingPutawayFilter() {
+    return this.page
+      .locator('#select-id_1 div')
+      .filter({ hasText: 'Exclude' })
+      .nth(1);
+  }
+
+  get includeLinesInPendingPutawayFilter() {
+    return this.page.getByRole('listitem').filter({ hasText: 'Include' });
+  }
+
+  get emptyCreatePageInformation() {
+    return this.page.locator('.rt-noData').getByText('No rows found');
+  }
 }
 
 export default CreatePutawayPage;

src/pages/putaway/components/CreatePutawayTable.ts

@@ -31,6 +31,16 @@ class Row extends BasePageModel {
   get checkbox() {
     return this.row.getByRole('checkbox');
   }
+
+  getExpandBinLocation(binLocation: string) {
+    return this.row
+      .getByTestId('cell-undefined-undefined')
+      .getByText(binLocation);
+  }
+
+  getProductName(name: string) {
+    return this.row.getByTestId('table-cell').getByText(name);
+  }
 }
 
 export default CreatePutawayTable;

src/pages/putaway/components/StartPutawayTable.ts

@@ -54,6 +54,10 @@ class Row extends BasePageModel {
       .getByRole('listitem')
       .getByText(putawayBin, { exact: true });
   }
+
+  getCurrentBin(currentBin: string) {
+    return this.row.getByTestId('cell-0-currentBin').getByText(currentBin);
+  }
 }
 
 export default StartPutawayTable;

src/pages/receiving/steps/ReceivingStep.ts

@@ -1,7 +1,7 @@
 import { expect, Page } from '@playwright/test';
 
-import AlertPopup from '@/components/AlertPopup';
 import FileHandler from '@/components/FileHandler';
+import NewAlertPopup from '@/components/NewAlertPopup';
 import BasePageModel from '@/pages/BasePageModel';
 import EditModal from '@/pages/receiving/components/EditModal';
 import ReceivingTable from '@/pages/receiving/components/ReceivingTable';
@@ -11,14 +11,14 @@ class ReceivingStep extends BasePageModel {
 
   editModal: EditModal;
 
-  updateExpiryDatePopup: AlertPopup;
+  updateExpiryDatePopup: NewAlertPopup;
   fileHandler: FileHandler;
 
   constructor(page: Page) {
     super(page);
     this.table = new ReceivingTable(page);
     this.editModal = new EditModal(page);
-    this.updateExpiryDatePopup = new AlertPopup(page, 'Yes', 'No');
+    this.updateExpiryDatePopup = new NewAlertPopup(page);
     this.fileHandler = new FileHandler(page);
   }
 

src/pages/transactions/TransactionListPage.ts

@@ -1,4 +1,4 @@
-import { Page } from '@playwright/test';
+import { expect, Page } from '@playwright/test';
 
 import BasePageModel from '@/pages/BasePageModel';
 
@@ -14,6 +14,12 @@ class TransactionListPage extends BasePageModel {
   get successMessage() {
     return this.page.locator('.message');
   }
+
+  async deleteTransaction(n: number) {
+    await this.table.row(n).actionsButton.click();
+    await this.table.deleteButton.click();
+    await expect(this.page.locator('.message')).toBeVisible();
+  }
 }
 
 export default TransactionListPage;