935319 - Improve gaia-ui-tests documentation for users getting starting with b2g-desktop builds.

Status: RESOLVED FIXED

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

Description

[L. Jonathan Di Trapani [:ditrapani]]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:25.0) Gecko/20100101 Firefox/25.0 (Beta/Release)
Build ID: 20131028113235

Steps to reproduce:

I tried to get gaiatest to run on a b2g-desktop build by following the documentation in the gaia-ui-tests README file.


Actual results:

It took a long time to figure it out because the documentation (README file) was out-of-date with incorrect links and incorrect instructions.



Expected results:

Running gaiatest on b2g-desktop is straightforward because the documentation (README file) is up-to-date and correct.

Comment 1

[L. Jonathan Di Trapani [:ditrapani]]

Attachment: Github pull request with updated documentation (gaia-ui-tests README)

Comment 2

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

Comment on attachment 827805 [details] [review]
Github pull request with updated documentation (gaia-ui-tests README)

The patch looks good, I've added a few comments to be addressed.

Comment 3

[L. Jonathan Di Trapani [:ditrapani]]

(In reply to Dave Hunt (:davehunt) from comment #2)
Dave,

I believe I corrected all the issues with the pull request.  Let me know if I need to do anything else for this one.  Thanks!

Comment 4

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

Looks good, just a couple of minor tweaks and it'll be ready to merge. When you push changes, please change the review flag back to ? so I know there's something new to review. Thanks.

Comment 5

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

Comment on attachment 827805 [details] [review]
Github pull request with updated documentation (gaia-ui-tests README)

Landed as:
https://github.com/mozilla-b2g/gaia/commit/c26480b22ce28c812c347290dd4bad090d83db6f (master)
https://github.com/mozilla-b2g/gaia/commit/dd6666b54c4fe6f25d0ca9f3dd2057059618286a (v1.2)

Thanks!