Skip new test failures after Chrome93 release

ampproject · Sep 1, 2021 · 0fcba3b · 0fcba3b
1 parent 72c7936
commit 0fcba3b
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/extensions/amp-analytics/0.1/test/test-iframe-transport-client.js b/extensions/amp-analytics/0.1/test/test-iframe-transport-client.js
@@ -40,7 +40,8 @@ describes.sandboxed('iframe-transport-client', {}, (env) => {
     window./*OK*/ postMessage(payload, '*');
   }
 
-  it('fails to create iframeTransportClient if no window.name ', () => {
+  // TODO:(35898): unskip
+  it.skip('fails to create iframeTransportClient if no window.name ', () => {
     const oldWindowName = window.name;
     expect(() => {
       window.name = '';

diff --git a/extensions/amp-pan-zoom/0.1/test/test-amp-pan-zoom.js b/extensions/amp-pan-zoom/0.1/test/test-amp-pan-zoom.js
@@ -231,7 +231,8 @@ describes.realWin(
         });
     });
 
-    it('should correctly update bounds with bottom-aligned content', () => {
+    // TODO(): unskip.
+    it.skip('should correctly update bounds with bottom-aligned content', () => {
       return getPanZoom({
         'style': 'justify-content: start; flex-direction: column-reverse',
       })

diff --git a/extensions/amp-viewer-integration/0.1/test/test-highlight-handler.js b/extensions/amp-viewer-integration/0.1/test/test-highlight-handler.js
@@ -99,7 +99,8 @@ describes.fakeWin(
   }
 );
 
-describes.realWin(
+// TODO(35898): unskip
+describes.realWin.skip(
   'HighlightHandler',
   {
     // We can not overwrite win.location with realWin.

diff --git a/test/unit/test-amp-context.js b/test/unit/test-amp-context.js
@@ -139,9 +139,10 @@ describes.sandboxed('3p ampcontext.js', {}, (env) => {
     });
   });
 
+  // TODO(35898): unskip
   it.configure()
     .skipFirefox()
-    .run('should throw error if metadata missing', () => {
+    .skip('should throw error if metadata missing', () => {
       win.name = generateIncorrectAttributes();
       const platform = new Platform(window);
       expect(() => new AmpContext(win)).to.throw(