isObject returns false for arrays [major]

README.md

@@ -44,13 +44,12 @@ All methods return `true` or `false`.
 * `isSymbol`
 * `isUndefined`
 * `isRegExp`
-* `isObject`
 * `isDate`
 * `isError`
 * `isFunction`
 * `isPrimitive`
 
-NB [core-util-is](https://www.npmjs.com/package/core-util-is)'s `isBuffer` is not included.
+NB [core-util-is](https://www.npmjs.com/package/core-util-is)'s `isBuffer` is not included. `isObject` method differs from [core-util-is](https://www.npmjs.com/package/core-util-is)'s method of same name (see [below](#objects)).
 
 ### Additional functions
 
@@ -69,6 +68,7 @@ NB [core-util-is](https://www.npmjs.com/package/core-util-is)'s `isBuffer` is no
 
 #### Objects
 
+* `isObject` - `true` if passed object (not including arrays)
 * `isEmptyObject` - `true` if passed object with no properties
 
 #### Other

src/index.js

@@ -46,10 +46,6 @@ export function isRegExp(arg) {
 	return isTypeByToString('RegExp', arg);
 }
 
-export function isObject(arg) {
-	return isType('object', arg) && !isNull(arg);
-}
-
 export function isDate(arg) {
 	return isTypeByToString('Date', arg);
 }
@@ -87,6 +83,10 @@ export function isFullString(arg) {
 
 // Objects
 
+export function isObject(arg) {
+	return isType('object', arg) && !isNull(arg) && !isArray(arg);
+}
+
 export function isEmptyObject(arg) {
 	return isObject(arg) && Object.keys(arg).length === 0;
 }

test/objects.test.js

@@ -5,10 +5,40 @@
 
 // Modules
 // eslint-disable-next-line import/no-unresolved, node/no-missing-import
-import {isEmptyObject} from 'is-it-type';
+import {isObject, isEmptyObject} from 'is-it-type';
 
 // Tests
 
+describe('isObject', () => {
+	it('returns true for {}', () => {
+		expect(isObject({})).toBe(true);
+	});
+
+	it('returns true for {a: 1}', () => {
+		expect(isObject({a: 1})).toBe(true);
+	});
+
+	it('returns false for []', () => {
+		expect(isObject([])).toBe(false);
+	});
+
+	it('returns false for [1, 2, 3]', () => {
+		expect(isObject([1, 2, 3])).toBe(false);
+	});
+
+	it("returns false for ''", () => {
+		expect(isObject('')).toBe(false);
+	});
+
+	it('returns false for null', () => {
+		expect(isObject(null)).toBe(false);
+	});
+
+	it('returns false for undefined', () => {
+		expect(isObject(undefined)).toBe(false);
+	});
+});
+
 describe('isEmptyObject', () => {
 	it('returns true for {}', () => {
 		expect(isEmptyObject({})).toBe(true);
@@ -18,11 +48,23 @@ describe('isEmptyObject', () => {
 		expect(isEmptyObject({a: 1})).toBe(false);
 	});
 
+	it('returns false for []', () => {
+		expect(isEmptyObject([])).toBe(false);
+	});
+
+	it('returns false for [1, 2, 3]', () => {
+		expect(isEmptyObject([1, 2, 3])).toBe(false);
+	});
+
 	it("returns false for ''", () => {
 		expect(isEmptyObject('')).toBe(false);
 	});
 
 	it('returns false for null', () => {
 		expect(isEmptyObject(null)).toBe(false);
 	});
+
+	it('returns false for undefined', () => {
+		expect(isEmptyObject(undefined)).toBe(false);
+	});
 });