-
Notifications
You must be signed in to change notification settings - Fork 5
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
EES-5288 Fix ui test failure #5112
Conversation
…-education-statistics into EES-5288
@@ -115,8 +115,12 @@ Add text block with content to Test section two | |||
user adds text block to editable accordion section Test section two ${RELEASE_CONTENT_EDITABLE_ACCORDION} | |||
user adds content to autosaving accordion section text block Test section two 1 | |||
... Test section two text ${RELEASE_CONTENT_EDITABLE_ACCORDION} | |||
user scrolls to accordion section View related dashboard(s) id:data-accordion |
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.
I'll still approve, but I don't understand why this scrolling is necessary before reloading the page?
If we're reloading the page, I wouldn't expect scrolling to an element first to have any effect.
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.
What I observed is sometimes page is getting frozen intermittently in pipelines. There are couple of issues behind the failure - 1 scrolling issue and 2 stale element issue. Anyway, I will try vice versa of both the steps, if it works then I will push the changes.
This PR addresses and fixes the issue with the 'View Related Dashboards' accordion not opening properly