[task 2017-11-01T21:01:16.870Z] 21:01:16 WARNING - TEST-UNEXPECTED-FAIL | test_click.py TestClick.test_inclusive_descendant | AssertionError: -1 == -1 [task 2017-11-01T21:01:16.871Z] 21:01:16 INFO - Traceback (most recent call last): [task 2017-11-01T21:01:16.871Z] 21:01:16 INFO - File "/builds/worker/workspace/build/venv/lib/python2.7/site-packages/marionette_harness/marionette_test/testcases.py", line 156, in run [task 2017-11-01T21:01:16.872Z] 21:01:16 INFO - testMethod() [task 2017-11-01T21:01:16.872Z] 21:01:16 INFO - File "/builds/worker/workspace/build/tests/marionette/tests/testing/marionette/harness/marionette_harness/tests/unit/test_click.py", line 143, in test_inclusive_descendant [task 2017-11-01T21:01:16.872Z] 21:01:16 INFO - self.assertNotEqual(select.get_property("selectedIndex"), -1) This is an Android only failure. And as it looks like the issued click does not correctly select any item of the <select> node. I would temporarily disable the assertion for Android, so it only gets run for desktop builds.
OS: Unspecified → Android
Hardware: Unspecified → ARM
I submitted a try build to: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a51ee8480c628fd1409d08607be939ec18ad3ad5
Summary: Intermittent test_click.py TestClick.test_inclusive_descendant, TestLegacyClick.test_inclusive_descendant | AssertionError: -1 == -1 → Permafail test_click.py TestClick.test_inclusive_descendant, TestLegacyClick.test_inclusive_descendant | AssertionError: -1 == -1
The last try job shows this test as passing: > [task 2017-11-02T09:38:12.174Z] 09:38:12 INFO - TEST-START | test_click.py TestClick.test_inclusive_descendant > [task 2017-11-02T09:38:22.570Z] 09:38:22 INFO - TEST-PASS | test_click.py TestClick.test_inclusive_descendant | took 10395ms I think we should skip the review cycle and land it due to merge day.
Attachment #8924461 - Flags: review?(ato) → review+
Pushed by email@example.com: https://hg.mozilla.org/mozilla-central/rev/b5a3b8ef6902 Skip final assertion in test_inclusive_descendant for Android. r=test-fix a=test-fix
We have to keep this bug open for the real fix. Marking as disabled instead.
You need to log in before you can comment on or make changes to this bug.