Bug 1207702 - Fix a bug in test_promises_object_timetosettle-02.js;r=fitzgen
--- a/devtools/server/tests/unit/test_promises_object_timetosettle-02.js
+++ b/devtools/server/tests/unit/test_promises_object_timetosettle-02.js
@@ -10,25 +10,27 @@
const { PromisesFront } = require("devtools/server/actors/promises");
const { setTimeout } = require("sdk/timers");
var events = require("sdk/event/core");
add_task(function*() {
let client = yield startTestDebuggerServer("test-promises-timetosettle");
let chromeActors = yield getChromeActors(client);
+ yield attachTab(client, chromeActors);
ok(Promise.toString().contains("native code"), "Expect native DOM Promise.");
yield testGetTimeToSettle(client, chromeActors,
v => new Promise(resolve => setTimeout(() => resolve(v), 100)));
let response = yield listTabs(client);
let targetTab = findTab(response.tabs, "test-promises-timetosettle");
ok(targetTab, "Found our target tab.");
+ yield attachTab(client, targetTab);
yield testGetTimeToSettle(client, targetTab, v => {
const debuggee =
DebuggerServer.getTestGlobal("test-promises-timetosettle");
return new debuggee.Promise(resolve => setTimeout(() => resolve(v), 100));
});
yield close(client);
@@ -42,19 +44,20 @@ function* testGetTimeToSettle(client, fo
yield front.attach();
yield front.listPromises();
let onNewPromise = new Promise(resolve => {
events.on(front, "promises-settled", promises => {
for (let p of promises) {
if (p.promiseState.state === "fulfilled" &&
p.promiseState.value === resolution) {
- equal(Math.floor(p.promiseState.timeToSettle / 100) * 100, 100,
+ let timeToSettle = Math.floor(p.promiseState.timeToSettle / 100) * 100;
+ ok(timeToSettle >= 100,
"Expect time to settle for resolved promise to be " +
- "approximately 100ms.");
+ "at least 100ms, got " + timeToSettle + "ms.");
found = true;
resolve();
} else {
dump("Found non-target promise.\n");
}
}
});
});