810307 - Add-on manager xpcshell test failures

Status: RESOLVED DUPLICATE of bug 810617

(Toolkit :: Add-ons Manager, defect)

Description

[Jim Mathies [:jimm]]

Various failures in the add-on manager. 


TEST-UNEXPECTED-FAIL | toolkit\mozapps\extensions\test\xpcshell\test_dss.js | test failed (with xpcshell return code: 0), see following log:
TEST-UNEXPECTED-FAIL | toolkit/mozapps/extensions/test/xpcshell/head_addons.js | true == false - See following stack:

^ light weight theme manager issue with _setCurrentTheme incorrectly requiring a restart after installing a new theme.

TEST-UNEXPECTED-FAIL | toolkit/mozapps/extensions/test/xpcshell/test_startup.js | false == true - See following stack:

^ gCachePurged doesn't get set during test6.

TEST-UNEXPECTED-FAIL | toolkit\mozapps\extensions\test\xpcshell\test_theme.js | test failed (with xpcshell return code: 0), see following log:

^ light weight theme manager issue with _setCurrentTheme.

https://tbpl.mozilla.org/php/getParsedLog.php?id=16875103&tree=Elm&full=1#error15

Comment 1

[Jim Mathies [:jimm]]

Also, test_backgroundupdate.js, which is currently disabled on elm to get complete test runs, hangs.

Comment 2

[Jim Mathies [:jimm]]

Attachment: test_backgroundupdate fix

Comment 3

[Jim Mathies [:jimm]]

TEST-UNEXPECTED-FAIL | toolkit/mozapps/extensions/test/xpcshell/test_startup.js | false == true - See following stack:

^ gCachePurged doesn't get set during test6.

This failure is caused by an assumption in the test about the presence of the default theme extension. Should be pretty easy to work around.

Comment 4

[Blair McBride [:Unfocused] (UNAVAILABLE)]

Comment on attachment 680114 [details] [diff] [review]
test_backgroundupdate fix

Review of attachment 680114 [details] [diff] [review]:
-----------------------------------------------------------------

Er, hmm. If that fixes it, it kinda makes it look like there's a real bug there (not just the test needing a fix).