Closed Bug 971835 Opened 12 years ago Closed 12 years ago

Intermittent places/main.test bookmark-item-moved | correct currentIndex - 0 == 4

Categories

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

Product:
Add-on SDK Graveyard
Component:
General
Platform:
x86_64
macOS
Type:
defect

Tracking

(firefox29 unaffected, firefox30 fixed, firefox31 fixed, firefox-esr24 unaffected)

Status:
RESOLVED FIXED
Milestone:
mozilla31
Tracking Flags:
Tracking Status
firefox29 --- unaffected
firefox30 --- fixed
firefox31 --- fixed
firefox-esr24 --- unaffected

People

(Reporter: RyanVM, Assigned: jsantell)

Details

(Keywords: intermittent-failure)

https://tbpl.mozilla.org/php/getParsedLog.php?id=34565585&tree=Fx-Team Rev4 MacOSX Snow Leopard 10.6 fx-team debug test jetpack on 2014-02-12 09:02:21 PST for push e30953666622 slave: talos-r4-snow-059 TEST-START | places/main.test bookmark-item-moved TEST-PASS | places/main.test bookmark-item-moved | correct type in bookmark-item-moved event TEST-PASS | places/main.test bookmark-item-moved | correct data in bookmark-item-moved event TEST-PASS | places/main.test bookmark-item-moved | correct id in bookmark-item-moved event TEST-PASS | places/main.test bookmark-item-moved | correct previousParentId TEST-PASS | places/main.test bookmark-item-moved | correct currentParentId TEST-PASS | places/main.test bookmark-item-moved | correct previousIndex TEST-UNEXPECTED-FAIL | places/main.test bookmark-item-moved | correct currentIndex - 0 == 4 TEST-INFO | Traceback (most recent call last): File "resource://gre/modules/commonjs/sdk/timers.js", line 70, in dispatcher try { immediate(); } File "resource://gre/modules/commonjs/sdk/timers.js", line 79, in setImmediate/< immediates.set(id, _ => callback.apply(callback, params)); File "resource://gre/modules/commonjs/sdk/lang/functional.js", line 68, in invoke const invoke = (callee, params, self) => callee.apply(self, params); File "resource://gre/modules/commonjs/sdk/event/core.js", line 96, in emit listener.apply(target, args); File "resource://gre/modules/commonjs/sdk/event/utils.js", line 43, in transform/< on(input, "data", data => f(data, next)); File "resource://gre/modules/commonjs/sdk/event/utils.js", line 53, in filter/< next(data); File "resource://gre/modules/commonjs/sdk/event/utils.js", line 36, in transform/next const next = data => receive(output, data); File "resource://gre/modules/commonjs/sdk/event/utils.js", line 115, in receive emit(input, "data", message); File "resource://gre/modules/commonjs/sdk/event/core.js", line 96, in emit listener.apply(target, args); File "resource://gre/modules/commonjs/sdk/places/host/host-bookmarks.js", line 206, in null promised(EVENT_MAP[event])(data).then(res => { File "resource://gre/modules/commonjs/sdk/core/promise.js", line 281, in promised then(execute); File "resource://gre/modules/commonjs/sdk/core/promise.js", line 153, in then result.then(resolve, reject); File "resource://gre/modules/commonjs/sdk/core/promise.js", line 153, in then result.then(resolve, reject); File "resource://gre/modules/commonjs/sdk/core/promise.js", line 43, in then return { then: function then(fulfill) { fulfill(value); } }; File "resource://gre/modules/commonjs/sdk/core/promise.js", line 118, in resolve deferred.resolve(onFulfill ? onFulfill(value) : value); File "resource://gre/modules/commonjs/sdk/core/promise.js", line 118, in resolve deferred.resolve(onFulfill ? onFulfill(value) : value); File "resource://gre/modules/commonjs/sdk/core/promise.js", line 250, in execute function execute(args) { return call.apply(call, args) } File "resource://gre/modules/commonjs/sdk/places/host/host-bookmarks.js", line 125, in saveBookmarkItem bmsrv.moveItem(id, data.group, data.index || -1); File "resource://gre/modules/commonjs/sdk/places/events.js", line 86, in createHandler/< data: data File "resource://gre/modules/commonjs/sdk/event/core.js", line 96, in emit listener.apply(target, args); File "resource://test-places-at-jetpack/places/tests/test-places-events.js", line 94, in handler assert.equal(data.currentIndex, 0, 'correct currentIndex'); File "resource://gre/modules/commonjs/sdk/test/assert.js", line 124, in equal operator: "==" File "resource://gre/modules/commonjs/sdk/test/assert.js", line 88, in fail this._log.fail(message); File "resource://gre/modules/commonjs/sdk/deprecated/unit-test.js", line 83, in fail this.console.testMessage(false, false, this.test.name, message); File "resource://gre/modules/commonjs/sdk/test/harness.js", line 527, in testMessage this.trace(); TEST-END | places/main.test bookmark-item-moved
This is actually a real bustage it appears. I'm suspecting bug 941196, but the OSX debug build on that push was coalesced out :( I've got a new build going on that push now, so we'll see.
Priority: -- → P1
Coming up on a month now. Should we just turn off the Jetpack tests on OSX 10.6 instead of wasting test slave resources on a perma-fail job?
Flags: needinfo?(dtownsend+bugmail)
I thought you'd done that already Wes? I'm fine with it
Flags: needinfo?(dtownsend+bugmail) → needinfo?(kwierso)
We hid the suite from the default view on TBPL, but it's still perma-failing and wasting resources in the background.
Ah I see. Jordan where are we at on investigating this?
Flags: needinfo?(jsantell)
This is weird. Looking into it
Flags: needinfo?(jsantell)
Assignee: nobody → jsantell
potential fix in bug 971964
This looks like it was fixed from the previous patch in bug 971964, but its new patch is disabling these in OSX 10.6 altogether, so, closing this one
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
status-firefox29: --- → unaffected
status-firefox30: --- → fixed
status-firefox31: --- → fixed
status-firefox-esr24: --- → unaffected
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.