-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: Custom Cypress tests #237
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Lighthouse ReportsHere are the Lighthouse reports of this Pull Request📝 Based on commit 1ed261a
|
This reverts commit ed7fd63.
@@ -70,6 +70,7 @@ module.exports = { | |||
experimental: { | |||
nodeVersion: 18, | |||
cypressVersion: 12, | |||
enableCypressExtension: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we going to enable it as default? I think we shouldn't, the user should enable it when needed, WTDY?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, we won't change that, it was just for testing purposes 🤝
## What's the purpose of this pull request? This PR intends to fix some details regarding stores' custom Cypress tests. ### Starters Deploy Preview vtex-sites/starter.store#237 --------- Co-authored-by: Mariana Caetano Pereira <[email protected]>
@lucasfp13 should we merge this PR or #244? |
No, we don't need to merge this PR as it was created for testing purposes only. |
What's the purpose of this pull request?
This PR intends to setup the Cypress test extension.
How to test it?
Faststore related PRs
vtex/faststore#2080