improve page elements

Author: kkrawczyk123

src/pages/putaway/putawayDetails/components/ItemStatusTable.ts

@@ -8,11 +8,11 @@ class ItemStatusTable extends BasePageModel {
   }
 
   get table() {
-    return this.page.getByRole('table');
+    return this.page.getByTestId('item-status-table');
   }
 
   get rows() {
-    return this.table.getByRole('cell');
+    return this.table.getByRole('row');
   }
 
   row(index: number) {
@@ -32,21 +32,15 @@ class Row extends BasePageModel {
   }
 
   get itemStatus() {
-    return this.page
-      .getByTestId('item-status-table')
-      .getByTestId('order-item-status-code');
+    return this.row.getByTestId('order-item-status-code');
   }
 
   get originBin() {
-    return this.page
-      .getByTestId('item-status-table')
-      .getByTestId('origin-bin-location');
+    return this.row.getByTestId('origin-bin-location');
   }
 
   get destinationBin() {
-    return this.page
-      .getByTestId('item-status-table')
-      .getByTestId('destination-bin-location');
+    return this.row.getByTestId('destination-bin-location');
   }
 }
 

src/tests/putaway/assertPutawayDetailsPage.test.ts

@@ -211,13 +211,13 @@ test.describe('Assert putaway details page', () => {
 
     await test.step('Assert content of items status table', async () => {
       await expect(
-        putawayDetailsPage.itemStatusTable.row(0).itemStatus
+        putawayDetailsPage.itemStatusTable.row(1).itemStatus
       ).toHaveText('PENDING');
       await expect(
-        putawayDetailsPage.itemStatusTable.row(0).originBin
+        putawayDetailsPage.itemStatusTable.row(1).originBin
       ).toHaveText(receivingBin);
       await expect(
-        putawayDetailsPage.itemStatusTable.row(0).destinationBin
+        putawayDetailsPage.itemStatusTable.row(1).destinationBin
       ).toHaveText(internalLocation.name);
     });