8000 fix: inRange test (#5821) · lodash/lodash@a67a085 · GitHub
[go: up one dir, main page]

Skip to content

Commit a67a085

Browse files
authored
fix: inRange test (#5821)
1 parent c7c70a7 commit a67a085

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

src/inRange.ts

+7-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
import baseInRange from './.internal/baseInRange.js';
2+
import toFinite from './toFinite';
3+
import toNumber from './toNumber';
24

35
/**
46
* Checks if `number` is between `start` and up to, but not including, `end`. If
@@ -37,11 +39,15 @@ import baseInRange from './.internal/baseInRange.js';
3739
* // => true
3840
*/
3941
function inRange(number, start, end) {
42+
start = toFinite(start);
4043
if (end === undefined) {
4144
end = start;
4245
start = 0;
46+
} else {
47+
end = toFinite(end);
4348
}
44-
return baseInRange(+number, +start, +end);
49+
number = toNumber(number);
50+
return baseInRange(number, start, end);
4551
}
4652

4753
export default inRange;

test/inRange.spec.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,6 @@ describe('inRange', () => {
4848
inRange(-1, -1, NaN),
4949
];
5050

51-
expect(actual, lodashStable.map(actual).toEqual(stubTrue));
51+
expect(actual).toEqual(lodashStable.map(actual,stubTrue));
5252
});
5353
});

0 commit comments

Comments
 (0)
0