599193 - Test failure in test_utils_file.js: NS_ERROR_FAILURE [nsIFile.remove]

Status: RESOLVED FIXED

(Firefox :: Sync, defect)

Description

[Philipp von Weitershausen [:philikon]]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1285282431.1285284398.17722.gz&fulltext=1
WINNT 5.2 mozilla-central debug test xpcshell on 2010/09/23 15:53:51
s: win32-slave36

TEST-UNEXPECTED-FAIL | e:\builds\moz2_slave\mozilla-central-win32-debug-unittest-xpcshell\build\xpcshell\tests\services\sync\tests\unit\test_utils_file.js | test failed (with xpcshell return code: 0), see following log:
  >>>>>>>
  ### XPCOM_MEM_LEAK_LOG defined -- logging leaks to c:\docume~1\cltbld\locals~1\temp\tmph7_uiz\runxpcshelltests_leaks.log
pldhash: for the table at address 063210C0, the given entrySize of 48 probably favors chaining over double hashing.
nsNativeModuleLoader::LoadModule("e:\builds\moz2_slave\mozilla-central-win32-debug-unittest-xpcshell\build\firefox\components\xpcomsample.dll") - load FAILED, rv: 80520012, error:
	<unknown; can't get error from NSPR>
WARNING: This method is lossy. Use GetCanonicalPath !: file e:/builds/moz2_slave/mozilla-central-win32-debug/build/xpcom/io/nsLocalFileWin.cpp, line 2977
pldhash: for the table at address 07149A98, the given entrySize of 52 probably favors chaining over double hashing.
Setting the identity for passphrase
Test file-related utility functions
TEST-INFO | (xpcshell/head.js) | test 1 pending
getTmp with no argument
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 27] true == true
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 28] true == true
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 29] true == true
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 30] tmp == tmp
getTmp with a string
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 37] true == true
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 38] true == true
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 39] name == name
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 40] true == true
Open for reading, providing a file
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_open : 63] c:\docume~1\cltbld\locals~1\temp\tmph7_uiz\weave\tmp\_test_ == c:\docume~1\cltbld\locals~1\temp\tmph7_uiz\weave\tmp\_test_
TEST-PASS | e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_open : 64] true == true
TEST-UNEXPECTED-FAIL | (xpcshell/head.js) | [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIFile.remove]"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js :: _test_open :: line 65"  data: no]
WARNING: nsExceptionService ignoring thread destruction after shutdown: file e:/builds/moz2_slave/mozilla-central-win32-debug/build/xpcom/base/nsExceptionService.cpp, line 197
nsXPConnect::CommenceShutdown()
WARNING: OOPDeinit() without successful OOPInit(): file e:/builds/moz2_slave/mozilla-central-win32-debug/build/toolkit/crashreporter/nsExceptionHandler.cpp, line 1582
nsStringStats
 => mAllocCount:           2030
 => mReallocCount:           84
 => mFreeCount:            2030
 => mShareCount:           7101
 => mAdoptCount:            153
 => mAdoptFreeCount:        153

  <<<<<<<

Comment 1

[Philipp von Weitershausen [:philikon]]

Disabled tests for now: http://hg.mozilla.org/mozilla-central/rev/59b25e24dc5e

sdwilsh says we need to do something like this to make Windows happy: http://mxr.mozilla.org/mozilla-central/source/storage/test/unit/head_storage.js#71

Comment 2

[Philipp von Weitershausen [:philikon]]

Attachment: v1

Pushed to try just now to see if this fixes the orange.

Comment 3

[Jesse Ruderman]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1285288592.1285292838.22516.gz&fulltext=1

Comment 4

[Legacy TBPL/Treeherder Robot]

jruderman%gmail.com
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1285291054.1285293124.23903.gz
WINNT 5.2 mozilla-central debug test xpcshell on 2010/09/23 18:17:34

s: win32-slave30
TEST-UNEXPECTED-FAIL | e:\builds\moz2_slave\mozilla-central-win32-debug-unittest-xpcshell\build\xpcshell\tests\services\sync\tests\unit\test_utils_file.js | test failed (with xpcshell return code: 0), see following log:
TEST-UNEXPECTED-FAIL | (xpcshell/head.js) | [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIFile.remove]"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: e:/builds/moz2_slave/mozilla-central-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js :: _test_open :: line 65"  data: no]

Comment 5

[Philipp von Weitershausen [:philikon]]

Still getting a failure with patch v1:

TEST-UNEXPECTED-FAIL | e:\builds\moz2_slave\tryserver-win32-debug-unittest-xpcshell\build\xpcshell\tests\services\sync\tests\unit\test_utils_file.js | test failed (with xpcshell return code: 0), see following log:  >>>>>>>  ### XPCOM_MEM_LEAK_LOG defined -- logging leaks to c:\docume~1\cltbld\locals~1\temp\tmplgufsq\runxpcshelltests_leaks.log
pldhash: for the table at address 06541420, the given entrySize of 48 probably favors chaining over double hashing.nsNativeModuleLoader::LoadModule("e:\builds\moz2_slave\tryserver-win32-debug-unittest-xpcshell\build\firefox\components\xpc
omsample.dll") - load FAILED, rv: 80520012, error:
        <unknown; can't get error from NSPR>WARNING: This method is lossy. Use GetCanonicalPath !: file e:/builds/moz2_slave/tryserver-win32-debug/build/xpcom/io/nsLoc
alFileWin.cpp, line 2977
pldhash: for the table at address 07577210, the given entrySize of 52 probably favors chaining over double hashing.
Setting the identity for passphrase
Test file-related utility functions
TEST-INFO | (xpcshell/head.js) | test 1 pending
getTmp with no argument
TEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test
_utils_file.js | [_test_getTmp : 27] true == true
TEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test
_utils_file.js | [_test_getTmp : 28] true == trueTEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test
_utils_file.js | [_test_getTmp : 29] true == trueTEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test
_utils_file.js | [_test_getTmp : 30] tmp == tmpgetTmp with a string
TEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 37] true == true
TEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 38] true == true
TEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 39] name == name
TEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_getTmp : 40] true == true
Open for reading, providing a fileTEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test
_utils_file.js | [_test_open : 63] c:\docume~1\cltbld\locals~1\temp\tmplgufsq\weave\tmp\_test_ == c:\docume~1\cltbld\locals~1\temp\tmplgufsq\weave\tmp\_test_
TEST-PASS | e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js | [_test_open : 64] true == true
Open for reading, providing a file nameTEST-UNEXPECTED-FAIL | (xpcshell/head.js) | [Exception... "Component returned failure code: 0x80520008 (NS_ERROR_FILE_ALREA
DY_EXISTS) [nsIFile.create]"  nsresult: "0x80520008 (NS_ERROR_FILE_ALREADY_EXISTS)"  location: "JS frame :: e:/builds/moz2_slave/tryserver-win32-debug-unittest-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js :: createFil
e :: line 50"  data: no]WARNING: nsExceptionService ignoring thread destruction after shutdown: file e:/builds/moz2_slave/tryserver-win32-debug/bui
ld/xpcom/base/nsExceptionService.cpp, line 197nsXPConnect::CommenceShutdown()
WARNING: OOPDeinit() without successful OOPInit(): file e:/builds/moz2_slave/tryserver-win32-debug/build/toolkit/crashreporter/nsExceptionHandler.cpp, line 1582
nsStringStats
 => mAllocCount:           2033 => mReallocCount:           75
 => mFreeCount:            2033 => mShareCount:           7105
 => mAdoptCount:            153
 => mAdoptFreeCount:        153

  <<<<<<<

See http://stage.mozilla.org/pub/mozilla.org/firefox/tryserver-builds/[e-mail]-8112acb2683f/tryserver-win32-debug/tryserver-win32-debug-unittest-xpcshell-build213.txt.gz

Not sure what the first TEST-UNEXPECTED-FAIL is about, for the second one we actually should change the test to choose a random file name to avoid clashes if the temp file exists.

Comment 6

[Legacy TBPL/Treeherder Robot]

rstrong%mozilla.com
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1285292852.1285297180.9777.gz
Rev3 WINNT 6.1 mozilla-central opt test xpcshell on 2010/09/23 18:47:32

s: talos-r3-w7-046
TEST-UNEXPECTED-FAIL | c:\talos-slave\mozilla-central_win7_test-xpcshell\build\xpcshell\tests\services\sync\tests\unit\test_utils_file.js | test failed (with xpcshell return code: 0), see following log:
TEST-UNEXPECTED-FAIL | (xpcshell/head.js) | [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIFile.remove]"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: c:/talos-slave/mozilla-central_win7_test-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js :: _test_open :: line 65"  data: no]

Comment 7

[Legacy TBPL/Treeherder Robot]

ehsan%mozilla.com
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1285296949.1285301331.27828.gz
Rev3 WINNT 6.1 mozilla-central opt test xpcshell on 2010/09/23 19:55:49

s: talos-r3-w7-021
TEST-UNEXPECTED-FAIL | c:\talos-slave\mozilla-central_win7_test-xpcshell\build\xpcshell\tests\services\sync\tests\unit\test_utils_file.js | test failed (with xpcshell return code: 0), see following log:
TEST-UNEXPECTED-FAIL | (xpcshell/head.js) | [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIFile.remove]"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: c:/talos-slave/mozilla-central_win7_test-xpcshell/build/xpcshell/tests/services/sync/tests/unit/test_utils_file.js :: _test_open :: line 65"  data: no]

Comment 8

[Mike Connor [:mconnor]]

Comment on attachment 478128 [details] [diff] [review]
v1

Maybe it's worth this being a helper instead of having to duplicate the try/catch + comment everywhere?

Comment 9

[Philipp von Weitershausen [:philikon]]

(In reply to comment #8)
> Maybe it's worth this being a helper instead of having to duplicate the
> try/catch + comment everywhere?

Yeah, I was aiming for a quick fix here (search + replace). As I noted in comment 5, there are still some failures even with patch v1, so I'll have to revisit this anyway.

Comment 10

[Philipp von Weitershausen [:philikon]]

This test was removed as part of bug 648338.