Permafail test_click.py TestClick.test_inclusive_descendant, TestLegacyClick.test_inclusive_descendant | AssertionError: -1 == -1

RESOLVED WONTFIX

Status

defect
P5
normal
RESOLVED WONTFIX
2 years ago
4 months ago

People

(Reporter: intermittent-bug-filer, Unassigned)

Tracking

({intermittent-failure, mobile})

Version 3
mozilla58
ARM
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 disabled, firefox58 disabled)

Details

Attachments

(1 attachment, 1 obsolete attachment)

[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
Attachment #8924455 - Attachment is obsolete: true
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+

Comment 6

2 years ago
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
https://hg.mozilla.org/mozilla-central/rev/b5a3b8ef6902
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
We have to keep this bug open for the real fix. Marking as disabled instead.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment hidden (Intermittent Failures Robot)

Lets try to unskip once Mn jobs run on the faster Android 7.0 x86 platform.

Depends on: 1500509
No longer depends on: 1500509

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: 2 years ago4 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.