[refactored] firevent inside act

mui · Nov 2, 2019 · 777fa1a · 777fa1a
1 parent 8f6a480
commit 777fa1a
Showing 1 changed file with 4 additions and 12 deletions.
diff --git a/packages/material-ui/src/Select/Select.test.js b/packages/material-ui/src/Select/Select.test.js
@@ -196,9 +196,7 @@ describe('<Select />', () => {
   it('should focus list if no selection', () => {
     const { getByRole } = render(<Select value="" autoFocus />);
 
-    act(() => {
-      fireEvent.mouseDown(getByRole('button'));
-    });
+    fireEvent.mouseDown(getByRole('button'));
 
     // TODO not matching WAI-ARIA authoring practices. It should focus the first (or selected) item.
     expect(getByRole('listbox')).to.be.focused;
@@ -612,9 +610,7 @@ describe('<Select />', () => {
       }
       const { getByRole, queryByRole } = render(<ControlledWrapper />);
 
-      act(() => {
-        fireEvent.mouseDown(getByRole('button'));
-      });
+      fireEvent.mouseDown(getByRole('button'));
 
       expect(getByRole('listbox')).to.be.ok;
 
@@ -655,9 +651,7 @@ describe('<Select />', () => {
       const button = getByRole('button');
       stub(button, 'clientWidth').get(() => 14);
 
-      act(() => {
-        fireEvent.mouseDown(button);
-      });
+      fireEvent.mouseDown(button);
 
       expect(getByTestId('paper').style).to.have.property('minWidth', '14px');
     });
@@ -671,9 +665,7 @@ describe('<Select />', () => {
       const button = getByRole('button');
       stub(button, 'clientWidth').get(() => 14);
 
-      act(() => {
-        fireEvent.mouseDown(button);
-      });
+      fireEvent.mouseDown(button);
 
       expect(getByTestId('paper').style).to.have.property('minWidth', '');
     });