810307 - Add-on manager xpcshell test failures

Status: RESOLVED DUPLICATE of bug 810617

(Toolkit :: Add-ons Manager, defect)

Description

[Jim Mathies [:jimm] (OOTO until Oct 7th)]

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] (OOTO until Oct 7th)]

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

Comment 2

[Jim Mathies [:jimm] (OOTO until Oct 7th)]

Attachment: test_backgroundupdate fix

Comment 3

[Jim Mathies [:jimm] (OOTO until Oct 7th)]

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).