Closed
Bug 1448188
Opened 7 years ago
Closed 7 years ago
Stand up GeckoView tests using e10s
Categories
(GeckoView :: Sandboxing, enhancement, P1)
Tracking
(firefox59 wontfix, firefox60 wontfix, firefox61 fixed)
RESOLVED
FIXED
mozilla61
People
(Reporter: cpeterson, Assigned: jchen)
References
Details
(Whiteboard: [geckoview:klar])
Attachments
(1 file)
We want to be able to test GeckoView using e10s and standalone without Fennec. We might need to limit these tests to the Android x86 emulators because the real devices are so slow.
Reporter | ||
Comment 1•7 years ago
|
||
Need to build test APK in automation. Once we are building the test APK, gbrown said he could take over standing up the tests. This bug is not dependent on standing up the x86 emulators.
Assignee: nobody → nchen
Priority: -- → P1
Summary: Stand up GeckoView tests using e10s (on x86 emulator?) → Stand up GeckoView tests using e10s
Whiteboard: [geckoview:klar]
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8965002 -
Flags: review?(nalexander)
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8965002 [details]
Bug 1448188 - Build geckoview instrumentation test in automation;
https://reviewboard.mozilla.org/r/233738/#review239474
This looks fine -- if it's green on try, it's good for me.
I feel like I talked to snorp about this, but by pushing the androidTest compilation into |mach android archive-geckoview|, you won't see changes when running |mach build| locally. I feel like you will get frustrated "breaking the build" when you just busted GV test compilation... but I'll let that be your decision.
Attachment #8965002 -
Flags: review?(nalexander) → review+
Assignee | ||
Comment 4•7 years ago
|
||
That's a good point. I tried moving :geckoview:assembleAndroidTest to the assemble-app step, but looks like that depends on building GeckoView first through geckoview:assemble (because of GV's special omni.ja, etc), and I'm not sure we want to be spending more time building all of GeckoView during |mach build/package|. So I think I'll leave it like this for now?
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d533d09f17f8
Build geckoview instrumentation test in automation; r=nalexander
Comment 6•7 years ago
|
||
(In reply to Jim Chen [:jchen] [:darchons] from comment #4)
> That's a good point. I tried moving :geckoview:assembleAndroidTest to the
> assemble-app step, but looks like that depends on building GeckoView first
> through geckoview:assemble (because of GV's special omni.ja, etc), and I'm
> not sure we want to be spending more time building all of GeckoView during
> |mach build/package|. So I think I'll leave it like this for now?
That's fine by me. I don't know what the long-term evolution of all this is, and I'm happy to have you drive that.
Comment 7•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Reporter | ||
Comment 8•7 years ago
|
||
Geoff, now that we will be building a GeckoView androidTest APK, what are the next steps for running standard reftests and mochitests in GeckoView?
Flags: needinfo?(gbrown)
Comment 9•7 years ago
|
||
The next thing I am focused on is bug 1445716 - running these geckoview junit tests in automation, on existing emulators.
It looks to me like it is almost possible to run mochitests and reftests locally in geckoview_example, except for a hiccup on shutdown -- I've just filed bug 1452399 to sort that out.
:bc is making progress on running tests on bitbar devices which is turning up some issues around running tests on newer versions of Android; we're both investigating.
Beyond those issues, to run geckoview mochitests and reftests in automation, we intend to run against new x86 emulators on packet.net, but are currently blocked waiting for taskcluster support for packet.net.
Flags: needinfo?(gbrown)
Assignee | ||
Comment 10•7 years ago
|
||
I think we want to eventually be using TestRunnerActivity, which is different from geckoview_example/GeckoViewActivity, for mochitests/reftests (Bug 1291387). TestRunnerActivity is included in the new androidTest APK.
Updated•6 years ago
|
Product: Firefox for Android → GeckoView
Updated•6 years ago
|
Target Milestone: Firefox 61 → mozilla61
Reporter | ||
Comment 11•2 years ago
|
||
Moving some e10s bugs to the new GeckoView::Sandboxing component.
Component: General → Sandboxing
You need to log in
before you can comment on or make changes to this bug.
Description
•