From 11d345264f59288477a014567dc9739fdb9cf972 Mon Sep 17 00:00:00 2001
From: Kevin Suen <ksrbkevinsuen@gmail.com>
Date: Tue, 31 Mar 2020 10:34:22 -0700
Subject: [PATCH] [Select] onChange fired with current value

---
 packages/material-ui/src/Select/Select.test.js | 2 +-
 packages/material-ui/src/Select/SelectInput.js | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/packages/material-ui/src/Select/Select.test.js b/packages/material-ui/src/Select/Select.test.js
index afe1738494ff92..fc559858e90997 100644
--- a/packages/material-ui/src/Select/Select.test.js
+++ b/packages/material-ui/src/Select/Select.test.js
@@ -214,7 +214,7 @@ describe('<Select />', () => {
     it('should get selected element from arguments', () => {
       const onChangeHandler = spy();
       const { getAllByRole, getByRole } = render(
-        <Select onChange={onChangeHandler} value="1">
+        <Select onChange={onChangeHandler} value="0">
           <MenuItem value="0" />
           <MenuItem value="1" />
           <MenuItem value="2" />
diff --git a/packages/material-ui/src/Select/SelectInput.js b/packages/material-ui/src/Select/SelectInput.js
index 6ace9e2bac9aa8..4bc58297fae498 100644
--- a/packages/material-ui/src/Select/SelectInput.js
+++ b/packages/material-ui/src/Select/SelectInput.js
@@ -139,6 +139,10 @@ const SelectInput = React.forwardRef(function SelectInput(props, ref) {
       newValue = child.props.value;
     }
 
+    if (value === newValue) {
+      return;
+    }
+
     setValue(newValue);
 
     if (onChange) {