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)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: alive, Assigned: zcampbell)
References
Details
(Keywords: intermittent-failure, Whiteboard: [xfail])
Attachments
(1 file, 1 obsolete file)
Should not cause by https://travis-ci.org/mozilla-b2g/gaia/builds/15321988
and the test run well on my local try.
No idea.
Updated•11 years ago
|
Component: Gaia::System → Gaia::UI Tests
Comment 1•11 years ago
|
||
We're seeing this intermittent a lot. Do you mind if we disable it?
Comment 2•11 years ago
|
||
I went ahead and disabled it for now. https://github.com/mozilla-b2g/gaia/commit/fa88c4e66760d9462ad6acd7b611eb64c9a917ce
Updated•11 years ago
|
Keywords: intermittent-failure
Assignee | ||
Comment 3•11 years ago
|
||
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.
Comment 5•11 years ago
|
||
I ran this test and I got 3 out of 21 fails: https://pastebin.mozilla.org/3789082
I will investigate the problem
Priority: P1 → --
Updated•11 years ago
|
Priority: -- → P1
Comment 6•11 years ago
|
||
Attachment #8348036 -
Flags: review?(zcampbell)
Attachment #8348036 -
Flags: review?(florin.strugariu)
Attachment #8348036 -
Flags: review?(andrei.hutusoru)
Comment 7•11 years ago
|
||
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.
Assignee | ||
Comment 8•11 years ago
|
||
(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?)
Comment 9•11 years ago
|
||
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.
Assignee | ||
Comment 10•11 years ago
|
||
AndreiH, could you replicate this *before* adding that sleep?
Comment 11•11 years ago
|
||
Yes zac, I could replicate it and it was failing badly, even a time.sleep(1) would do the trick
Assignee | ||
Comment 12•11 years ago
|
||
I don't see why you can't just fix the Python wait that's in there?
Assignee | ||
Comment 13•11 years ago
|
||
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+
Updated•11 years ago
|
Attachment #8348036 -
Flags: review?(florin.strugariu) → review+
Comment 14•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 15•11 years ago
|
||
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 → ---
Comment 16•11 years ago
|
||
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.
Assignee | ||
Comment 17•11 years ago
|
||
This is P1 because we want to re-enable the test.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → zcampbell
Updated•11 years ago
|
Whiteboard: [xfail]
Assignee | ||
Comment 18•11 years ago
|
||
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.
Assignee | ||
Comment 19•11 years ago
|
||
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)
Updated•11 years ago
|
Attachment #8366098 -
Flags: review?(fyen) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #8366098 -
Flags: review?(florin.strugariu)
Assignee | ||
Comment 20•11 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Comment 21•11 years ago
|
||
This test is failing on device:
http://qa-selenium.mv.mozilla.com:8080/job/b2g.hamachi.mozilla-central.ui/499/testReport/test_cards_view_kill_apps_with_two_apps/TestCardsViewTwoApps/test_kill_app_from_cards_view/
I will reopen the bug and we need to xfail it on device
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 23•11 years ago
|
||
Assignee | ||
Comment 24•11 years ago
|
||
Resolved.
there's a separate task to fix up the intermittent failure on this test.
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•