950001 - TEST-UNEXPECTED-FAIL | _tests/xpcshell/toolkit/components/jsdownloads/test/unit/test_DownloadIntegration.js | "/tmp/tmpfbNupO" == "/tmp/tmpfbNupO/Downloads"

Status: NEW

(Toolkit :: Downloads API, defect)

Description

[Jan Beich]

Attachment: extend conditional to include BSDs

$ ./mach xpcshell-test toolkit/components/jsdownloads
...
TEST-UNEXPECTED-FAIL | _tests/xpcshell/toolkit/components/jsdownloads/test/unit/test_DownloadIntegration.js | test failed (with xpcshell return code: 0), see following log:
>>>>>>>
TEST-INFO | (xpcshell/head.js) | test MAIN run_test pending (1)
TEST-INFO | (xpcshell/head.js) | test run_next_test 0 pending (2)
TEST-INFO | (xpcshell/head.js) | test MAIN run_test finished (2)
TEST-INFO | (xpcshell/head.js) | running event loop
TEST-INFO | _tests/xpcshell/toolkit/components/jsdownloads/test/unit/test_DownloadIntegration.js | Starting test_common_initialize
TEST-INFO | (xpcshell/head.js) | test test_common_initialize pending (2)
TEST-INFO | (xpcshell/head.js) | test run_next_test 1 pending (3)
TEST-INFO | (xpcshell/head.js) | test test_common_initialize finished (3)
TEST-INFO | (xpcshell/head.js) | test run_next_test 0 finished (2)
TEST-INFO | _tests/xpcshell/toolkit/components/jsdownloads/test/unit/test_DownloadIntegration.js | Starting test_getSystemDownloadsDirectory
TEST-INFO | (xpcshell/head.js) | test test_getSystemDownloadsDirectory pending (2)
TEST-INFO | (xpcshell/head.js) | test run_next_test 1 finished (2)
TEST-UNEXPECTED-FAIL | _tests/xpcshell/toolkit/components/jsdownloads/test/unit/test_DownloadIntegration.js | "/tmp/tmpfbNupO" == "/tmp/tmpfbNupO/Downloads" - See following stack:
JS frame :: _tests/xpcshell/toolkit/components/jsdownloads/test/unit/test_DownloadIntegration.js :: test_getSystemDownloadsDirectory :: line 103
JS frame :: resource://gre/modules/Task.jsm :: TaskImpl_run :: line 233
JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 118
JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: then :: line 43
JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: then :: line 153
JS frame :: resource://gre/modules/Task.jsm :: TaskImpl_handleResultValue :: line 269
JS frame :: resource://gre/modules/Task.jsm :: TaskImpl_run :: line 235
JS frame :: resource://gre/modules/Promise.jsm :: Handler.prototype.process :: line 767
JS frame :: resource://gre/modules/Promise.jsm :: this.PromiseWalker.walkerLoop :: line 531
native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0
TEST-INFO | (xpcshell/head.js) | exiting test
TEST-UNEXPECTED-FAIL | resource://gre/modules/commonjs/sdk/core/promise.js | Unexpected exception 2147500036
TEST-INFO | (xpcshell/head.js) | exiting test
!!! error running onStopped callback: TypeError: callback is not a function
<<<<<<<

Comment 1

[:Paolo Amadini]

Hm, is there a way to match the preprocessor tests in DownloadIntegration.jsm
with JavaScript tests, but without having to write a list of values for the
appinfo.OS property? Maybe using a different property or test?

Comment 2

[:Paolo Amadini]

Comment on attachment 8347179 [details] [diff] [review]
extend conditional to include BSDs

Given there has been no answer to comment 1 and I didn't find a better test either, I think checking a few more system names is fine to fix this report until we find a better solution.