TestEngineering/Web/Automation/Gaia UI Testing: Difference between revisions

Jump to navigation Jump to search

TestEngineering/Web/Automation/Gaia UI Testing (view source)

Revision as of 22:35, 25 November 2013

573 bytes removed ,  25 November 2013
→‎Reviewing and Merging Uplifts
Line 160: Line 160:
Often a pull request which is an uplift from master to a branch of interest will contain code that is identical to that which was committed to master. That code has already been reviewed for correctness and style, so it may not be necessary to review the code itself as carefully. You will need to determine whether that is the case yourself. You could do that by comparing the files changed, after applying the commit in the pull request, with the same files in the master branch.
Often a pull request which is an uplift from master to a branch of interest will contain code that is identical to that which was committed to master. That code has already been reviewed for correctness and style, so it may not be necessary to review the code itself as carefully. You will need to determine whether that is the case yourself. You could do that by comparing the files changed, after applying the commit in the pull request, with the same files in the master branch.


Regardless of the decision made above, before merging the code:
Regardless of the decision made above, follow the procedure in [https://wiki.mozilla.org/QA/Execution/Web_Testing/Automation/Gaia_UI_Testing#Merging_Pull_Requests Merging Pull Requests]
# Any affected tests should be run against the branch in question.
# A passing Travis build of the pull request should have completed.
## If the Travis build failed check it to see if the ''gaia_ui_tests'' step failed, and if it did whether any of the affected tests failed. Although it is preferred that no pull request be merged with a failing Travis build, if you are confident that the Travis failure had nothing to do with the changes in your pull request you may merge it, at your own discretion.
## You may also request a rerun of the Travis build, which you do via the [https://travis-ci.org/mozilla-b2g/gaia/pull_requests Travis UI], if you would prefer to see it pass before merging.


== Finding Bugs That Require Uplifts ==
== Finding Bugs That Require Uplifts ==
Bobsilverberg
Confirmed users
425

edits

Retrieved from "https://wiki.mozilla.org/Special:MobileDiff/774377"

Navigation menu