corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1956080 - Use async experiment cleanup in test_backgroundtask_experiments.js r=nalexander

Browse Source 
ExperimentManager.unenroll() is now effectively async when running in a
browser test (because we are now executing SQL queries during
unenrollment) so all cleanup functions that trigger unenrollment are
async as well.

Differential Revision: https://phabricator.services.mozilla.com/D250571
This commit is contained in:
Beth Rennie
2025-05-22 23:21:34 +00:00
committed by brennie@mozilla.com
parent 2ce17bec1f
commit 4560d3bad6
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
toolkit/components/backgroundtasks/tests/xpcshell/test_backgroundtask_experiments.js
View File

@@ -75,7 +75,7 @@ add_setup(async () => {
}),
"test"
);
manager.unenroll("foo");
await manager.unenroll("foo");
await manager.enroll(
NimbusTestUtils.factories.recipe.withFeatureConfig("bar", {
branchSlug: "treatment",
@@ -83,7 +83,7 @@ add_setup(async () => {
}),
"test"
);
manager.unenroll("bar");
await manager.unenroll("bar");
await manager.enroll(
NimbusTestUtils.factories.recipe.withFeatureConfig("baz", {
branchSlug: "treatment",
@@ -96,7 +96,7 @@ add_setup(async () => {
NimbusTestUtils.factories.recipe("rol1", { isRollout: true }),
"test"
);
manager.unenroll("rol1");
await manager.unenroll("rol1");
await manager.enroll(
NimbusTestUtils.factories.recipe("rol2", { isRollout: true }),
"test"
@@ -104,9 +104,9 @@ add_setup(async () => {
});
registerCleanupFunction(async () => {
manager.unenroll("baz");
manager.unenroll("rol2");
cleanup();
await manager.unenroll("baz");
await manager.unenroll("rol2");
await cleanup();
});
function resetProfile(profile) {