1073436 - Update launch atom to work for apps that use hash in the src

Status: RESOLVED WONTFIX

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

Description

[Viorela Ioia [:viorela]]

When we use an app sometimes we may change its URL/iframe's src while using it. 
We have the scenario of Cost Control app, where its URL changed with an anchor tag on the end of it after part of the app was navigated to.

When the app was launched the second time (although it was only open in the background), the atom's expected launch_path does not match the src of the app and the atom times out. The atom can only handle the default expected launch path, not that of a running app.

We need to update our atom to handle the case when there's a hash added to an app's src.
And also update the existing unit test https://github.com/mozilla-b2g/gaia/blob/055c380b17ae0b7e18f4886d3988eca96d6c2048/tests/python/gaia-ui-tests/gaiatest/tests/unit/test_warm_launch.py to append a hash string at the end.

Comment 1

[Viorela Ioia [:viorela]]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/24470

Comment 2

[Viorela Ioia [:viorela]]

Dave, can you, please, give more info about how should we modify the unit test?
In order to get a hash added to the src, we need to navigate through the Cost Control first time use steps.
Is that what I should do?

Comment 3

[Dave Hunt [:davehunt] [he/him] ⌚GMT]

(In reply to Viorela Ioia [:viorela] from comment #2)
> Dave, can you, please, give more info about how should we modify the unit
> test?
> In order to get a hash added to the src, we need to navigate through the
> Cost Control first time use steps.
> Is that what I should do?

Can you not just execute some JavaScript to append #something to the end of the Window location?

Comment 4

[Zac C (:zac)]

Comment on attachment 8495902 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/24470

r-, this is causing apps launched through a launch path to fail to launch on TBPL.

I also think test_warm_launch should use an app that works on desktopb2g so we can continue to run that test on TBPL.

Comment 5

[BMO Automation]

Firefox OS is not being worked on