Use ensureValueInRange & add precision to test.

react-component · Oct 10, 2017 · d8429a9 · d8429a9
1 parent 310daf2
commit d8429a9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/src/Range.jsx b/src/Range.jsx
@@ -64,13 +64,7 @@ class Range extends React.Component {
 
     if (bounds.some(v => utils.isValueOutOfRange(v, nextProps))) {
       const newValues = value.map((v) => {
-        if (v < nextBounds[0]) {
-          v = nextBounds[0];
-        }
-        if (v > nextBounds[1]) {
-          v = nextBounds[1];
-        }
-        return v;
+        return utils.ensureValueInRange(v, nextProps);
       });
       this.props.onChange(newValues);
     }

diff --git a/tests/Range.test.js b/tests/Range.test.js
@@ -81,7 +81,7 @@ describe('Range', () => {
 
   it('should only update bounds that are out of range', () => {
     const props = { min: 0, max: 10000, value: [0.01, 10000], onChange: jest.fn() };
-    const range = mount(<Range {...props} />);
+    const range = mount(<Range {...props} step={0.1} />);
     range.setProps({ min: 0, max: 500 });
 
     expect(props.onChange).toHaveBeenCalledWith([0.01, 500]);