Merge pull request #6 from hckrnews/feature/array-with-object-diff

Now you can also check the diff of arrays with objects

Author: w3nl

.github/workflows/nodejs.yml

@@ -9,7 +9,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [10.x, 12.x, 14.x, 16.x]
+        node-version: [10.x, 12.x, 14.x, 16.x, 18.x]
 
     steps:
     - uses: actions/checkout@v1

.travis.yml

@@ -7,6 +7,7 @@ node_js:
     - "12"
     - "14"
     - "16"
+    - "18"
 before_script:
     - npm run lint
     - npm test

package-lock.json

@@ -1,6 +1,6 @@
 {
     "name": "@hckrnews/arrays",
-    "version": "2.0.3",
+    "version": "2.1.0",
     "description": "Usefull array helpers.",
     "files": [
         "src/helpers.js",

package.json

@@ -7,7 +7,10 @@ class Diff {
 
     get differenceArray() {
         return this.currentArray.filter(
-            (value) => this.otherArray.indexOf(value) < 0
+            (value) =>
+                this.otherArray
+                    .map((item) => JSON.stringify(item))
+                    .indexOf(JSON.stringify(value)) < 0
         );
     }
 
@@ -17,7 +20,10 @@ class Diff {
         }
 
         return this.otherArray.filter(
-            (value) => this.currentArray.indexOf(value) < 0
+            (value) =>
+                this.currentArray
+                    .map((item) => JSON.stringify(item))
+                    .indexOf(JSON.stringify(value)) < 0
         );
     }
 

src/modules/diff.js

@@ -1,17 +1,23 @@
 export default function intersect(original, array, multi) {
     return original.filter((value) => {
         if (multi) {
-            const found = array.reduce((accumulator, currentValue) => {
-                if (currentValue.indexOf(value) >= 0) {
-                    return accumulator + 1;
-                }
+            const found = array
+                .map((item) => JSON.stringify(item))
+                .reduce((accumulator, currentValue) => {
+                    if (currentValue.indexOf(JSON.stringify(value)) >= 0) {
+                        return accumulator + 1;
+                    }
 
-                return accumulator;
-            }, 0);
+                    return accumulator;
+                }, 0);
 
             return found === array.length;
         }
 
-        return array.indexOf(value) >= 0;
+        return (
+            array
+                .map((item) => JSON.stringify(item))
+                .indexOf(JSON.stringify(value)) >= 0
+        );
     });
 }

src/modules/intersect.js

@@ -0,0 +1,21 @@
+import { expect, describe, it } from '@jest/globals';
+import { Arr } from '../src/helpers.js';
+
+const a = new Arr([{ name: 'John' }, { name: 'Peter' }, { name: 'Luke' }]);
+const b = [{ name: 'Peter' }, { name: 'Luke' }, { name: 'Paul' }];
+
+describe('Diff array with objects', () => {
+    describe('One side', () => {
+        it('Should return the difference, in this example should it be John.', () => {
+            expect([{ name: 'John' }]).toEqual(a.diff(b));
+        });
+    });
+
+    describe('Average both sides', () => {
+        it('Should return the difference of both sides, in this example should it be John and Paul.', () => {
+            expect([{ name: 'John' }, { name: 'Paul' }]).toEqual(
+                a.diff(b, true)
+            );
+        });
+    });
+});

test/diff-object.js

@@ -0,0 +1,22 @@
+import { expect, describe, it } from '@jest/globals';
+import { Arr } from '../src/helpers.js';
+
+const a = new Arr([{ name: 'John' }, { name: 'Peter' }, { name: 'Luke' }]);
+const b = [{ name: 'Peter' }, { name: 'Luke' }, { name: 'Paul' }];
+const c = [{ name: 'Luke' }, { name: 'Paul' }, { name: 'John' }];
+
+describe('Intersect array with objects', () => {
+    describe('One side', () => {
+        it('Should return the intersect, in this example should it be Peter and Luke.', () => {
+            expect([{ name: 'Peter' }, { name: 'Luke' }]).toEqual(
+                a.intersect(b)
+            );
+        });
+    });
+
+    describe('Average both sides', () => {
+        it('Should return the intersect of more than 2 arrays, in this example should it be Luke.', () => {
+            expect([{ name: 'Luke' }]).toEqual(a.intersect([b, c], true));
+        });
+    });
+});