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

Add Container Cypress test for pathExists and showConfirmationModal #3959

Closed
ndricimrr opened this issue Sep 30, 2024 · 0 comments · Fixed by #3971
Closed

Add Container Cypress test for pathExists and showConfirmationModal #3959

ndricimrr opened this issue Sep 30, 2024 · 0 comments · Fixed by #3971
Assignees
Labels
container headless-core internal Stuff like NFRs, CI pipeline ora ora related issues

Comments

@ndricimrr
Copy link
Contributor

ndricimrr commented Sep 30, 2024

Check if there are any present Cypress tests for this already, and if not, add Cypress test for them.

You may check the implementation to find out how it works:
pathExists and showConfirmationModal .

But it shouldn't be more complicated than having the test click on a button attached to the function call, and then checking the correct alert/log result is shown.

Consider adding it for wc-based luigi container tests and luigi compound tests both if possible.

--

This issue is part of #3757 epic

@ndricimrr ndricimrr added the internal Stuff like NFRs, CI pipeline label Sep 30, 2024
@ndricimrr ndricimrr changed the title Add Cypress test for pathExists and showConfirmationModal Add Container Cypress test for pathExists and showConfirmationModal Sep 30, 2024
@walmazacn walmazacn self-assigned this Oct 1, 2024
@walmazacn walmazacn linked a pull request Oct 2, 2024 that will close this issue
@hardl hardl added ora ora related issues headless-core labels Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
container headless-core internal Stuff like NFRs, CI pipeline ora ora related issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants