-
Notifications
You must be signed in to change notification settings - Fork 102
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
Robot fix xpath to dialog button #4418
Conversation
This seems to have been changed by some jQuery UI issue, which is not yet understood since we have not updated jQuery UI
Today with this fix, we have only 19 failing tests https://ci.openmicroscopy.org/view/Failing/job/OMERO-DEV-merge-robotframework/203/ instead of 62 yesterday https://ci.openmicroscopy.org/view/Failing/job/OMERO-DEV-merge-robotframework/202/. Will look at the remaining failures... |
https://ci.openmicroscopy.org/view/Failing/job/OMERO-DEV-merge-robotframework/209/ Doesn't have any "robot results". @sbesson any ideas? Just re-run again? |
Ooops - my bad (above). Robot tests are running again, but failing with E.g.
which suggests that this PR is not included. @sbesson Any ideas what's up here? |
@will-moore: looking at the job console log it looks like the job is failing when populating the data for the tests:
|
@sbesson Sorry - yes, I just found that. The problem is caused by #4446. |
We are back to a small number of Robot tests failing: https://ci.openmicroscopy.org/view/Failing/job/OMERO-DEV-merge-robotframework/217/robot/ so I think this is working now. |
The whole reason for the failure of Robot tests on the dialog buttons was webtagging React PR adding in Bootstrap.button(), which created dialog buttons without the extra |
Only 1 failure now, so things are looking good: https://ci.openmicroscopy.org/view/Failing/job/OMERO-DEV-merge-robotframework/221/robot/ |
Looks good to me. Test are passing two days in a raw. Good to merge |
Waiting an extra day, but certainly looks much better |
Robot fix xpath to dialog button
The buttons in the jQuery UI dialog seems to have been changed by some jQuery UI issue, which
causes a lot of tests to fail.
I previously saw this on my old laptop where tests failed (without this PR change) but it has been passing on the CI job until recently.