Bump jsdom from 16.0.1 to 16.1.0 (#19533)

Author: dependabot-preview[bot]

packages/material-ui/src/Tabs/Tabs.test.js

@@ -25,6 +25,8 @@ const hasRightScrollButton = container => findScrollButton(container, 'right') !
 
 describe('<Tabs />', () => {
   const isSafari = /^((?!chrome|android).)*safari/i.test(navigator.userAgent);
+  // tests mocking getBoundingClientRect prevent mocha to exit
+  const isJSDOM = navigator.userAgent === 'node.js';
 
   // The test fails on Safari with just:
   //
@@ -173,7 +175,11 @@ describe('<Tabs />', () => {
         expect(container.querySelector(`.${classes.indicator}`)).to.be.ok;
       });
 
-      it('should update the indicator at each render', () => {
+      it('should update the indicator at each render', function test() {
+        if (isJSDOM) {
+          this.skip();
+        }
+
         const { setProps, container, getByRole } = render(
           <Tabs value={1}>
             <Tab />
@@ -281,7 +287,10 @@ describe('<Tabs />', () => {
       expect(container.querySelectorAll(selector)).to.have.lengthOf(1);
     });
 
-    it('should response to scroll events', () => {
+    it('should response to scroll events', function test() {
+      if (isJSDOM) {
+        this.skip();
+      }
       const { container, setProps, getByRole } = render(tabs);
       const tablistContainer = getByRole('tablist').parentElement;
 
@@ -358,7 +367,11 @@ describe('<Tabs />', () => {
       expect(container.querySelectorAll(`.${classes.scrollButtons}`)).to.have.lengthOf(2);
     });
 
-    it('should handle window resize event', () => {
+    it('should handle window resize event', function test() {
+      if (isJSDOM) {
+        this.skip();
+      }
+
       const { container, setProps, getByRole } = render(
         <Tabs
           value={0}
@@ -555,7 +568,11 @@ describe('<Tabs />', () => {
       clock.restore();
     });
 
-    it('should scroll left tab into view', () => {
+    it('should scroll left tab into view', function test() {
+      if (isJSDOM) {
+        this.skip();
+      }
+
       const { setProps, getByRole } = render(
         <Tabs value={0} variant="scrollable" style={{ width: 200 }}>
           <Tab />
@@ -598,7 +615,11 @@ describe('<Tabs />', () => {
   });
 
   describe('prop: orientation', () => {
-    it('should support orientation="vertical"', () => {
+    it('should support orientation="vertical"', function test() {
+      if (isJSDOM) {
+        this.skip();
+      }
+
       const { setProps, container, getByRole } = render(
         <Tabs value={1} variant="scrollable" scrollButtons="on" orientation="vertical">
           <Tab />

packages/material-ui/src/internal/animate.test.js

@@ -2,19 +2,21 @@ import { assert } from 'chai';
 import animate from './animate';
 
 describe('animate', () => {
-  const isSafari = /^((?!chrome|android).)*safari/i.test(navigator.userAgent);
+  let container;
 
-  // The test fails on Safari with just:
-  //
-  // container.scrollLeft = 200;
-  // assert.strictEqual(container.scrollLeft, 200); 💥
-  if (isSafari) {
-    return;
-  }
+  before(function beforeHook() {
+    const isSafari = /^((?!chrome|android).)*safari/i.test(navigator.userAgent);
+    const isJSDOM = navigator.userAgent === 'node.js';
+    if (isJSDOM || isSafari) {
+      // The test fails on Safari with just:
+      //
+      // container.scrollLeft = 200;
+      // assert.strictEqual(container.scrollLeft, 200); 💥
 
-  let container;
+      // in JSDOM the test prevents mocha from exiting
+      this.skip();
+    }
 
-  before(() => {
     container = document.createElement('div');
     container.style.cssText = [
       'height: 100px',
@@ -29,7 +31,9 @@ describe('animate', () => {
   });
 
   after(() => {
-    document.body.removeChild(container);
+    if (container !== undefined) {
+      document.body.removeChild(container);
+    }
   });
 
   it('should work', done => {

yarn.lock

@@ -5286,10 +5286,10 @@ cssom@~0.3.6:
   resolved "https://registry.yarnpkg.com/cssom/-/cssom-0.3.8.tgz#9f1276f5b2b463f2114d3f2c75250af8c1a36f4a"
   integrity sha512-b0tGHbfegbhPJpxpiBPU2sCkigAqtM9O121le6bbOlgyV+NyGyCmVfJ6QW9eRjz8CpNfWEOYBIMIGRYkLwsIYg==
 
-cssstyle@^2.0.0:
-  version "2.0.0"
-  resolved "https://registry.yarnpkg.com/cssstyle/-/cssstyle-2.0.0.tgz#911f0fe25532db4f5d44afc83f89cc4b82c97fe3"
-  integrity sha512-QXSAu2WBsSRXCPjvI43Y40m6fMevvyRm8JVAuF9ksQz5jha4pWP1wpaK7Yu5oLFc6+XAY+hj8YhefyXcBB53gg==
+cssstyle@^2.1.0:
+  version "2.2.0"
+  resolved "https://registry.yarnpkg.com/cssstyle/-/cssstyle-2.2.0.tgz#e4c44debccd6b7911ed617a4395e5754bba59992"
+  integrity sha512-sEb3XFPx3jNnCAMtqrXPDeSgQr+jojtCeNf8cvMNMh1cG970+lljssvQDzPq6lmmJu2Vhqood/gtEomBiHOGnA==
   dependencies:
     cssom "~0.3.6"
 
@@ -8807,15 +8807,15 @@ jscodeshift@^0.7.0:
     write-file-atomic "^2.3.0"
 
 jsdom@^16.0.0:
-  version "16.0.1"
-  resolved "https://registry.yarnpkg.com/jsdom/-/jsdom-16.0.1.tgz#5a5214cf69609531bf6fa5b6322fdd90ceed5bf7"
-  integrity sha512-wKJe/APzq+ak9i+2ybWE20lDIhF9AkGKSZf8UsjPN39acatFB6oA7K397kQvHVikds0yQono2h6J7UjbPtPOWw==
+  version "16.1.0"
+  resolved "https://registry.yarnpkg.com/jsdom/-/jsdom-16.1.0.tgz#7e245db708ee00302947be7beb88a2ffa587ad2b"
+  integrity sha512-kpIcNAuZYc/L17WADOOHslz/q5+3SipP/iRb3j6zd1zQ6pFJubLi/VCdD3NqBpj/IKKK4YXny1vv44rbEUSGFg==
   dependencies:
     abab "^2.0.3"
     acorn "^7.1.0"
     acorn-globals "^4.3.2"
     cssom "^0.4.4"
-    cssstyle "^2.0.0"
+    cssstyle "^2.1.0"
     data-urls "^2.0.0"
     decimal.js "^10.2.0"
     domexception "^2.0.1"