Skip to content

Commit b64d43c

Browse files
committed
improvements after review
1 parent 368b723 commit b64d43c

1 file changed

Lines changed: 13 additions & 8 deletions

File tree

src/tests/putaway/performPutawayAsManagerUser.test.ts

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import CreatePutawayPage from '@/pages/putaway/CreatePutawayPage';
66
import PutawayDetailsPage from '@/pages/putaway/putawayDetails/PutawayDetailsPage';
77
import StockMovementShowPage from '@/pages/stockMovementShow/StockMovementShowPage';
88
import { StockMovementResponse } from '@/types';
9+
import RefreshCachesUtils from '@/utils/RefreshCaches';
910
import { getShipmentId, getShipmentItemId } from '@/utils/shipmentUtils';
1011

1112
test.describe('Perform putaway as manager user', () => {
@@ -92,6 +93,7 @@ test.describe('Perform putaway as manager user', () => {
9293
managerUserContext,
9394
internalLocationService,
9495
productService,
96+
page,
9597
}) => {
9698
const receivingBin =
9799
AppConfig.instance.receivingBinPrefix + STOCK_MOVEMENT.identifier;
@@ -102,16 +104,18 @@ test.describe('Perform putaway as manager user', () => {
102104
const internalLocation = await internalLocationService.getLocation();
103105

104106
const managerUserPage = await managerUserContext.newPage();
105-
const stockMovementShowPage = new StockMovementShowPage(managerUserPage);
106107
const navbar = new Navbar(managerUserPage);
108+
const stockMovementShowPage = new StockMovementShowPage(managerUserPage);
107109
const createPutawayPage = new CreatePutawayPage(managerUserPage);
108110
const putawayDetailsPage = new PutawayDetailsPage(managerUserPage);
109111

110112
await test.step('Go to create putaway page', async () => {
111113
await stockMovementShowPage.goToPage(STOCK_MOVEMENT.id);
112114
await stockMovementShowPage.isLoaded();
113-
await navbar.profileButton.click();
114-
await navbar.refreshCachesButton.click();
115+
await RefreshCachesUtils.refreshCaches({
116+
navbar,
117+
page,
118+
});
115119
await navbar.inbound.click();
116120
await navbar.createPutaway.click();
117121
await createPutawayPage.isLoaded();
@@ -158,25 +162,26 @@ test.describe('Perform putaway as manager user', () => {
158162
await createPutawayPage.startStep.nextButton.click();
159163
await createPutawayPage.completeStep.isLoaded();
160164
await expect(
161-
createPutawayPage.completeStep.table.row(2).qtyField
165+
createPutawayPage.completeStep.table.row(2).quantity
162166
).toContainText('5');
163167
await expect(
164-
createPutawayPage.completeStep.table.row(3).qtyField
168+
createPutawayPage.completeStep.table.row(3).quantity
165169
).toContainText('5');
170+
await expect(createPutawayPage.completeStep.table.rows).toHaveCount(4);
166171
});
167172

168173
await test.step('Go backward and use delete button as manager user', async () => {
169174
await createPutawayPage.completeStep.editButton.click();
170175
await createPutawayPage.startStep.isLoaded();
176+
await expect(createPutawayPage.startStep.table.rows).toHaveCount(3);
171177
await createPutawayPage.startStep.table.row(2).deleteButton.click();
172-
await createPutawayPage.startStep.table.row(2).row.isHidden();
178+
await expect(createPutawayPage.startStep.table.rows).toHaveCount(2);
173179
});
174180

175181
await test.step('Go to next page and assert displayed rows', async () => {
176182
await createPutawayPage.startStep.nextButton.click();
177183
await createPutawayPage.completeStep.isLoaded();
178-
await createPutawayPage.completeStep.table.row(2).row.isVisible();
179-
await createPutawayPage.completeStep.table.row(2).row.isHidden();
184+
await expect(createPutawayPage.completeStep.table.rows).toHaveCount(3);
180185
});
181186

182187
await test.step('Complete putaway', async () => {

0 commit comments

Comments
 (0)