Closed Bug 1097334 Opened 10 years ago Closed 10 years ago

console tests are failing in mochitest-jetpack

Categories

(Add-on SDK Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla36

People

(Reporter: mossop, Assigned: mossop)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
cfx sets the log level to info by default so we need to do the same in the mochitest-harness to make sure the console tests work right.
Attachment #8521014 - Flags: review?(evold)
Hey Dave,

Can you add a link to the failing tests?
Flags: needinfo?(dtownsend+bugmail)
Flags: needinfo?(dtownsend+bugmail)
Attachment #8521014 - Flags: review?(evold) → review+
https://hg.mozilla.org/integration/fx-team/rev/6286826a1ebb

Examples of failures:

TEST-UNEXPECTED-FAIL | jetpack-package/addon-sdk/source/test/test-content-worker-parent.js.test:console method log functions properly | undefined - [
TEST-UNEXPECTED-FAIL | jetpack-package/addon-sdk/source/test/test-content-worker-parent.js.test:ensure console.xxx works in cs | console methods have been called successfully, in expected order - "["time","log","info","warn","error","error","timeEnd"]" == "["error","error"]"
TEST-UNEXPECTED-FAIL | jetpack-package/addon-sdk/source/test/test-content-worker-parent.js.test:ensure console.xxx works in cs | console messages have correct log levels, in expected order - "["time","log","info","warn","error","error","timeEnd"]" == "["error","error"]"
TEST-UNEXPECTED-FAIL | jetpack-package/addon-sdk/source/test/test-page-mod.js.testConsole | Should have seen the console message - true == false
TEST-UNEXPECTED-FAIL | jetpack-package/addon-sdk/source/test/test-page-mod.js.testConsole | Should have seen the right inner ID - 1152 == undefined
https://hg.mozilla.org/mozilla-central/rev/6286826a1ebb
Assignee: nobody → dtownsend+bugmail
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.