Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(console): stop running playwright tests during build #3722

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

skyrpex
Copy link
Contributor

@skyrpex skyrpex commented Aug 7, 2023

Renames the @wingconsole/app#test script to @wingconsole/app#test:playwright, so it doesn't run with the root scripts build or test. The problem is that in order to run Playwright tests, users need to run playwright install or playwright install --with-deps first, and may require sudo permissions.

Additionally, moves the playwright tests from tests/ to test/, which is the standard directory for both playwright and our turbo configuration.

Supersedes #3721.

@skyrpex skyrpex requested review from ainvoner, polamoros, eladcon and a team as code owners August 7, 2023 09:52
@mergify
Copy link
Contributor

mergify bot commented Aug 7, 2023

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit ae438f1 into main Aug 7, 2023
14 checks passed
@mergify mergify bot deleted the fix-playwright-tests-in-new-envs branch August 7, 2023 10:15
@monadabot
Copy link
Contributor

Congrats! 🚀 This was released in Wing 0.24.78.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants