search

storybookjs / test-runner

🚕 Turn stories into executable tests
https://storybook.js.org/docs/writing-tests/interaction-testing
MIT License
233 stars 72 forks source link

[Bug]: Migrate @jest-playwright-preset to @playwright/test to avoid security issue. #510

Open Leah-Xia-Microsoft opened 2 months ago

Leah-Xia-Microsoft commented 2 months ago

Describe the bug

@jest-playwright-preset is not getting maintained anymore. See: https://github.com/playwright-community/jest-playwright?tab=readme-ov-file#%EF%B8%8F-we-recommend-the-official-playwright-test-runner-playwrighttest-%EF%B8%8F And it's using a dependency that's using axios <1.7.4. which is causing a security issue that's fixed in a later version. (https://github.com/jeffbski/wait-on/commit/672fd83c0205d034e6ae68d3f33f70d1b61c2e6a) The suggestion from the @jest-playwright-preset team is to migrate to @playwright/test. Migration should be pretty straightforward, following https://playwright.dev/docs/intro

To Reproduce

No repro step needed.

System

Any system

Additional context

No response

501NotImplemented commented 1 day ago

Also would like to add that completing this ticket would help to resolve the issue when playwright package is not found that occurs after the direct playwright package was deprecated by PW team in favor of using the @playwright/test.