Closed Bug 894467 Opened 13 years ago Closed 11 years ago

Make mochitest-chrome work on b2g

Categories

(Testing :: Mochitest, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 797164

People

(Reporter: ochameau, Unassigned)

References

Details

Attachments

(1 file, 1 obsolete file)

I haven't found any way to run mochitest-chrome on b2g, I can't write extensive tests for apps with existing test frameworks running on b2g. xpcshell tests are too limited as you can't have DOM scripting working. mochitest plain are also not very well fited when you need chrome priviledges.
At the end we just miss very minimal changes... Here is a way to make them work on b2g desktop. We may try to set these prefs in runtest.py in order to avoid having a different codepath for b2g.
Attachment #776488 - Flags: feedback?(21)
We mostly need some b2g specifics to ensure loading mochitest testURL into b2g content iframe. (B2g ignores the URL being passed as argument to the process) The redirect.html looks useless on it and more painfull to support than to bypass. Otherwise, I had to move testURL computation before the call to buildProfile in order to be able to set the two b2g prefs with the testURL. But I had to still ensure calling makeTestConfig after the call to buildProfile... With that patch, ./mach mochitest-plain and ./mach mochitest-chrome work both out of the box. https://tbpl.mozilla.org/?tree=Try&rev=16e076404f1c
Attachment #776488 - Attachment is obsolete: true
Attachment #776488 - Flags: feedback?(21)
Attachment #777180 - Flags: review?(jmaher)
Attachment #777180 - Flags: feedback?(21)
Comment on attachment 777180 [details] [diff] [review] Make mochitest-plain and mochitest-chrome work on b2g Review of attachment 777180 [details] [diff] [review]: ----------------------------------------------------------------- This looks harmless and simple. Does this allow chrome:// URIs to be loaded from the network? Do you have all the tests on the device? Right now the tests are bundled in a .jar file and loaded up dynamically- is this .jar file on the device? I didn't see any code that indicated changing that.
Attachment #777180 - Flags: review?(jmaher) → review+
Comment on attachment 777180 [details] [diff] [review] Make mochitest-plain and mochitest-chrome work on b2g Do we need anything else before landing that patch?
Attachment #777180 - Flags: feedback?(21) → feedback+
Hum this is now completely broken due to the switch to shell.html The mochitest runner (the platform one, not the b2g specific one) depends on xul overlays to work/run...
Blocks: 963501
No longer blocks: devtools-layers
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
Component: Mochitest Chrome → Mochitest
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: