Merge pull request #71 from openboxes/OBPIH-7536

OBPIH-7536 Putaway - split line

Author: alannadolny

.env.example

@@ -15,6 +15,7 @@ LOCATION_SUPPLIER_ALT=locationId
 LOCATION_DEPOT=locationId
 LOCATION_NO_PICK_AND_PUTAWAY_STOCK_DEPOT=locationId
 LOCATION_INTERNAL=locationId
+LOCATION_INTERNAL_TWO=locationId
 LOCATION_WARD=locationId
 PRODUCT_ONE=productId
 PRODUCT_TWO=productId

.github/workflows/playwright.yml

@@ -68,6 +68,7 @@ jobs:
         LOCATION_WARD: ${{ secrets.LOCATION_WARD }}
         LOCATION_NO_PICK_AND_PUTAWAY_STOCK_DEPOT: ${{ secrets.LOCATION_NO_PICK_AND_PUTAWAY_STOCK_DEPOT }}
         LOCATION_INTERNAL: ${{ secrets.LOCATION_INTERNAL }}
+        LOCATION_INTERNAL_TWO: ${{ secrets.LOCATION_INTERNAL_TWO }}
         PRODUCT_ONE: ${{ secrets.PRODUCT_ONE }}
         PRODUCT_TWO: ${{ secrets.PRODUCT_TWO }}
         PRODUCT_THREE: ${{ secrets.PRODUCT_THREE }}

src/config/AppConfig.ts

@@ -26,6 +26,7 @@ export enum LOCATION_KEY {
   WARD = 'ward',
   NO_PICK_AND_PUTAWAY_STOCK = 'noPickAndPutawayStockDepot',
   BIN_LOCATION = 'internalLocation',
+  BIN_LOCATION2 = 'internalLocation2',
 }
 
 export enum PRODUCT_KEY {
@@ -270,6 +271,19 @@ class AppConfig {
         type: LocationTypeCode.BIN_LOCATION,
         parentLocation: env.get('LOCATION_MAIN').required().asString(),
       }),
+
+      internalLocation2: new LocationConfig({
+        id: env.get('LOCATION_INTERNAL_TWO').asString(),
+        key: LOCATION_KEY.BIN_LOCATION,
+        name: this.uniqueIdentifier.generateUniqueString('bin-location2'),
+        requiredActivityCodes: new Set([
+          ActivityCode.PICK_STOCK,
+          ActivityCode.PUTAWAY_STOCK,
+        ]),
+        required: false,
+        type: LocationTypeCode.BIN_LOCATION,
+        parentLocation: env.get('LOCATION_MAIN').required().asString(),
+      }),
     };
 
     this.products = {

src/fixtures/fixtures.ts

@@ -92,6 +92,8 @@ type Fixtures = {
   wardLocationService: LocationData;
   noPickAndPutawayStockDepotService: LocationData;
   internalLocationService: LocationData;
+  internalLocation2Service: LocationData;
+
   // PRODUCT DATA
   mainProductService: ProductData;
   otherProductService: ProductData;
@@ -179,6 +181,8 @@ export const test = baseTest.extend<Fixtures>({
     use(new LocationData(LOCATION_KEY.NO_PICK_AND_PUTAWAY_STOCK, page.request)),
   internalLocationService: async ({ page }, use) =>
     use(new LocationData(LOCATION_KEY.BIN_LOCATION, page.request)),
+  internalLocation2Service: async ({ page }, use) =>
+    use(new LocationData(LOCATION_KEY.BIN_LOCATION2, page.request)),
   // PRODUCTS
   mainProductService: async ({ page }, use) =>
     use(new ProductData(PRODUCT_KEY.ONE, page.request)),

src/pages/putaway/components/CompletePutawayTable.ts

@@ -12,7 +12,7 @@ class CompletePutawayTable extends BasePageModel {
   }
 
   get rows() {
-    return this.table.getByRole('rowgroup');
+    return this.table.getByRole('row');
   }
 
   row(index: number) {
@@ -27,6 +27,10 @@ class Row extends BasePageModel {
     super(page);
     this.row = row;
   }
+
+  getputawayBin(rowIndex: number) {
+    return this.row.getByTestId(`cell-${rowIndex}-undefined`).nth(9);
+  }
 }
 
 export default CompletePutawayTable;

src/pages/putaway/components/SplitModalTable.ts

@@ -47,6 +47,14 @@ class Row extends BasePageModel {
   get quantityField() {
     return this.row.getByRole('cell').getByTestId('quantity-input');
   }
+
+  get clearBinSelect() {
+    return this.row.locator('.react-select__clear-indicator');
+  }
+
+  get putawayBinField() {
+    return this.row.getByTestId('bin-select').getByRole('textbox');
+  }
 }
 
 export default SplitModalTable;

src/pages/putaway/components/StartPutawayTable.ts

@@ -62,6 +62,10 @@ class Row extends BasePageModel {
   get quantityField() {
     return this.row.getByTestId('cell-0-quantity').getByRole('spinbutton');
   }
+
+  get splitLineinPutawayBin() {
+    return this.row.getByTestId('cell-0-putawayBin');
+  }
 }
 
 export default StartPutawayTable;