Closed
Bug 925723
Opened 12 years ago
Closed 12 years ago
Need to differentiate b2g and b2g desktop in mochitest manifests
Categories
(Testing :: Mochitest, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(2 files, 1 obsolete file)
|
984 bytes,
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
|
28.85 KB,
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
Right now b2g uses the old .json manifest format for excluding mochitests. B2G desktop uses the same file, which means we can't disable a test on one platform without doing the other. We need to either create a separate .json for b2g desktop, or switch to using the .ini format.
Joel, I know you landed the manifests for mochitest patch, is there a bug tracking converting to the new format?
| Assignee | ||
Comment 1•12 years ago
|
||
Joel estimates that getting the initial manifests built out could take a month or more. For now I'll just create a separate b2g-desktop.json and work with that.
Comment 2•12 years ago
|
||
to get this in the short term, lets create a new .json file. About 25-33% of the mochitest makefiles are converted to mochitest.ini formats and are being consumed by the build system now. Once those are fully converted, we can make the test runners use these, I imagine it would take a week or so to get the entire test infrastructure using manifests once they are fully implemented.
| Assignee | ||
Comment 3•12 years ago
|
||
Attachment #815839 -
Flags: review?(jgriffin)
| Assignee | ||
Comment 4•12 years ago
|
||
Attachment #815840 -
Flags: review?(jgriffin)
| Assignee | ||
Comment 5•12 years ago
|
||
Oops, forgot to add the manifest to the Makefile in the last patch.
Attachment #815839 -
Attachment is obsolete: true
Attachment #815839 -
Flags: review?(jgriffin)
Attachment #815843 -
Flags: review?(jgriffin)
Updated•12 years ago
|
Attachment #815843 -
Flags: review?(jgriffin) → review+
Updated•12 years ago
|
Attachment #815840 -
Flags: review?(jgriffin) → review+
| Assignee | ||
Comment 6•12 years ago
|
||
| Assignee | ||
Updated•12 years ago
|
Whiteboard: [leave-open]
| Assignee | ||
Comment 8•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•12 years ago
|
Whiteboard: [leave-open]
You need to log in
before you can comment on or make changes to this bug.
Description
•