922708 - Reenable test_cards_view_kill_app_with_three_apps.py on TBPL once bug 915138 is addressed

Status: RESOLVED INVALID

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

Description

[Bob Silverberg [:bsilverberg]]

This test is about to be disabled, so this is a reminder bug to reenable it once it can be run again successfully on TBPL.

Comment 1

[Zac C (:zac)]

Attachment: github pr

Comment 2

[Zac C (:zac)]

So what I discovered here is that this test is failing due to a memory spike launching the apps rapidly. 
This rapid launching of apps is a bit of an unrealistic scenario anyway.

Although it's agreed in bug 915138 that there is some kind of memory regression, this test is a cards view functional test not a memory test. I don't like the sleeps but I'd rather have the functionality of this test and B2G team can chase memory regressions using other techniques.

I did experiment with the length of sleep, apps in use and order of launching them, too.

Comment 3

[Zac C (:zac)]

Attachment: github pr

I refactored it to use the app object and thus take advantage of any waits inside the app object's launch method.
It's a little better but didn't entirely remove the need for the sleep.

Comment 4

[Bob Silverberg [:bsilverberg]]

Comment on attachment 8333897 [details] [review]
github pr

My bad. This doesn't seem to be a permanent fix.

Comment 5

[Zac C (:zac)]

:( What have you found?

Comment 6

[Bob Silverberg [:bsilverberg]]

Sorry, Zac. More comments in the PR. Basically, there is still an intermittent failure with these changes.

Comment 7

[Zac C (:zac)]

OK! I've closed the pull.
Not sure what to do for the time being.

I'll unassign myself from this incase somebody else wants to have a shot.

Comment 8

[Florin Strugariu [:Bebe]]

Comment on attachment 8333897 [details] [review]
github pr

>https://github.com/mozilla-b2g/gaia/pull/13771

Comment 9

[Florin Strugariu [:Bebe]]

Made this obsolete as the pull is closed

Comment 10

[Zac C (:zac)]

Test no loner exists, marking invalid.