Skip to content

Commit 02e47fd

Browse files
authored
[Select] Add new test for onChange (#20444)
1 parent a685203 commit 02e47fd

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

packages/material-ui/src/Select/Select.test.js

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -227,6 +227,21 @@ describe('<Select />', () => {
227227
const selected = onChangeHandler.args[0][1];
228228
expect(React.isValidElement(selected)).to.equal(true);
229229
});
230+
231+
it('should not be called if selected element has the current value (value did not change)', () => {
232+
const onChangeHandler = spy();
233+
const { getAllByRole, getByRole } = render(
234+
<Select onChange={onChangeHandler} value="1">
235+
<MenuItem value="0" />
236+
<MenuItem value="1" />
237+
<MenuItem value="2" />
238+
</Select>,
239+
);
240+
fireEvent.mouseDown(getByRole('button'));
241+
getAllByRole('option')[1].click();
242+
243+
expect(onChangeHandler.callCount).to.equal(0);
244+
});
230245
});
231246

232247
describe('prop: value', () => {

0 commit comments

Comments
 (0)