Closed Bug 1291391 Opened 4 years ago Closed 3 years ago

[geckoview] Embedding testing for GeckoView API: run Android UI, Marionette test suites

Categories

(GeckoView :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: nalexander, Unassigned)

References

Details

This ticket tracks testing the "GeckoView API".  This is new ground:
we built Robocop, which has not served the product well, and is now
significantly out of date, to do this for Fennec, but we'll need
something different for GeckoView.

My guess is that this looks like JUnit 4 + Espresso tests, or possibly
JUnit 4 + Marionette tests, running in a custom GeckoView consuming
Android application.
Depends on: 1298540
Nick: is this bug about testing in continuous integration?  I filed bug 1298540 to implement a minimally-viable API, and I intend for that bug to land with unit tests (and a sample app).  So I don't think we need a separate bug to track testing.  Although we do need to figure out our testing story, and perhaps it helps to do so in a separate comment stream.  (And we definitely need to enable testing in CI.  So this bug could be used for that.)
(In reply to Myk Melez [:myk] [@mykmelez] from comment #1)
> Nick: is this bug about testing in continuous integration?

It could be.  I filed it as a meta-ish ticket to track whatever arises here, including the code, settings, documentation, CI, etc.

> (And we definitely need to enable testing in CI.  So this bug could be used
> for that.)

Whatever you prefer.
(In reply to Nick Alexander :nalexander from comment #2)
> (In reply to Myk Melez [:myk] [@mykmelez] from comment #1)
> > Nick: is this bug about testing in continuous integration?
> 
> It could be.  I filed it as a meta-ish ticket to track whatever arises here,
> including the code, settings, documentation, CI, etc.
> 
> > (And we definitely need to enable testing in CI.  So this bug could be used
> > for that.)
> 
> Whatever you prefer.

Hmm, let's use this bug to track the meta-ish work to figure out how we're going to do the testing generally. In which case this actually blocks bug 1298540 rather than depending on it.
Blocks: 1298540
No longer depends on: 1298540
Depends on: 1323185
We've got some tests going but they are handled in other bugs.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME
Product: Firefox for Android → GeckoView
You need to log in before you can comment on or make changes to this bug.