Merge pull request #1116 from adobe/fixHandleConsentFlickerTest

Fix test that wasn't passing in Safari
adobe · May 17, 2024 · beac464 · beac464
2 parents 119ad1d + 076d9f6
commit beac464
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/test/unit/specs/components/Personalization/createHandleConsentFlicker.spec.js b/test/unit/specs/components/Personalization/createHandleConsentFlicker.spec.js
@@ -30,8 +30,8 @@ describe("Personalization::createHandleConsentFlicker", () => {
     consent.current.and.returnValue({ state: "out", wasSet: true });
     handleConsentFlicker();
     expect(showContainers).toHaveBeenCalled();
-    flushPromiseChains().then(() => {
-      // expect(consent.awaitConsent).not.toHaveBeenCalled();
+    return flushPromiseChains().then(() => {
+      expect(consent.awaitConsent).not.toHaveBeenCalled();
     });
   });
 
@@ -40,8 +40,8 @@ describe("Personalization::createHandleConsentFlicker", () => {
     consent.awaitConsent.and.returnValue(Promise.reject());
     handleConsentFlicker();
     expect(consent.awaitConsent).toHaveBeenCalled();
-    flushPromiseChains().then(() => {
-      // expect(showContainers).toHaveBeenCalled();
+    return flushPromiseChains().then(() => {
+      expect(showContainers).toHaveBeenCalled();
     });
   });
 
@@ -50,8 +50,8 @@ describe("Personalization::createHandleConsentFlicker", () => {
     consent.awaitConsent.and.returnValue(Promise.resolve());
     handleConsentFlicker();
     expect(consent.awaitConsent).toHaveBeenCalled();
-    flushPromiseChains().then(() => {
-      // expect(showContainers).not.toHaveBeenCalled();
+    return flushPromiseChains().then(() => {
+      expect(showContainers).not.toHaveBeenCalled();
     });
   });
 });