Closed Bug 902124 Opened 9 years ago Closed 9 years ago

Write mozharness script for gaia integration tests

Categories

(Release Engineering :: Applications: MozharnessCore, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jgriffin, Assigned: gaye)

References

Details

Attachments

(3 files)

We're getting close to the point where we can start writing a mozharness script for invoking gaia-integration-tests in buildbot.
Product: mozilla.org → Release Engineering
Assignee: nobody → gaye
Hi Aki!

Thanks for all of your help. This patch adds a new mozharness script to run |MOCHA_REPORTER=mocha-tbpl-reporter| in the gaia directory and then publish the results to TBPL. Would you mind taking a look? I'd also like to know how I can push this to a staging-type environment to make sure that it works like I expect for it to. Do you all usually run these locally? There weren't any tests to exercise the old gaia_unit.py, so I'm a bit confused.
Attachment #809361 - Flags: review?(aki)
Attached file Pull request to Gaia
Hi James,

In addition to the patch I sent to Aki, I needed a tiny bit of plumbing to help the Makefile propagate choosing the mocha-tbpl-reporter over the dot reporter for our marionette tests.
Attachment #809372 - Flags: review?(jlal)
Comment on attachment 809372 [details] [review]
Pull request to Gaia

wait for the build to go green then land :D
Attachment #809372 - Flags: review?(jlal) → review+
Comment on attachment 809361 [details] [diff] [review]
Patch for bug 902124

I think this is overall good.

However, I don't think we should be importing anything from scripts/.
Can you move scripts/gaia.py to mozharness/mozilla/testing/gaia.py ?
Then the +sys.path.insert(1, os.path.dirname(sys.path[0])) line will need to change (three layers of os.path.dirname()) and the |+from scripts.gaia import GaiaTest| line will need to change to |from mozharness.mozilla.testing.gaia import GaiaTest|.

Other than that this looks good.  Thanks!
Attachment #809361 - Flags: review?(aki) → review+
Depends on: 900663
Sure I'll make those changes... thanks Aki!
Blocks: 925022
Aki! Please check this in for me!
Attachment #815148 - Flags: checked-in?
Flags: needinfo?(aki)
Whiteboard: checkin-needed
Comment on attachment 815148 [details] [diff] [review]
Updated patch to mozharness

Checked in, minus the .hgtags changes.
https://hg.mozilla.org/build/mozharness/rev/e2e19076b0d0
Attachment #815148 - Flags: checked-in? → checked-in+
Flags: needinfo?(aki)
Whiteboard: checkin-needed
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
in production
Attachment mime type: text/plain → text/x-github-pull-request
Blocks: 999711
Component: General Automation → Mozharness
You need to log in before you can comment on or make changes to this bug.