Android xpcshell harness sets GRE_HOME inappropriately
Categories
(Testing :: XPCShell Harness, defect, P1)
Tracking
(firefox78 fixed)
Tracking | Status | |
---|---|---|
firefox78 | --- | fixed |
People
(Reporter: gbrown, Assigned: gbrown)
Details
(Whiteboard: [kaios])
Attachments
(1 file)
The Android xpcshell-test harness sets environment variable GRE_HOME to /data/data/org.mozilla.geckoview.test, when running against a geckoview build, or /data/data/<package-name> if it can determine a package name from an apk (as was supported in fennec). But org.mozilla.geckoview.test is not necessarily installed (never in CI, never for KaiOS) -- this is just pointing an environment variable to a non-existent directory. Surely we can do better!
Assignee | ||
Comment 1•4 years ago
|
||
It turns out this is why uriloader/exthandler/tests/unit/test_punycodeURIs.js has failed on Android for years.
Assignee | ||
Comment 2•4 years ago
|
||
Previously this would typically set GRE_HOME to /data/data/org.mozilla.geckoview.test,
even though org.mozilla.geckoview.test is not normally installed when running
xpcshell tests -- a non-existent directory in a privileged location!
The new location, remoteBinDir, is typically /data/local/xpcb, the location of the
xpcshell executable.
Pushed by gbrown@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8b703baa8a50 Improve GRE_HOME setting in remote xpcshell tests; r=bc
Comment 4•4 years ago
|
||
bugherder |
Description
•