test: Cypress e2e setup#2552
Conversation
Thanks for opening this pull request! |
|
I will reformat the title to use the proper commit message syntax. |
Uffizzi Ephemeral Environment
|
|
Could you add the UI test to the CI by adding something like this to ci.yml? And could you check the failing lint job? |
|
@mtrezza I have resolved the above. Can you please approve the workflow request ? |
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
|
@mtrezza Some cypress tests are failing currently. I'll fix them in sometime. |
| ); | ||
|
|
||
| cy.visit('/') | ||
| cy.wait(3000) |
There was a problem hiding this comment.
Is it really necessary to wait here? Doesn't cypress handle the loading and continue once the page loading finished?
|
Could we move the Cypress tests into a subdirectory And what are the benefits of choosing Cyprus over puppet to you? Is it faster, more popular, more versatile, etc? One criterium we should keep in mind is that we want to make it as easy as possible for contributors to write tests along with their PRs, so we improve the coverage. |
|
@mehulmathur16 would you want to pick this up again? Or maybe @dplewis? Note there is a bounty on this issue. |
Follow this for running e2e tests on local :
npm dashboardfor running the server.npm run cypress:open. This script will open cypress UI, then you need to select e2e testing, after which you can run specific tests.OR
npm run cypress:run.Currently, some tests might fail, as I have made changes to src/ react components, which are not updating while I was testing this out on my local system. The
npm run dashboardcommand seems to serve the same UI always.Closes: #2007