Closed
Bug 991446
Opened 11 years ago
Closed 11 years ago
Fix coverage when tests are run in their own sandbox
Categories
(Firefox OS Graveyard :: Gaia::TestAgent, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kgrandon, Assigned: rickychien)
References
Details
Attachments
(4 files)
Bug 864178 adds support for sandboxing of test files which is greatly needed to stop us from breaking TBPL. Unfortunately this has the side-effect of making the COVERAGE=1 flag not as nice because it gets reported for each and every file.
Assignee | ||
Comment 1•11 years ago
|
||
I'm going to fix this if I feel free
Assignee: nobody → ricky060709
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #8408112 -
Flags: review?(yurenju.mozilla)
Assignee | ||
Comment 3•11 years ago
|
||
Attachment #8408113 -
Flags: review?(yurenju.mozilla)
Assignee | ||
Comment 4•11 years ago
|
||
Attachment #8408114 -
Flags: review?(yurenju.mozilla)
Comment 5•11 years ago
|
||
Comment on attachment 8408113 [details]
Js-test-agent PR
discussed with Ricky offline and r=yurenju if issues mentioned on github are fixed.
Attachment #8408113 -
Flags: review?(yurenju.mozilla) → review+
Updated•11 years ago
|
Attachment #8408112 -
Flags: review?(yurenju.mozilla) → review+
Updated•11 years ago
|
Attachment #8408114 -
Flags: review?(yurenju.mozilla) → review+
Assignee | ||
Comment 6•11 years ago
|
||
Merged.
https://github.com/mozilla-b2g/gaia/commit/f96abb3cc87a6222c50f4cfeed729c86c6bf0a51
https://github.com/mozilla-b2g/js-test-agent/commit/18635979112886b05336e4962ff54a1c7d3e095f
https://github.com/mozilla-b2g/gaia-node-modules/commit/c97b707394d717512e280de5376ee435505f8eb6
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 7•11 years ago
|
||
Reporter | ||
Comment 8•11 years ago
|
||
Landed a follow-up to include the latest revision in gaia_node_modules.revision which is now necessary: https://github.com/mozilla-b2g/gaia/commit/7e93b6de9633a162b735252e8182974456f741f4
Assignee | ||
Comment 9•11 years ago
|
||
Thanks Kevin :)
You need to log in
before you can comment on or make changes to this bug.
Description
•