unskip some more tests (#40221)

this was fixed in #40218
ampproject · Jan 8, 2025 · d679610 · d679610
1 parent 87bb0ef
commit d679610
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/extensions/amp-lightbox/0.1/test/test-amp-lightbox.js b/extensions/amp-lightbox/0.1/test/test-amp-lightbox.js
@@ -105,8 +105,7 @@ describes.realWin(
       });
     });
 
-    // TODO(#40214): fix flaky test.
-    it.skip('should close on close-watcher signal', async () => {
+    it('should close on close-watcher signal', async () => {
       const lightbox = createLightbox();
       const impl = await lightbox.getImpl(true);
 

diff --git a/extensions/amp-skimlinks/0.1/test/test-affiliate-link-resolver.js b/extensions/amp-skimlinks/0.1/test/test-affiliate-link-resolver.js
@@ -107,8 +107,7 @@ describes.fakeWin(
           resolver.resolveUnknownAnchors(anchorList);
         });
 
-        // TODO(#40214): fix flaky test.
-        it.skip('Should only ask for new domains the next times', () => {
+        it('Should only ask for new domains the next times', () => {
           // Set the domains like if we had already done a call to the API.
           resolver.domains_ = alreadyResolvedDomains;