From d1f12d648ebc1b5db3ef68f2522e0b6ffc35e49b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Artur=20Bie=C5=84?= <artur.bien@gmail.com>
Date: Mon, 6 Apr 2020 20:38:07 +0200
Subject: [PATCH] [Select] onChange fired with current value test

---
 packages/material-ui/src/Select/Select.test.js | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/packages/material-ui/src/Select/Select.test.js b/packages/material-ui/src/Select/Select.test.js
index bfcab77b2dd4d5..48b77cfbc24407 100644
--- a/packages/material-ui/src/Select/Select.test.js
+++ b/packages/material-ui/src/Select/Select.test.js
@@ -227,6 +227,21 @@ describe('<Select />', () => {
       const selected = onChangeHandler.args[0][1];
       expect(React.isValidElement(selected)).to.equal(true);
     });
+
+    it('should not be called if selected element has the current value (value did not change)', () => {
+      const onChangeHandler = spy();
+      const { getAllByRole, getByRole } = render(
+        <Select onChange={onChangeHandler} value="1">
+          <MenuItem value="0" />
+          <MenuItem value="1" />
+          <MenuItem value="2" />
+        </Select>,
+      );
+      fireEvent.mouseDown(getByRole('button'));
+      getAllByRole('option')[1].click();
+
+      expect(onChangeHandler.callCount).to.equal(0);
+    });
   });
 
   describe('prop: value', () => {