Closed Bug 1448831 Opened 3 years ago Closed 3 years ago

TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_db_path.js and more (Windows only)


(Thunderbird :: Testing Infrastructure, defect)

Not set


(Not tracked)

Thunderbird 61.0


(Reporter: jorgk-bmo, Unassigned)


(Whiteboard: [Thunderbird-testfailure: X Windows only])


(2 files)

TEST-UNEXPECTED-FAIL | toolkit/components/telemetry/tests/unit/test_TelemetryEnvironment.js | xpcshell return code: 0
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_db_path.js | xpcshell return code: 0
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_update_webextensions.js | xpcshell return code: 0

Perma-red since Wed, Mar 21, 2018, 11:54:05

M-C last good: 6d0f84afc194e287b6f144d17cd88371a5
M-C first bad: e636edf00e6fbdc3206c9df4a1548ae38b
  mozilla/mach xpcshell-test toolkit/mozapps/extensions/test/xpcshell/test_db_path.js
locally gives a good selection of errors :-(

 0:02.28 INFO xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_db_path.js | Starting test_non_ascii_path
 0:02.28 INFO (xpcshell/head.js) | test test_non_ascii_path pending (2)
 0:02.69 pid:8608 JavaScript strict warning: resource://testing-common/AddonTestUtils.jsm, line 228: ReferenceError: reference to undefined property "testScope"
 0:02.30 INFO "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "testScope"" {file: "resource://testing-common/AddonTestUtils.jsm" line: 228}]"
 0:02.69 pid:8608 [8608, LoadRoots] WARNING: This method is lossy. Use GetCanonicalPath !: file c:/mozilla-source/comm-central/mozilla/xpcom/io/nsLocalFileWin.cpp, line 3378
 0:02.69 pid:8608 Assertion failure: IsIdle(oldState) || IsRead(oldState), at c:\\mozilla-source\\comm-central\\mozilla\\xpcom\\ds\\PLDHashTable.h:118
 0:02.69 pid:8608 Assertion failure: IsWrite(oldState), at c:\\mozilla-source\\comm-central\\mozilla\\xpcom\\ds\\PLDHashTable.h:142
 0:02.69 pid:8608 ### ERROR: SymInitialize: The parameter is incorrect.
Looks like a consequence of bug 1446571, especially the "testScope" added here:
Other recent changes:

Kris, any idea why this is only failing in the TB Xpcshell test suite and only on Windows?
Flags: needinfo?(kmaglione+bmo)
(In reply to Jorg K (GMT+1) from comment #2)
> Looks like a consequence of bug 1446571, especially the "testScope" added
> here:

The testScope warning is a red herring. It's more or less expected.

> Kris, any idea why this is only failing in the TB Xpcshell test suite and
> only on Windows?

Nope. I'd have to see the full test output with a complete stack. That assertion generally means that something's trying to change a hash table while it's iterating over it. Bug 1446571 seems like a pretty unlikely candidate. Possibly bug 1446833.
Flags: needinfo?(kmaglione+bmo)
Here's the log file after rebuilding at today's tip on M-C.

"Assertion failure: IsWrite(oldState)" is no longer there, but the test still fails, in fact, right at the very start at line 7 of the log. I can't see what fails since the four Assert.ok() in the test itself pass.
Flags: needinfo?(kmaglione+bmo)
Running test_update_webextensions.js gives a similar result, all the 27 subtests I can see pass, but there is some failure right at the start at line 7.
Ah, this is a file locking issue:

 0:02.99 ERROR Error: Found unexpected files in temporary directory: generated-extension-1.xpi at resource://testing-common/AddonTestUtils.jsm:342

 0:02.99 INFO "CONSOLE_MESSAGE: (error) [JavaScript Error: "[Exception... "Component returned failure code: 0x80520015 (NS_ERROR_FILE_ACCESS_DENIED) [nsIFile.remove]"  nsresult: "0x80520015 (NS_ERROR_FILE_ACCESS_DENIED)"  location: "JS frame :: resource://testing-common/AddonTestUtils.jsm :: cleanupTempXPIs :: line 471"  data: no]"]

Not sure why it only fails on Thunderbird. It might be fixed by the follow-ups to bug 1447903, since they flush the jar cache at add-on manager shutdown.
Flags: needinfo?(kmaglione+bmo)
As far as I can see, this last failed at the beginning of May 2018 here:

I never checked any more since the Xpcshell tests were perma-red on Windows. Looking closer, this failure here disappeared and was "replaced" by TEST-UNEXPECTED-FAIL | uriloader/exthandler/tests/unit/test_handlerService.js. That has also cleared, so Xpcshell on Windows is mostly green now.

So let's mark this fixed.
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 61.0
You need to log in before you can comment on or make changes to this bug.