feat: init is primitive funciton

Apply suggestions from code review

Co-authored-by: Rich Trott <rtrott@gmail.com>

chore: use jest 'toBeFalsy'

fix: inRange test (#5821)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "lodash",
-  "version": "5.0.0",
+  "version": "5.1.0",
   "license": "MIT",
   "private": true,
   "main": "dist/lodash.js",
@@ -47,4 +47,4 @@
     "yarn": "1.22.19"
   },
   "engineStrict": true
-}
+}

src/inRange.ts

@@ -1,4 +1,6 @@
 import baseInRange from './.internal/baseInRange.js';
+import toFinite from './toFinite';
+import toNumber from './toNumber';
 
 /**
  * Checks if `number` is between `start` and up to, but not including, `end`. If
@@ -37,11 +39,15 @@ import baseInRange from './.internal/baseInRange.js';
  * // => true
  */
 function inRange(number, start, end) {
+    start = toFinite(start);
     if (end === undefined) {
         end = start;
         start = 0;
+    } else {
+        end = toFinite(end);
     }
-    return baseInRange(+number, +start, +end);
+    number = toNumber(number);
+    return baseInRange(number, start, end);
 }
 
 export default inRange;

src/isPrimitive.ts

@@ -0,0 +1,24 @@
+/**
+ * Checks if `value` is a primitive.
+ *
+ * @since 4.1.0
+ * @category Lang
+ * @param {*} value The value to check.
+ * @returns {boolean} Returns `true` if `value` is a primitive, else `false`.
+ * @link https://developer.mozilla.org/en-US/docs/Glossary/Primitive
+ * @example
+ *
+ * isPrimitive('abc')
+ * // => true
+ *
+ * isPrimitive(null)
+ * // => true
+ *
+ * isPrimitive({})
+ * // => false
+ */
+function isPrimitive(value?: unknown): boolean {
+    return value !== Object(value);
+}
+
+export default isPrimitive;

test/inRange.spec.js

@@ -48,6 +48,6 @@ describe('inRange', () => {
             inRange(-1, -1, NaN),
         ];
 
-        expect(actual, lodashStable.map(actual).toEqual(stubTrue));
+        expect(actual).toEqual(lodashStable.map(actual,stubTrue));
     });
 });

test/isPrimitive.spec.js

@@ -0,0 +1,24 @@
+import isPrimitive from '../src/isPrimitive';
+
+describe('isPrimitive', () => {
+    it('should return `true` for primitive', () => {
+        expect(isPrimitive()).toBeTruthy();
+        expect(isPrimitive('a')).toBeTruthy();
+        expect(isPrimitive(13)).toBeTruthy();
+        expect(isPrimitive(0)).toBeTruthy();
+        expect(isPrimitive(true)).toBeTruthy();
+        expect(isPrimitive(false)).toBeTruthy();
+        expect(isPrimitive(null)).toBeTruthy();
+        expect(isPrimitive(10n)).toBeTruthy();
+    });
+
+    it('should return `false` for non-primitive', () => {
+        expect(isPrimitive(Object('args'))).toBeFalsy();
+        expect(isPrimitive({ title: 'lodash' })).toBeFalsy();
+        expect(isPrimitive([1, 2, 3])).toBeFalsy();
+        expect(isPrimitive(new Date())).toBeFalsy();
+        expect(isPrimitive(new Error())).toBeFalsy();
+        expect(isPrimitive(/lodash/)).toBeFalsy();
+        expect(isPrimitive({ 0: 1, length: 1 })).toBeFalsy();
+    });
+});