Merge pull request #18 from palashmon/feature/add-ci

Author: palashmon

.github/dependabot.yml

@@ -0,0 +1,11 @@
+# To get started with Dependabot version updates, you'll need to specify which
+# package ecosystems to update and where the package manifests are located.
+# Please see the documentation for all configuration options:
+# https://docs.github.com/github/administering-a-repository/configuration-options-for-dependency-updates
+
+version: 2
+updates:
+  - package-ecosystem: "npm" # See documentation for possible values
+    directory: "/" # Location of package manifests
+    schedule:
+      interval: "weekly"

.github/workflows/main.yaml

@@ -0,0 +1,22 @@
+name: CI
+on:
+  - push
+  - pull_request
+jobs:
+  test:
+    name: Node.js ${{ matrix.node-version }}
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        node-version:
+          - 18
+          - 16
+          - 14
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: npm install
+      - run: npm test

index.js

@@ -1,6 +1,15 @@
-module.exports = (arrayInput = []) => {
+function cloneArrayObjects(arrayInput = []) {
 	if (!Array.isArray(arrayInput)) {
 		throw new TypeError(`Expected an array, got ${typeof arrayInput}`);
 	}
-	return arrayInput.map(obj => ({...obj}));
-};
+
+	return arrayInput.map(object => {
+		if (typeof object !== 'object' || object === null) {
+			throw new TypeError(`Expected an object, got ${typeof object}`);
+		}
+
+		return {...object};
+	});
+}
+
+export default cloneArrayObjects;

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "clone-array-objects",
-	"version": "1.0.0",
+	"version": "1.0.2",
 	"description": "Clone an array of objects",
 	"license": "MIT",
 	"repository": "https://github.com/palashmon/clone-array-objects",
@@ -10,7 +10,7 @@
 		"url": "https://github.com/palashmon"
 	},
 	"engines": {
-		"node": ">=8"
+		"node": ">=14"
 	},
 	"scripts": {
 		"test": "xo && ava",
@@ -30,8 +30,9 @@
 		"test"
 	],
 	"devDependencies": {
-		"ava": "^3.0.0",
+		"ava": "^5.1.0",
 		"nyc": "^15.0.0",
-		"xo": "^0.30.0"
-	}
-}
+		"xo": "^0.54.0"
+	},
+	"type": "module"
+}

readme.md

@@ -1,11 +1,6 @@
-# clone-array-objects
+# clone-array-objects ![CI](https://github.com/palashmon/clone-array-objects/actions/workflows/main.yaml/badge.svg)
 
-[![Build Status](https://travis-ci.org/palashmon/clone-array-objects.svg?branch=master)](https://travis-ci.org/palashmon/clone-array-objects)
-[![npm](https://img.shields.io/npm/v/clone-array-objects.svg)](https://www.npmjs.org/package/clone-array-objects)
-[![codecov](https://codecov.io/gh/palashmon/clone-array-objects/branch/master/graph/badge.svg)](https://codecov.io/gh/palashmon/clone-array-objects)
-[![Gzip Size](https://img.badgesize.io/https://unpkg.com/clone-array-objects?compression=gzip)](https://bundlephobia.com/result?p=clone-array-objects) 
-
-Tiny module to clone an array of objects
+> Tiny module to clone an array of objects
 
 ## Install
 

test.js

@@ -1,24 +1,44 @@
 import test from 'ava';
-import cloneArrayObjects from '.';
+import cloneArrayObjects from './index.js';
 
-test('Return typeerror when arrayInput is not an array.', t => {
-	const err = t.throws(() => {
-		cloneArrayObjects(23);
-	}, TypeError);
-	t.is(err.message, 'Expected an array, got number');
+test('should return a new array with cloned objects', t => {
+	const input = [
+		{name: 'John', age: 30},
+		{name: 'Jane', age: 25},
+	];
+	const output = cloneArrayObjects(input);
+
+	t.not(output, input); // Should not modify the original array
+	t.deepEqual(output, input); // Should have the same values as the input
+});
+
+test('should throw a TypeError if the input is not an array', t => {
+	const input = 'not an array';
+
+	const error = t.throws(() => cloneArrayObjects(input));
+
+	t.is(error.message, `Expected an array, got ${typeof input}`);
+});
+
+test('should throw a TypeError if any object in the array is null', t => {
+	const input = [{name: 'John', age: 30}, null];
+
+	const error = t.throws(() => cloneArrayObjects(input));
+
+	t.is(error.message, 'Expected an object, got object');
 });
 
-test('Return empty array when no valid input passed.', t => {
-	t.deepEqual(cloneArrayObjects(), []);
-	t.deepEqual(cloneArrayObjects([]), []);
+test('should throw a TypeError if any object in the array is not an object', t => {
+	const input = [{name: 'John', age: 30}, 'not an object'];
+
+	const error = t.throws(() => cloneArrayObjects(input));
+
+	t.is(error.message, 'Expected an object, got string');
 });
 
-test('Test clone', t => {
-	const actual = [{a: 1}, {b: 2}];
-	const expected = [{a: 1}, {b: 2}];
-	const modified = [{a: 1}, {b: 3}];
-	t.deepEqual(cloneArrayObjects(actual), expected);
-	expected[1].b = 3;
-	t.notDeepEqual(cloneArrayObjects(actual), expected);
-	t.deepEqual(modified, expected);
+test('should return an empty array if the input is an empty array', t => {
+	const input = [];
+	const output = cloneArrayObjects(input);
+
+	t.deepEqual(output, []);
 });