Closed
Bug 894467
Opened 13 years ago
Closed 11 years ago
Make mochitest-chrome work on b2g
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 797164
People
(Reporter: ochameau, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
|
5.23 KB,
patch
|
jmaher
:
review+
vingtetun
:
feedback+
|
Details | Diff | Splinter Review |
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.
| Reporter | ||
Comment 1•13 years ago
|
||
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)
| Reporter | ||
Comment 2•13 years ago
|
||
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 3•13 years ago
|
||
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 4•12 years ago
|
||
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+
| Reporter | ||
Comment 5•12 years ago
|
||
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...
Updated•12 years ago
|
Blocks: devtools-layers
Updated•12 years ago
|
No longer blocks: devtools-layers
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
| Assignee | ||
Updated•8 years ago
|
Component: Mochitest Chrome → Mochitest
You need to log in
before you can comment on or make changes to this bug.
Description
•