polish test case

mui · Jun 5, 2021 · c810afd · c810afd
1 parent 0428852
commit c810afd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/packages/material-ui/src/Popover/Popover.js b/packages/material-ui/src/Popover/Popover.js
@@ -121,7 +121,7 @@ const Popover = React.forwardRef(function Popover(inProps, ref) {
     ...other
   } = props;
   const paperRef = React.useRef();
-  const handleRef = useForkRef(paperRef, PaperProps.ref);
+  const handlePaperRef = useForkRef(paperRef, PaperProps.ref);
 
   const styleProps = {
     ...props,
@@ -372,7 +372,7 @@ const Popover = React.forwardRef(function Popover(inProps, ref) {
         <PopoverPaper
           elevation={elevation}
           {...PaperProps}
-          ref={handleRef}
+          ref={handlePaperRef}
           className={clsx(classes.paper, PaperProps.className)}
         >
           {children}

diff --git a/packages/material-ui/src/Popover/Popover.test.js b/packages/material-ui/src/Popover/Popover.test.js
@@ -257,20 +257,17 @@ describe('<Popover />', () => {
   describe('PaperProps.ref', () => {
     it('should position popover correctly', () => {
       const handleEntering = spy();
-      mount(
+      render(
         <Popover
           {...defaultProps}
           open
           PaperProps={{ 'data-testid': 'Popover', ref: () => null }}
-          anchorOrigin={{ vertical: 'top', horizontal: 'left' }}
           TransitionProps={{ onEntering: handleEntering }}
         >
           <div />
         </Popover>,
       );
-
-      const element = handleEntering.args[0][0];
-      expect(element.style.top === '16px' && element.style.left === '16px').to.equal(true);
+      expect(handleEntering.args[0][0]).toHaveInlineStyle({ top: '16px', left: '16px' });
     });
   });