Closed Bug 979039 Opened 10 years ago Closed 9 years ago

Intermittent tests/test-child_process.testChildProperties | Timed out

Categories

(Add-on SDK Graveyard :: General, defect, P1)

All
Windows XP
defect

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: KWierso, Unassigned)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

https://tbpl.mozilla.org/php/getParsedLog.php?id=35558689&tree=Fx-Team
slave: t-xp32-ix-075



TEST-START | tests/test-child_process.testChildEventsSpawningError
TEST-PASS | tests/test-child_process.testChildEventsSpawningError | error was passed into callback
TEST-PASS | tests/test-child_process.testChildEventsSpawningError | stdout is empty
TEST-PASS | tests/test-child_process.testChildEventsSpawningError | stderr is empty
TEST-PASS | tests/test-child_process.testChildEventsSpawningError | error passed into error handler
TEST-PASS | tests/test-child_process.testChildEventsSpawningError | process was never spawned, therefore exit code is -1
TEST-PASS | tests/test-child_process.testChildEventsSpawningError | signal should be null
TEST-END | tests/test-child_process.testChildEventsSpawningError
TEST-START | tests/test-child_process.testChildExecFileKillSignal
TEST-PASS | tests/test-child_process.testChildExecFileKillSignal | correctly used custom killSignal
TEST-END | tests/test-child_process.testChildExecFileKillSignal
TEST-START | tests/test-child_process.testChildProperties
TEST-PASS | tests/test-child_process.testChildProperties | Windows environment does not have `pid`
TEST-INFO | [JavaScript Error: "Error adding value null to histogram PLACES_ANNOS_PAGES_SIZE_KB: Error: Not a number" {file: "resource://gre/modules/PlacesDBUtils.jsm" line: 978}]
TEST-UNEXPECTED-FAIL | tests/test-child_process.testChildProperties | Timed out
TEST-INFO | Traceback (most recent call last):
  File "resource://gre/modules/commonjs/sdk/timers.js", line 40, in notify
    callback.apply(null, args);
  File "resource://gre/modules/commonjs/sdk/deprecated/unit-test.js", line 423, in tiredOfWaiting
    self.console.testMessage(false, false, self.test.name, "Timed out");
  File "resource://gre/modules/commonjs/sdk/test/harness.js", line 527, in testMessage
    this.trace();
TEST-END | tests/test-child_process.testChildProperties
TEST-START | tests/test-child_process.testChildStdinStreamLarge
TEST-PASS | tests/test-child_process.testChildStdinStreamLarge | exited succesfully
TEST-PASS | tests/test-child_process.testChildStdinStreamLarge | no kill signal given
TEST-PASS | tests/test-child_process.testChildStdinStreamLarge | all data processed from stdin
TEST-END | tests/test-child_process.testChildStdinStreamLarge
Priority: -- → P1
Adding some debugging for the promise fetching in child_process tests
Attached file GH PR 1483 —
Attachment #8417623 - Flags: review?(dtownsend+bugmail)
Attachment #8417623 - Attachment is patch: true
Attachment #8417623 - Attachment mime type: text/x-github-pull-request → text/plain
Attachment #8417623 - Attachment is patch: false
Comment on attachment 8417623 [details]
GH PR 1483

Don't know how you attached this but it didn't work right
Attachment #8417623 - Flags: review?(dtownsend+bugmail) → review+
Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/f62c4521a7d4903f267d9d1c028262cb9be86590
Bug 979039 - handle errors in promises for child_process tests

https://github.com/mozilla/addon-sdk/commit/c3bd6bb218bb0be35c9215191911015a11ca673a
Merge pull request #1483 from jsantell/979039-better-debugging-child-proc-tests

Bug 979039 - handle errors in promises for child_process tests, r=@mossop
This is high up on OrangeFactor - could you take another look when you get back? :-)
Flags: needinfo?(jsantell)
Blocks: 1020473
Clearing old needinfo requests where I've given up expecting a response, to clear up my bugzilla-todos dashboard "to nag" list.
Flags: needinfo?(jsantell)
Inactive; closing (see bug 1180138).
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: