Closed
Bug 600095
Opened 14 years ago
Closed 14 years ago
Followup test fixes after lazy console lands
Categories
(DevTools :: General, defect, P3)
DevTools
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ddahl, Assigned: ddahl)
References
Details
(Whiteboard: [patch in bug 587734])
There are a few tests that need to be fixed or removed or re-worked after the lazy console lands:
toolkit/components/console/hudservice/tests/browser/browser_webconsole_consoleonpage.js-60- // is(consoleIFrame, undefined, "Console object was removed fro
m iFrame");
toolkit/components/console/hudservice/tests/browser/browser_webconsole_consoleonpage.js:61: // TODO: file a followup bug to fix this test
toolkit/components/console/hudservice/tests/browser/browser_webconsole_bug_593003_iframe_wrong_hud.js-157-function test() {
toolkit/components/console/hudservice/tests/browser/browser_webconsole_bug_593003_iframe_wrong_hud.js-158- return;
toolkit/components/console/hudservice/tests/browser/browser_webconsole_bug_593003_iframe_wrong_hud.js:159: // TODO: this test is borked by the new lazy console
.
toolkit/components/console/hudservice/tests/browser/browser_webconsole_bug_586142_insert_newlines.js-27-function test()
toolkit/components/console/hudservice/tests/browser/browser_webconsole_bug_586142_insert_newlines.js-28-{
toolkit/components/console/hudservice/tests/browser/browser_webconsole_bug_586142_insert_newlines.js-29- return;
toolkit/components/console/hudservice/tests/browser/browser_webconsole_bug_586142_insert_newlines.js:30: // TODO: figure out why this test fails. file a follow
up bug
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js.rej-26-+ // return here as the below bits fail. not sure why. moving on.
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js.rej:27:+ // TODO: file a bug about this failing test.
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js.rej-28-+
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js:303: // TODO fix this tes, file followup bug
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js-304- // let logLogged = (outputChildren[2].textContent.indexOf("logging")
!= -1);
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js-305- // ok(logLogged, "log() is logged");
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js-306-
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js-225- return;
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js-226- // return here as the below bits fail. not sure why. moving on.
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js:227: // TODO: file a bug about this failing test.
toolkit/components/console/hudservice/tests/browser/browser_HUDServiceTestsAll.js-228-
Assignee | ||
Updated•14 years ago
|
Assignee | ||
Comment 1•14 years ago
|
||
(In reply to comment #0)
> There are a few tests that need to be fixed or removed or re-worked after the
> lazy console lands:
>
> toolkit/components/console/hudservice/tests/browser/browser_webconsole_consoleonpage.js-60-
> // is(consoleIFrame, undefined, "Console object was removed fro
> m iFrame");
> toolkit/components/console/hudservice/tests/browser/browser_webconsole_consoleonpage.js:61:
> // TODO: file a followup bug to fix this test
This test is now unneeded as there is "always" a console available. You can remove it, but it will re-attach itself automatically
Assignee | ||
Comment 2•14 years ago
|
||
fixed this one:
toolkit/components/console/hudservice/tests/browser/browser_webconsole_bug_593003_iframe_wrong_hud.js
Very strange, as it was a test infrastructure / timing issue that only reared its head with the new lazy console
Assignee | ||
Comment 3•14 years ago
|
||
toolkit/components/console/hudservice/tests/browser/browser_webconsole_bug_586142_insert_newlines.js
was an easy fix. Looks like I lopped out the bits that added newlines to API logged messages. added it back, tests pass.
Assignee | ||
Comment 4•14 years ago
|
||
In HUDServiceTestsAll.js, the input focus test fails because mochitest changes the focus from what I can gather. I tried to use the focusManager to get the currently focused element and it was null. Have to find a better way to test that.
Updated•14 years ago
|
Blocks: devtools4b8
Assignee | ||
Comment 5•14 years ago
|
||
removed testInputFocus() from browser_HUDServiceTestsAll.js as that test is duplicated in browser_webconsole_bug_588342_document_focus.js
Assignee | ||
Updated•14 years ago
|
Whiteboard: [patch in bug 587734]
Updated•14 years ago
|
Assignee: nobody → ddahl
Assignee | ||
Comment 7•14 years ago
|
||
Fixed by bug 587734 landing
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•