Skip to content

Commit a803a7b

Browse files
authored
test: remove wait-for-expect dependency and refactor tests (#111)
1 parent 4be4bea commit a803a7b

3 files changed

Lines changed: 7 additions & 16 deletions

File tree

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,6 @@
8888
"sockjs-client": "^1.6.1",
8989
"style-loader": "^4.0.0",
9090
"typescript": "^5.9.3",
91-
"wait-for-expect": "^3.0.2",
9291
"webpack": "^5.105.3",
9392
"webpack-dev-middleware": "^7.4.5",
9493
"webpack-merge": "^6.0.1"

pnpm-lock.yaml

Lines changed: 0 additions & 8 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tests/e2e/overlay.test.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ const path = require('node:path');
22
const fs = require('node:fs');
33
const { rspack } = require('@rspack/core');
44
const { RspackDevServer: Server } = require('@rspack/dev-server');
5-
const waitForExpect = require('wait-for-expect');
65
const config = require('../fixtures/overlay-config/webpack.config');
76
const trustedTypesConfig = require('../fixtures/overlay-config/trusted-types.webpack.config');
87
const runBrowser = require('../helpers/run-browser');
@@ -386,9 +385,12 @@ describe('overlay', () => {
386385
hidden: true,
387386
});
388387

389-
await waitForExpect(async () => {
390-
pageHtml = await page.evaluate(() => document.body.outerHTML);
391-
});
388+
await expect
389+
.poll(async () => {
390+
pageHtml = await page.evaluate(() => document.body.outerHTML);
391+
return pageHtml;
392+
})
393+
.toBeTypeOf('string');
392394
overlayHandle = await page.$('#rspack-dev-server-client-overlay');
393395

394396
expect(overlayHandle).toBe(null);
@@ -636,9 +638,7 @@ describe('overlay', () => {
636638

637639
await errorHandle.click();
638640

639-
await waitForExpect(() => {
640-
expect(mockLaunchEditorCb).toHaveBeenCalledTimes(1);
641-
});
641+
await expect.poll(() => mockLaunchEditorCb.mock.calls.length).toBe(1);
642642
} finally {
643643
fs.writeFileSync(pathToFile, originalCode);
644644
await browser.close();

0 commit comments

Comments
 (0)