Closed Bug 949897 Opened 11 years ago Closed 11 years ago

test_cards_view_kill_apps_with_two_apps.py intermittently fails

Categories

(Firefox OS Graveyard :: Gaia::UI Tests, defect, P1)

ARM
Gonk (Firefox OS)
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: alive, Assigned: zcampbell)

References

Details

(Keywords: intermittent-failure, Whiteboard: [xfail])

Attachments

(1 file, 1 obsolete file)

(deleted), text/x-github-pull-request
askeing
: review+
Details
Should not cause by https://travis-ci.org/mozilla-b2g/gaia/builds/15321988 and the test run well on my local try. No idea.
Component: Gaia::System → Gaia::UI Tests
We're seeing this intermittent a lot. Do you mind if we disable it?
This wasn't an intermittent yesterday and we didn't change the test so it's probably a real bug, if somebody could be bothered to investigate it.
Somebody investigate and re-enable this ASAP.
Priority: -- → P1
I ran this test and I got 3 out of 21 fails: https://pastebin.mozilla.org/3789082 I will investigate the problem
Priority: P1 → --
Priority: -- → P1
Attachment #8348036 - Flags: review?(zcampbell)
Attachment #8348036 - Flags: review?(florin.strugariu)
Attachment #8348036 - Flags: review?(andrei.hutusoru)
I added a wait and I think it fixes the intermittent problem we had. But the thing is that I discovered another fail, before modifying the test. So I'll work on that one immediatelly. This is the fail I'm talking about https://pastebin.mozilla.org/3789554. I'll make a pull request once I fix it. I'll add it to this bug, since it still an intermittent fail.
(In reply to Viorela Ioia [:viorela] from comment #7) > I added a wait and I think it fixes the intermittent problem we had. But the > thing is that I discovered another fail, before modifying the test. So I'll > work on that one immediatelly. This is the fail I'm talking about > https://pastebin.mozilla.org/3789554. > I'll make a pull request once I fix it. I'll add it to this bug, since it > still an intermittent fail. The fail in the pastebin there is the one reported in this bug so I don't know what your patch is for! Alive's original test failure has been over-written by the test run but I recall the exception message from memory (Alive next time copy/paste the exception?)
After investigating this I added a time.sleep(3) before the 'holdhome' Javascript here: https://github.com/mozilla-b2g/gaia/blob/master/tests/python/gaia-ui-tests/gaiatest/apps/system/regions/cards_view.py#L30 After 50+ runs on desktopb2g I got no fail I think that the javascript is running to quickly.
AndreiH, could you replicate this *before* adding that sleep?
Yes zac, I could replicate it and it was failing badly, even a time.sleep(1) would do the trick
I don't see why you can't just fix the Python wait that's in there?
Comment on attachment 8348036 [details] Github PR: https://github.com/mozilla-b2g/gaia/pull/14710 r+ pls merge this when the tree re-opens :)
Attachment #8348036 - Flags: review?(zcampbell) → review+
Attachment #8348036 - Flags: review?(florin.strugariu) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
This bug was not resolved, the test is still disabled and we should also fix it for Travis disabled = bug 949897 intermittently fails
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
When I am seeing this fail, javascript "holdHome" is not really holding Home button, instead it just goes to Homescreen, like homescreen button was only tapped and Cards View is not opened. It just stays there in Homescreen waiting for cards view.
This is P1 because we want to re-enable the test.
Assignee: nobody → zcampbell
Depends on: 956780
Whiteboard: [xfail]
Re-investigate enabling this by doing a long run on Travis, running it a few hundred times. It's enabled for TBPL and is stable there.
Attached file github pr (deleted) —
Let's re-enable this test, it's stable now.
Attachment #8348036 - Attachment is obsolete: true
Attachment #8348036 - Flags: review?(andrei.hutusoru)
Attachment #8366098 - Flags: review?(fyen)
Attachment #8366098 - Flags: review?(florin.strugariu)
Attachment #8366098 - Flags: review?(fyen) → review+
Attachment #8366098 - Flags: review?(florin.strugariu)
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Resolved. there's a separate task to fix up the intermittent failure on this test.
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: