[core] Avoid popper.js npm warning (#20433)

* [core] Avoid popper.js warning

* [core] Use lts

Author: oliviertassinari

packages/material-ui/package.json

@@ -55,7 +55,7 @@
     "@types/react-transition-group": "^4.2.0",
     "clsx": "^1.0.2",
     "hoist-non-react-statics": "^3.3.2",
-    "popper.js": "^1.14.1",
+    "popper.js": "^1.16.1-lts",
     "prop-types": "^15.7.2",
     "react-is": "^16.8.0",
     "react-transition-group": "^4.3.0"

packages/material-ui/src/Popper/Popper.js

@@ -1,6 +1,6 @@
 import * as React from 'react';
 import PropTypes from 'prop-types';
-import PopperJS from 'popper.js';
+import PopperJs from 'popper.js';
 import { chainPropTypes, refType } from '@material-ui/utils';
 import { useTheme } from '@material-ui/styles';
 import Portal from '../Portal';
@@ -123,7 +123,7 @@ const Popper = React.forwardRef(function Popper(props, ref) {
       }
     }
 
-    const popper = new PopperJS(getAnchorEl(anchorEl), tooltipRef.current, {
+    const popper = new PopperJs(getAnchorEl(anchorEl), tooltipRef.current, {
       placement: rtlPlacement,
       ...popperOptions,
       modifiers: {

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

@@ -7,7 +7,7 @@ import { ThemeProvider, createMuiTheme } from '@material-ui/core/styles';
 import describeConformance from '@material-ui/core/test-utils/describeConformance';
 import { createClientRender } from 'test/utils/createClientRender';
 import consoleErrorMock from 'test/utils/consoleErrorMock';
-import PopperJS from 'popper.js';
+import PopperJs from 'popper.js';
 import Grow from '../Grow';
 import Popper from './Popper';
 
@@ -238,12 +238,12 @@ describe('<Popper />', () => {
       const ref1 = React.createRef();
       const ref2 = React.createRef();
       const wrapper = mount(<Popper {...defaultProps} popperRef={ref1} />);
-      assert.strictEqual(ref1.current instanceof PopperJS, true);
+      assert.strictEqual(ref1.current instanceof PopperJs, true);
       wrapper.setProps({
         popperRef: ref2,
       });
       assert.strictEqual(ref1.current, null);
-      assert.strictEqual(ref2.current instanceof PopperJS, true);
+      assert.strictEqual(ref2.current instanceof PopperJs, true);
     });
   });
 

yarn.lock

@@ -12414,10 +12414,10 @@ [email protected]:
   dependencies:
     ts-pnp "^1.1.2"
 
-popper.js@^1.14.1:
-  version "1.16.0"
-  resolved "https://registry.yarnpkg.com/popper.js/-/popper.js-1.16.0.tgz#2e1816bcbbaa518ea6c2e15a466f4cb9c6e2fbb3"
-  integrity sha512-+G+EkOPoE5S/zChTpmBSSDYmhXJ5PsW8eMhH8cP/CQHMFPBG/kC9Y5IIw6qNYgdJ+/COf0ddY2li28iHaZRSjw==
+popper.js@^1.16.1-lts:
+  version "1.16.1"
+  resolved "https://registry.yarnpkg.com/popper.js/-/popper.js-1.16.1.tgz#2a223cb3dc7b6213d740e40372be40de43e65b1b"
+  integrity sha512-Wb4p1J4zyFTbM+u6WuO4XstYx4Ky9Cewe4DWrel7B0w6VVICvPwdOpotjzcf6eD8TsckVnIMNONQyPIUFOUbCQ==
 
 posix-character-classes@^0.1.0:
   version "0.1.1"