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

Robot fix xpath to dialog button #4418

Merged
merged 6 commits into from
Feb 22, 2016

Conversation

will-moore
Copy link
Member

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.

This seems to have been changed by some jQuery UI issue, which
is not yet understood since we have not updated jQuery UI
@will-moore
Copy link
Member Author

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...

@will-moore will-moore closed this Jan 21, 2016
@will-moore will-moore reopened this Jan 21, 2016
@will-moore will-moore closed this Jan 22, 2016
@will-moore will-moore reopened this Jan 22, 2016
@will-moore
Copy link
Member Author

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?

@will-moore
Copy link
Member Author

Ooops - my bad (above). Robot tests are running again, but failing with E.g.

ValueError: Element locator 'xpath=//div[contains(@class,'ui-dialog')][contains(@style,'display: block')]//button/span[contains(text(), 'OK')]' did not match any elements.

which suggests that this PR is not included. @sbesson Any ideas what's up here?

@sbesson
Copy link
Member

sbesson commented Feb 2, 2016

@will-moore: looking at the job console log it looks like the job is failing when populating the data for the tests:

05:52:03 + python lib/python/omero/util/populate_metadata.py -k 02f36117-55e8-4bb8-99de-748c6ec0cdee Plate:261 bulk_annotation.csv
05:52:03 Traceback (most recent call last):
05:52:03   File "lib/python/omero/util/populate_metadata.py", line 33, in <module>
05:52:03     import omero.clients
05:52:03 ImportError: No module named omero.clients

@will-moore
Copy link
Member Author

@sbesson Sorry - yes, I just found that.
I was confused because the page at https://ci.openmicroscopy.org/view/Failing/job/OMERO-DEV-merge-robotframework/ links to Robot Results from a few days ago.

The problem is caused by #4446.
I previously had a line to set the $PYTHONPATH but removed it because it was failing
will-moore@562e88f
and now it's failing to import.
Any idea how I can get dist/lib/python onto the PYTHONPATH?

@will-moore
Copy link
Member Author

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.

@will-moore
Copy link
Member Author

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 span element.

@will-moore
Copy link
Member Author

Only 1 failure now, so things are looking good: https://ci.openmicroscopy.org/view/Failing/job/OMERO-DEV-merge-robotframework/221/robot/

@atarkowska
Copy link
Member

@atarkowska
Copy link
Member

Looks good to me. Test are passing two days in a raw. Good to merge

@jburel
Copy link
Member

jburel commented Feb 16, 2016

Waiting an extra day, but certainly looks much better

@jburel
Copy link
Member

jburel commented Feb 22, 2016

jburel added a commit that referenced this pull request Feb 22, 2016
@jburel jburel merged commit 8f24bb2 into ome:develop Feb 22, 2016
@jburel jburel added this to the 5.2.2 milestone Feb 29, 2016
@will-moore will-moore deleted the robot_fix_dialog_button branch February 18, 2019 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants