Closed Bug 979039 Opened 11 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 (deleted) —
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: