Closed Bug 910230 Opened 12 years ago Closed 11 years ago

Intermittent tests/test-panel.test Parent Resize Hack | Test output exceeded timeout (300s).

Categories

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

Product:
Add-on SDK Graveyard
Component:
General
Platform:
All
Windows 8
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: KWierso, Assigned: zer0)

References

Details

(Keywords: intermittent-failure, Whiteboard: [leave open])

Attachments

(1 file)

Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1230
358 bytes, text/html
mossop
: review+
Details
TEST-START | tests/test-panel.test Parent Resize Hack console.error: addon-sdk: Message: TypeError: target.URL is undefined Stack: exports["test multiple tabs"]/<@resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://extensions.modules.9be77f16-617c-4736-8b4e-c1402a8c01c1-at-jetpack.commonjs.path.tests/test-content-events.js:51 lazy@resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:110 emit@resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:83 @resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/utils.js:82 lazy@resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:110 emit@resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:83 @resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/chrome.js:36 emit@resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/system/events.js:38 onContentChange@resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/panel/utils.js:260 @resource://gre/modules/XPIProvider.jsm -> jar:file:///c:/users/cltbld~1.t-w/appdata/local/temp/tmphl_azf.mozrunner/extensions/9be77f16-617c-4736-8b4e-c1402a8c01c1@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/system/events.js:62 TEST-PASS | tests/test-panel.test Parent Resize Hack | Size doesn't change by calling resizeTo/By/... TEST-PASS | tests/test-panel.test Parent Resize Hack | Size doesn't change by calling resizeTo/By/... TEST-UNEXPECTED-FAIL | tests/test-panel.test Parent Resize Hack | Test output exceeded timeout (300s). No handlers could be found for logger "mozrunner" Traceback (most recent call last): File "jetpack/bin/cfx", line 33, in <module> cuddlefish.run() File "C:\slave\test\build\jetpack\python-lib\cuddlefish\__init__.py", line 612, in run test_all_packages(env_root, defaults=options.__dict__) File "C:\slave\test\build\jetpack\python-lib\cuddlefish\__init__.py", line 494, in test_all_packages env_root=env_root) File "C:\slave\test\build\jetpack\python-lib\cuddlefish\__init__.py", line 945, in run pkgdir=options.pkgdir) File "C:\slave\test\build\jetpack\python-lib\cuddlefish\runner.py", line 742, in run_app OUTPUT_TIMEOUT, test_name, parseable) Exception: Test output exceeded timeout (300s). program finished with exit code 1
Note that this showed up prior to the uplift to fx-team today.
Summary: Intermittent TEST-UNEXPECTED-FAIL | tests/test-panel.test Parent Resize Hack | Test output exceeded timeout (300s). → Intermittent tests/test-panel.test Parent Resize Hack | Test output exceeded timeout (300s).
Matteo, you last touched this test in 2012. Any ideas?
Flags: needinfo?(zer0)
Alex, you're also in the blame for that test. Same question to you.
Flags: needinfo?(poirot.alex)
Commit pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/f243501d68990533b5ec61b20c384294563448bd Merge pull request #1224 from Mossop/bug910230 Add debugging information when panel.destroy throws to help figure out bug 910230 r=kwierso
(In reply to TBPL Robot from comment #68) > KWierso > https://tbpl.mozilla.org/php/getParsedLog.php?id=27217644&tree=Jetpack > jetpack-fx-team-win7-ix-opt on 2013-08-30 03:55:09 > revision: 40868a4d9a76 > slave: t-w732-ix-083 > > TEST-UNEXPECTED-FAIL | tests/test-panel.test Parent Resize Hack | Test > output exceeded timeout (300s). Output was identical to the previous failures, so the debugging information wasn't triggered.
Attachment #799376 - Flags: review?(dtownsend+bugmail)
I don't have any clue about this behavior. However, I made some changes in order to remove the dependencies from timeout, and some other minor changes. It should helps, maybe we destroy the panel too soon or in a weird timing condition.
Flags: needinfo?(zer0)
Attachment #799376 - Flags: review?(dtownsend+bugmail) → review+
Priority: -- → P1
Commits pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/293f3c26b5c06c3632d8b6c5026cb90866ef28fb Bug 910230 - Intermittent tests/test-panel.test Parent Resize Hack | Test output exceeded timeout (300s) - Removed dependency from timeout - Removed code relative to bug 635673 https://github.com/mozilla/addon-sdk/commit/b85ca7dba40fcc0f92e42592e972cde1525a520f Merge pull request #1230 from ZER0/panel-resize-test/910230 Bug 910230 - Intermittent tests/test-panel.test Parent Resize Hack | Test output exceeded timeout (300s) r=@Mossop
Wes, please can you disable this test for now, it's been too long without any progress - cheers :-)
Flags: needinfo?(kwierso)
Flags: needinfo?(kwierso)
Whiteboard: [leave open]
Sorry for getting back to the needinfo so late... app manager took me at 101%! So I'm not sure this issue is related to test-panel. We shouldn't be seeing code from test-content-events being executed while we run test-panel!! But then I don't know why test-content-events's `handler` closure isn't correctly unregistered. May be because we unload its loader before calling off() here: https://github.com/mozilla/addon-sdk/blob/master/test/test-content-events.js#L83 loader.unload(); off(events, "data", handler); It might be worth flipping these two lines, reenabling this test and see if this still happen to fail on TBPL...
Flags: needinfo?(poirot.alex)
Assignee: nobody → zer0
It doesn't seems happen since a while, shall we close this bug Dave? Maybe it's worthy land the suggestion of Alex anyway?
Flags: needinfo?(dtownsend+bugmail)
(In reply to Matteo Ferretti [:matteo] [:zer0] from comment #187) > It doesn't seems happen since a while, shall we close this bug Dave? > Maybe it's worthy land the suggestion of Alex anyway? It hasn't happened in a while because the test got disabled 6 months ago. If Alex's suggestion looks reasonable then let's go for it.
Flags: needinfo?(dtownsend+bugmail)
we haven't seen any test failures here in > 3 months
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: