1097334 - console tests are failing in mochitest-jetpack

Status: RESOLVED FIXED

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

Description

[Dave Townsend [:mossop]]

Attachment: patch

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.

Comment 1

[Erik Vold [:erikvold] (please needinfo? me)]

Hey Dave,

Can you add a link to the failing tests?

Comment 2

[Dave Townsend [:mossop]]

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

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6286826a1ebb