Skip to content
Snippets Groups Projects
Commit 2d303eeb authored by Barret Rennie's avatar Barret Rennie
Browse files

Bug 1889157 - Dont use Promise.withResolvers in browser_LegacyHeartbeat.js r=chumphreys a=RyanVM

Bug 1884039 introduced a new test that used Promise.withResolvers, but this API
is not present in ESR115.

Differential Revision: https://phabricator.services.mozilla.com/D206391
parent 4d356bb4
No related branches found
No related tags found
No related merge requests found
......@@ -181,18 +181,21 @@ decorate_task(
sandbox.stub(client, "get").resolves([]);
// Override Heartbeat so we can get the instance and manipulate it directly.
const heartbeatDeferred = Promise.withResolvers();
let resolveHeartbeatPromise;
const heartbeatPromise = new Promise(resolve => {
resolveHeartbeatPromise = resolve;
});
class TestHeartbeat extends Heartbeat {
constructor(...args) {
super(...args);
heartbeatDeferred.resolve(this);
resolveHeartbeatPromise(this);
}
}
ShowHeartbeatAction.overrideHeartbeatForTests(TestHeartbeat);
try {
await RecipeRunner.run();
const heartbeat = await heartbeatDeferred.promise;
const heartbeat = await heartbeatPromise;
// We are going to simulate the timer timing out, so we do not want it to
// *actually* time out.
heartbeat.endTimerIfPresent("surveyEndTimer");
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment