943338 - Merge test_cards_view_launch tests

Status: RESOLVED FIXED

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

Description

[Zac C (:zac)]

Merge 
test_cards_view_launch_app
test_cards_view_with_three_apps

Merge them and refactor to use two apps so we don't run into memory issues.

Change the test case to:
1. Load two apps
2. Load cards view
3. Swipe across to second app
4. Tap app
5. Check that self.apps.displayed_app is the expected one.

Comment 1

[Florin Strugariu [:Bebe]]

akruglov are you still working on this task?
Can we help you?

Comment 2

[akruglov]

Yes, I need some advise on one part of the test.
I can't accomplish swipe from one app card to another.
I've tried tap, flick, press().wait().move_to().release() and so on.
I'm out of ideas, perhaps, I'm missing something simple or it's just blocked due to some issue in how marionette interacts with those elements.

Comment 3

[akruglov]

Attachment: https://github.com/mozilla-b2g/gaia/pull/14367

Comment 4

[Florin Strugariu [:Bebe]]

Comment on attachment 8342486 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/14367

Thanks for the update  Sasha I think Zac provided the necessary feedback for this pull

Comment 5

[Bob Silverberg [:bsilverberg]]

Comment on attachment 8342486 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/14367

r+, but please squash the commits into a single commit.

Comment 6

[akruglov]

Done

Comment 7

[Zac C (:zac)]

Comment on attachment 8342486 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/14367

r+

Comment 8

[Zac C (:zac)]

Merged:
https://github.com/mozilla-b2g/gaia/commit/96085a416d4396933c16caefae909028e535b5a0