[Jenkins] - Fix e2e diagnostic tabbable uncaught exeption#17076
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the generic Diagnostics page e2e spec to be more stable in CI by integrating Qase IDs, tolerating known focus-trap-related uncaught exceptions during download modal close, and cleaning up downloaded artifacts between tests.
Changes:
- Wraps the diagnostics download test with
qase(1454, ...)for Qase reporting. - Extends
uncaught:exceptionfiltering to ignore additional “tabbable” errors associated with the focus-trap modal behavior. - Adds an
afterEachcleanup step to remove the downloaded diagnostics JSON.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@IsaSih please apply changes based on copilot's suggestions |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Isabela Guimaraes <isabela.guimaraes@suse.com>
Signed-off-by: Isabela Guimaraes <isabela.guimaraes@suse.com>
Signed-off-by: Isabela Guimaraes <isabela.guimaraes@suse.com>
Signed-off-by: Isabela Guimaraes <isabela.guimaraes@suse.com>
Signed-off-by: Isabela Guimaraes <isabela.guimaraes@suse.com>
Signed-off-by: Isabela Guimaraes <isabela.guimaraes@suse.com>
Summary
Fixes #
(rancher/qa-tasks#2270)
Occurred changes and/or fixed issues
Added tabbable exception to cypress
uncaught:exceptionand added an aftereach hook to cleanup the test environment by deleting the downloaded file. Also, added the qase test ID.Areas or cases that should be tested
the
diagnostics.spec.tsAreas which could experience regressions
Diagnostics page
Screenshot/Video
Checklist
Admin,Standard UserandUser Base