Closed Bug 600095 Opened 14 years ago Closed 14 years ago

Followup test fixes after lazy console lands

Categories

(DevTools :: General, defect, P3)

defect

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-
Blocks: 529086
Depends on: 587734
(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
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
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.
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.
Blocks: devtools4b8
removed testInputFocus() from browser_HUDServiceTestsAll.js as that test is duplicated in browser_webconsole_bug_588342_document_focus.js
Whiteboard: [patch in bug 587734]
Assignee: nobody → ddahl
mass change: filter on PRIORITYSETTING
Blocks: devtools4
Priority: -- → P3
Fixed by bug 587734 landing
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.