Closed Bug 1553225 Opened 6 months ago Closed 3 months ago

Android 7.0 xpcshell tests run against the fennec apk

Categories

(Testing :: General, defect, P3)

defect

Tracking

(firefox70 fixed)

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: gbrown, Assigned: gbrown)

References

Details

Attachments

(1 file)

Android 7.0 xpcshell tests run against the fennec apk. Structurally, it is easy to run against geckoview, but when I initially tried, there were unexplained test failures.

Priority: -- → P2

I have verified the same test failures (like netwerk/test/httpserver/test/test_byte_range.js) running locally against the 7.0 x86_64 emulator. In that environment, I have verified that the same libraries and the same modules (like httpd.js) are pushed to device regardless of apk. Of course the apk differs (used for --greomni).

The failure in the geckoview run of netwerk/test/httpserver/test/test_byte_range.js occurs because the GetService call at

https://searchfox.org/mozilla-central/rev/6c9f60f8cc064a1005cd8141ecd526578ae9da7a/netwerk/test/httpserver/httpd.js#2767

fails:

JavaScript error: resource://gre/modules/XPCOMUtils.jsm, line 433: NS_NOINTERFACE: 
Depends on: 1553943
Priority: P2 → P1
Depends on: 1554276
Depends on: 1554326
Assignee: gbrown → nobody
Priority: P1 → P3

:snorp - When you get a chance, look at the dependent bugs please.

Flags: needinfo?(snorp)
See Also: → 1558185
See Also: → 1563281
Blocks: 1566423

This is blocking bug 1566423 and the fennec/geckoview issue is causing confusion: Let's try to sort this out soon. All the issues are documented in the dependent bugs, but we need decisions about how to resolve those.

Depends on: 1569053
No longer depends on: 1553943
See Also: → 1553943

(In reply to Geoff Brown [:gbrown] from comment #5)

:snorp - When you get a chance, look at the dependent bugs please.

I've r?'d you with proposed solutions for the remaining issues.

Flags: needinfo?(snorp)
Assignee: nobody → gbrown

Switch the target apk for Android 7.0 xpcshell-test to geckoview. At the same time,
update the mach command for local runs to select the geckoview apk rather than the
fennec one, by default.

Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6025ec39360f
Run Android 7.0 xpcshell tests against geckoview apk; r=jmaher
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.