Closed Bug 1406042 Opened 4 years ago Closed 4 years ago

Copy relevant old console test to the new frontend


(DevTools :: Console, enhancement, P1)



(firefox58 fixed)

Firefox 58
Tracking Status
firefox58 --- fixed


(Reporter: nchevobbe, Assigned: nchevobbe)


(Blocks 1 open bug)


(Whiteboard: [reserve-console-html])


(1 file)

In this bug, all the bug tagged as relevant in, or that have the "Browser Console" extra should be copied other the new frontend test directory.
- The tests that contains the bug number in their file should be renamed
- The tests that are prefixed with browser_console without actually testing the browser console should be renamed browser_webconsole_xx
- The tests that are about the console input should be renamed browser_jsterm_xx
- The support files should also be copied (and renamed if matches some of the elements described above)

Then, in the browser.ini file, all the new test should have a `skip-if=true`, followed by the bug number in which should happen the migration, if there is one in the dashboard.

Beware of also copying any extra info (subsuite, skip on specific platforms, …) from the old browser-ini
Assignee: nobody → nchevobbe
Priority: -- → P1
Whiteboard: [console-html]
Whiteboard: [console-html] → [console-html] [triage]
Whiteboard: [console-html] [triage] → [reserve-console-html]
Comment on attachment 8915887 [details]
Bug 1406042 - Copy relevant old console test to the new frontend;

Looks reasonable to me.

R+ assuming try is green.

This is great amount of work, thanks!

Attachment #8915887 - Flags: review?(odvarko) → review+
I'm seeing ESLint failures, I think eslint was disabled on the old tests. Let me eslint ignore the new tests, and we'll enable them one by one when doing the migration.
Pushed by
Copy relevant old console test to the new frontend; r=Honza
Backed out for failing devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_property_provider.js:

That got catched by the early running test-verify which runs modified tests. It failed in the first run, so either it will also fail in devtools runs or depends on the status from a test which usually runs before it.

Push with failures:
Failure log:

[task 2017-10-07T14:26:42.079Z] 14:26:42     INFO - *** Start BrowserChrome Test Results ***
[task 2017-10-07T14:26:42.120Z] 14:26:42     INFO - checking window state
[task 2017-10-07T14:26:42.182Z] 14:26:42     INFO - TEST-START | devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_property_provider.js
[task 2017-10-07T14:26:42.359Z] 14:26:42     INFO - TEST-INFO | started process screentopng
[task 2017-10-07T14:26:43.327Z] 14:26:43     INFO - TEST-INFO | screentopng: exit 0
[task 2017-10-07T14:26:43.328Z] 14:26:43     INFO - TEST-UNEXPECTED-FAIL | devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_property_provider.js | Exception thrown - at chrome://mochitests/content/browser/devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_property_provider.js:14 - ReferenceError: loadTab is not defined
Flags: needinfo?(nchevobbe)
This is a test migration and I forgot to mark that one as always skipped.
Doing it now, and it will be enabled in a specific bug.
Flags: needinfo?(nchevobbe)
Pushed by
Copy relevant old console test to the new frontend; r=Honza
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.