Closed Bug 972224 Opened 10 years ago Closed 6 years ago

Intermittent places/main.testSearchSort | latest visited should be first - "http://github.com/" == "http://mozilla.com/", latest visited should be at the end - "http://github.com/" == "http://mozilla.com/"

Categories

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

Product:
Add-on SDK Graveyard
Component:
General
Platform:
x86
Windows XP
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED INCOMPLETE

People

(Reporter: philor, Unassigned)

References

Details

(Keywords: intermittent-failure) 

https://tbpl.mozilla.org/php/getParsedLog.php?id=34578833&tree=Fx-Team
Windows XP 32-bit fx-team opt test jetpack on 2014-02-12 13:36:05 PST for push 96accde0e1dc
slave: t-xp32-ix-060

TEST-UNEXPECTED-FAIL | places/main.testSearchSort | latest visited should be first - "http://github.com/" == "http://mozilla.com/"
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/addon/events.js", line 27, in receive<
    else request.resolve(clone(data));
  File "resource://gre/modules/commonjs/sdk/core/promise.js", line 185, in resolve
    result.then(observer.resolve, observer.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/places/history.js", line 55, in query/</<
    emit(emitter, 'end', results);
  File "resource://gre/modules/commonjs/sdk/event/core.js", line 96, in emit
    listener.apply(target, args);
  File "resource://gre/modules/commonjs/sdk/places/utils.js", line 165, in promisedEmitter/<
    else resolve(items);
  File "resource://gre/modules/commonjs/sdk/core/promise.js", line 185, in resolve
    result.then(observer.resolve, observer.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://test-places-at-jetpack/places/tests/test-places-history.js", line 203, in exports.testSearchSort/<
    'latest visited should be first');
  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-UNEXPECTED-FAIL | places/main.testSearchSort | latest visited should be at the end - "http://github.com/" == "http://mozilla.com/"
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/addon/events.js", line 27, in receive<
    else request.resolve(clone(data));
  File "resource://gre/modules/commonjs/sdk/core/promise.js", line 185, in resolve
    result.then(observer.resolve, observer.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/places/history.js", line 55, in query/</<
    emit(emitter, 'end', results);
  File "resource://gre/modules/commonjs/sdk/event/core.js", line 96, in emit
    listener.apply(target, args);
  File "resource://gre/modules/commonjs/sdk/places/utils.js", line 165, in promisedEmitter/<
    else resolve(items);
  File "resource://gre/modules/commonjs/sdk/core/promise.js", line 185, in resolve
    result.then(observer.resolve, observer.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://test-places-at-jetpack/places/tests/test-places-history.js", line 207, in exports.testSearchSort/<
    'latest visited should be at the end');
  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.testSearchSort
Priority: -- → P1
It's been awhile since this has failed, should we keep it open?
Flags: needinfo?(dtownsend+bugmail)
I'd say anything over a month can be safely closed. We can always reopen
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(dtownsend+bugmail)
Resolution: --- → WORKSFORME
This is still happening https://tbpl.mozilla.org/php/getParsedLog.php?id=43529830&tree=Mozilla-Central&full=1
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
We haven't seen this test fail for many months now, closing.
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → WONTFIX
WORKSFORME would have been more appropriate than WONTFIX for that.

In any case, it's not true anymore.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Add-on SDK is no longer supported so resolving bugs as INCOMPLETE
Status: REOPENED → RESOLVED
Closed: 10 years ago → 6 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.