[geckoview] Move GeckoView unit tests to under geckoview/

RESOLVED FIXED in Firefox 60

Status

enhancement
RESOLVED FIXED
Last year
6 months ago

People

(Reporter: jchen, Assigned: jchen)

Tracking

58 Branch
mozilla60
Unspecified
Android
Dependency tree / graph

Firefox Tracking Flags

(firefox60 fixed)

Details

Attachments

(3 attachments)

Right now we have some unit tests under Fennec that really belong in GeckoView because they test GeckoView code.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 4

Last year
mozreview-review
Comment on attachment 8945640 [details]
Bug 1433322 - 1. Move publicsuffix utils to under Fennec;

https://reviewboard.mozilla.org/r/215770/#review221498

This will fail, since moz.build still exists -- so that needs to be addressed.  But I think this is the right home for this functionality!
Attachment #8945640 - Flags: review?(nalexander) → review+

Comment 5

Last year
mozreview-review
Comment on attachment 8945641 [details]
Bug 1433322 - 2. Mass move GeckoView unit tests to under geckoview/;

https://reviewboard.mozilla.org/r/215772/#review221500

I don't quite follow the test runner details, but I'm happy for you to own this!
Attachment #8945641 - Flags: review?(nalexander) → review+

Comment 6

Last year
mozreview-review
Comment on attachment 8945642 [details]
Bug 1433322 - 3. Run GeckoView unit tests with 'mach android test';

https://reviewboard.mozilla.org/r/215774/#review221502

As usual, r- is f+.

Three things:

1) my guess is that test failures don't print anything right now, since https://searchfox.org/mozilla-central/source/mobile/android/mach_commands.py#86 needs to be enlarged.  You can test this by introducing failures and running |mach android test| locally.
2) https://searchfox.org/mozilla-central/source/taskcluster/ci/build/android-stuff.yml#19 needs to be enlarged, so that reports are shared.
3) this will conflict a little with my patches for https://bugzilla.mozilla.org/show_bug.cgi?id=1411654, which I'm working hard to get landed.  In particular, those patches change some of the variant names, and update the Robolectric bits.  Can you try to build on top of that series?

Thanks for digging in!
Attachment #8945642 - Flags: review?(nalexander) → review-
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 10

Last year
mozreview-review
Comment on attachment 8945642 [details]
Bug 1433322 - 3. Run GeckoView unit tests with 'mach android test';

https://reviewboard.mozilla.org/r/215774/#review223122

Thanks for catching some errors, which I no doubt introduced in my 3.0 patch!  I'm going to be more careful about empty input sets in the future, 'cuz you make a good point.  Bombs away!
Attachment #8945642 - Flags: review?(nalexander) → review+

Comment 11

Last year
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f7862b1b5e9b
1. Move publicsuffix utils to under Fennec; r=nalexander
https://hg.mozilla.org/integration/mozilla-inbound/rev/0a74b6699ff3
2. Mass move GeckoView unit tests to under geckoview/; r=nalexander
https://hg.mozilla.org/integration/mozilla-inbound/rev/e91adf8f9294
3. Run GeckoView unit tests with 'mach android test'; r=nalexander

Updated

6 months ago
Product: Firefox for Android → GeckoView
Version: Firefox 58 → 58 Branch
Target Milestone: Firefox 60 → mozilla60
You need to log in before you can comment on or make changes to this bug.