From 74d85f5bb2c6240559608f9c65339df05624d7a8 Mon Sep 17 00:00:00 2001
From: Olivier Tassinari <olivier.tassinari@gmail.com>
Date: Sun, 7 Jun 2020 02:07:12 +0200
Subject: [PATCH] [core] Batch small changes (#21335)

---
 .../pages/components/material-icons/SearchIcons.js   |  7 +++++--
 packages/material-ui/src/Popper/Popper.test.js       |  2 +-
 packages/material-ui/src/Select/Select.test.js       | 12 +-----------
 3 files changed, 7 insertions(+), 14 deletions(-)

diff --git a/docs/src/pages/components/material-icons/SearchIcons.js b/docs/src/pages/components/material-icons/SearchIcons.js
index 74d2a4db1a62b7..e0b7644b1ee415 100644
--- a/docs/src/pages/components/material-icons/SearchIcons.js
+++ b/docs/src/pages/components/material-icons/SearchIcons.js
@@ -107,6 +107,9 @@ let Icons = (props) => {
               onClick={handleClickOpen}
               title={icon.key}
               className={classes.iconSvg}
+              data-ga-event-category="material-icons"
+              data-ga-event-action="click"
+              data-ga-event-label={icon.key}
             />
             {/* eslint-disable jsx-a11y/no-noninteractive-element-interactions, jsx-a11y/click-events-have-key-events */}
             <p onClick={handleClick}>{icon.key}</p>
@@ -421,8 +424,8 @@ export default function SearchIcons() {
               window.ga('send', {
                 hitType: 'event',
                 eventCategory: 'material-icons',
-                eventAction: value,
-                eventLabel: 'no results',
+                eventAction: 'no-results',
+                eventLabel: value,
               });
             }
           });
diff --git a/packages/material-ui/src/Popper/Popper.test.js b/packages/material-ui/src/Popper/Popper.test.js
index 0dc8f518f7cffe..50c78b8a61e33a 100644
--- a/packages/material-ui/src/Popper/Popper.test.js
+++ b/packages/material-ui/src/Popper/Popper.test.js
@@ -52,7 +52,7 @@ describe('<Popper />', () => {
           </Popper>
         </ThemeProvider>,
       );
-      expect(renderSpy.callCount).to.equal(2); // 2 for strict mode
+      expect(renderSpy.callCount).to.equal(2); // strict mode renders twice
       expect(renderSpy.args[0][0]).to.equal('top');
     });
 
diff --git a/packages/material-ui/src/Select/Select.test.js b/packages/material-ui/src/Select/Select.test.js
index ed6ffb36fa8473..ac3f47dafbe5f3 100644
--- a/packages/material-ui/src/Select/Select.test.js
+++ b/packages/material-ui/src/Select/Select.test.js
@@ -215,16 +215,6 @@ describe('<Select />', () => {
   });
 
   describe('prop: onChange', () => {
-    let clock;
-
-    before(() => {
-      clock = useFakeTimers();
-    });
-
-    after(() => {
-      clock.restore();
-    });
-
     it('should get selected element from arguments', () => {
       const onChangeHandler = spy();
       const { getAllByRole, getByRole } = render(
@@ -1024,7 +1014,7 @@ describe('<Select />', () => {
     expect(getByRole('button')).to.have.text('France');
   });
 
-  it('should support native from validation', function test() {
+  it('should support native form validation', function test() {
     if (/jsdom/.test(window.navigator.userAgent)) {
       // see https://github.com/jsdom/jsdom/issues/123
       this.skip();