1086603 - Add test test_browser_navigation.py to share a link to the SMS app

Status: RESOLVED FIXED

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

Description

[Johan Lorenzo [:jlorenzo]]

We need to automate this manual test: https://moztrap.mozilla.org/manage/case/2027/

Comment 1

[Johan Lorenzo [:jlorenzo]]

Attachment: Gaia PR

I added a new test, test_browser_navigation does already a lot of stuff. It was inspired by test_browser_bookmark.py

Comment 2

[Robert Chira [:RobertC]]

Comment on attachment 8508784 [details] [review]
Gaia PR

r-, comments in PR

Comment 3

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8508784 [details] [review]
Gaia PR

The things you pointed out should be fixed now.

Comment 4

[Geo Mealer [:geo] -- This account is inactive after 2015-07-07]

Comment on attachment 8508784 [details] [review]
Gaia PR

LGTM, pending my minor naming comment and Robert's r+

Comment 5

[Robert Chira [:RobertC]]

Comment on attachment 8508784 [details] [review]
Gaia PR

Merged on master:
https://github.com/mozilla-b2g/gaia/commit/dde9918a8e76cec4e29bb4f590cdc09dfbbe3cf2

Comment 6

[Robert Chira [:RobertC]]

Johan, can you uplift this to v2.1 please and check if it works as expected?

Comment 7

[Johan Lorenzo [:jlorenzo]]

Attachment: Gaia PR 2.1

Tested and working. Nothing had to be changed between the master PR and this one.

Comment 8

[Robert Chira [:RobertC]]

Comment on attachment 8513352 [details] [review]
Gaia PR 2.1

Merged on v2.1:
https://github.com/mozilla-b2g/gaia/commit/151f0cf9c4bf92fde7462780af330667e163ac5f