Closed
Bug 1413821
Opened 5 years ago
Closed 3 years ago
Permafail test_click.py TestClick.test_inclusive_descendant, TestLegacyClick.test_inclusive_descendant | AssertionError: -1 == -1
Categories
(Testing :: Marionette, defect, P5)
Tracking
(firefox57 disabled, firefox58 disabled)
RESOLVED
WONTFIX
mozilla58
People
(Reporter: intermittent-bug-filer, Unassigned)
References
Details
(Keywords: intermittent-failure, mobile)
Attachments
(1 file, 1 obsolete file)
1.60 KB,
patch
|
whimboo
:
review+
|
Details | Diff | Splinter Review |
Filed by: hskupin [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=141450131&repo=autoland https://queue.taskcluster.net/v1/task/ShuG2OcQRya5yWTrUrb4cw/runs/0/artifacts/public/logs/live_backing.log
Comment 1•5 years ago
|
||
[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.
Updated•5 years ago
|
OS: Unspecified → Android
Hardware: Unspecified → ARM
Comment 2•5 years ago
|
||
I submitted a try build to: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a51ee8480c628fd1409d08607be939ec18ad3ad5
Comment 3•5 years ago
|
||
Comment 4•5 years ago
|
||
Attachment #8924455 -
Attachment is obsolete: true
Updated•5 years ago
|
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
Updated•5 years ago
|
Attachment #8924461 -
Flags: review?(ato)
Comment 5•5 years ago
|
||
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.
Updated•5 years ago
|
Attachment #8924461 -
Flags: review?(ato) → review+
Updated•5 years ago
|
Keywords: checkin-needed
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/mozilla-central/rev/b5a3b8ef6902 Skip final assertion in test_inclusive_descendant for Android. r=test-fix a=test-fix
Keywords: checkin-needed
![]() |
||
Comment 7•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b5a3b8ef6902
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox58:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Comment 8•5 years ago
|
||
We have to keep this bug open for the real fix. Marking as disabled instead.
Updated•5 years ago
|
Keywords: leave-open
Comment 9•5 years ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/6cf9fe9ff66d (FIREFOX_57b_RELBRANCH) https://hg.mozilla.org/releases/mozilla-release/rev/22f767a4ca1d
status-firefox57:
--- → disabled
Comment hidden (Intermittent Failures Robot) |
Updated•4 years ago
|
Keywords: test-disabled
Comment 11•4 years ago
|
||
Lets try to unskip once Mn jobs run on the faster Android 7.0 x86 platform.
Depends on: 1500509
Comment 12•3 years ago
|
||
We moved away from the ARM platform and this failure no longer occurs on packet.net with Android 7.0 x86_64.
Status: REOPENED → RESOLVED
Closed: 5 years ago → 3 years ago
Keywords: leave-open,
test-disabled
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•