-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
tests(ui): add unit testing boilerplate and first tests #12047
Conversation
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.
Gave a first brief look and left some comments, will need to check this more in deep, but of course great work :)
Awesome! Looking forward to merge this after comments above are resolved. |
(chore): add list of components prioritized
There is a weird problem with a faulty version of Cypress AE I published with the wrong name some months ago, and which apparently I'm not able to unpublish @rstoenescu if you have permits, could you remove this version? Anyway, since it was due some weeks ago, I directly released Cypress AE v4 as stable and used that one instead :) I had to rebase on latest changes and adapt it to the externalization of vue and vue-router Everything seems to run pretty smoothly, except a test into QSelect which is failing and the snapshots images which appear to not match anymore, as new ones are created into a different folder now. |
Also fix one failing unit test
What kind of change does this PR introduce?
Does this PR introduce a breaking change?
The PR fulfills these requirements:
dev
branch (orv[X]
branch)fix: #xxx[,#xxx]
, where "xxx" is the issue number)If adding a new feature, the PR's description includes:
Other information: