chore: remove extra deps

Author: KuznetsovRoman

index.js

@@ -1,6 +1,5 @@
 'use strict';
 
-const _ = require('lodash');
 const colorDiff = require('color-diff');
 const img = require('./lib/image');
 const areColorsSame = require('./lib/same-colors');
@@ -122,7 +121,7 @@ const buildDiffImage = async (img1, img2, options) => {
 };
 
 const getToleranceFromOpts = (opts) => {
-    if (!_.hasIn(opts, 'tolerance')) {
+    if (!Object.hasOwn(opts, 'tolerance')) {
         return JND;
     }
 
@@ -137,11 +136,13 @@ const prepareOpts = (opts) => {
     opts = opts || {};
     opts.tolerance = getToleranceFromOpts(opts);
 
-    return _.defaults(opts, {
+    const defaults = {
         ignoreCaret: true,
         ignoreAntialiasing: true,
         antialiasingTolerance: 0
-    });
+    };
+
+    return Object.assign(defaults, opts);
 };
 
 const getMaxDiffBounds = (first, second) => {

lib/image/index.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const fs = require('fs-extra');
+const fs = require('fs');
 const NestedError = require('nested-error-stacks');
 const sharp = require('sharp');
 const OriginalIMG = require('./original-image');
@@ -19,7 +19,7 @@ exports.fromBuffer = async (buffer, opts) => {
 
 exports.fromFile = async (filePath, opts = {}) => {
     try {
-        const buffer = await fs.readFile(filePath);
+        const buffer = await fs.promises.readFile(filePath);
         return exports.fromBuffer(buffer, opts);
     } catch (err) {
         throw new NestedError(`Can't load img file ${filePath}`, err);

lib/img-buffer/index.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const fs = require('fs-extra');
+const fs = require('fs');
 const NestedError = require('nested-error-stacks');
 const OriginalBuffer = require('./original-buffer');
 const BoundedBuffer = require('./bounded-buffer');
@@ -13,7 +13,7 @@ exports.create = (buffer, {boundingBox} = {}) => {
 
 exports.fromFile = async (filePath, opts = {}) => {
     try {
-        const buffer = await fs.readFile(filePath);
+        const buffer = await fs.promises.readFile(filePath);
         return exports.create(buffer, opts);
     } catch (err) {
         throw new NestedError(`Can't load img file ${filePath}`, err);

lib/utils.js

@@ -1,6 +1,5 @@
 'use strict';
 
-const _ = require('lodash');
 const parseColor = require('parse-color');
 const img = require('./image');
 const buffer = require('./img-buffer');
@@ -88,7 +87,7 @@ exports.formatImages = (img1, img2) => {
     validators.validateImages(img1, img2);
 
     return [img1, img2].map((i) => {
-        return _.isObject(i) && !Buffer.isBuffer(i) ? i : {source: i, boundingBox: null};
+        return i !== null && typeof i === 'object' && !Buffer.isBuffer(i) ? i : {source: i, boundingBox: null};
     });
 };
 

lib/validators.js

@@ -1,11 +1,10 @@
 'use strict';
 
-const _ = require('lodash');
 const {REQUIRED_IMAGE_FIELDS, REQUIRED_BOUNDING_BOX_FIELDS} = require('./constants');
 
 const validateRequiredFields = (value, fields) => {
     [].concat(fields).forEach((field) => {
-        if (!_.hasIn(value, field)) {
+        if (!value || !Object.hasOwn(value, field)) {
             throw new TypeError(`Field "${field}" does not exist in ${JSON.stringify(value)}`);
         }
     });
@@ -23,7 +22,7 @@ const validateBoundingBoxCoords = ({boundingBox}) => {
 
 exports.validateImages = (img1, img2) => {
     [img1, img2].forEach((i) => {
-        if (Buffer.isBuffer(i) || !_.isObject(i)) {
+        if (Buffer.isBuffer(i) || i === null || typeof i !== 'object') {
             return;
         }
 

package-lock.json

@@ -9,9 +9,7 @@
   },
   "dependencies": {
     "color-diff": "^1.1.0",
-    "fs-extra": "^8.1.0",
     "js-graph-algorithms": "1.0.18",
-    "lodash": "^4.17.3",
     "nested-error-stacks": "^2.1.0",
     "parse-color": "^1.0.0",
     "sharp": "0.32.6"

package.json

@@ -1,6 +1,5 @@
 'use strict';
 
-const _ = require('lodash');
 const expect = require('chai').expect;
 const sinon = require('sinon');
 const proxyquire = require('proxyquire');
@@ -12,7 +11,7 @@ describe('IgnoreCaretComparator', () => {
     let areColorsSame;
 
     const compareImages = (pixels, comparator) => {
-        const emptyPixels = _.map(pixels, (pixelRow) => Array(pixelRow.length).fill(0));
+        const emptyPixels = pixels.map(pixelRow => Array(pixelRow.length).fill(0));
         const width = pixels[0].length;
         const height = pixels.length;
 

test/ignore-caret-comparator.js

@@ -2,7 +2,7 @@
 
 const sinon = require('sinon');
 const proxyquire = require('proxyquire');
-const fs = require('fs-extra');
+const fs = require('fs');
 
 const stubBuffer = Buffer.from([123]);
 
@@ -16,7 +16,7 @@ describe('lib/image/index.js', () => {
         mkSharpImage_ = sandbox.stub();
         image = proxyquire('../../lib/image', {'sharp': mkSharpImage_});
 
-        sandbox.stub(fs, 'readFile').resolves(stubBuffer);
+        sandbox.stub(fs.promises, 'readFile').resolves(stubBuffer);
     });
 
     afterEach(() => sandbox.restore());
@@ -30,7 +30,7 @@ describe('lib/image/index.js', () => {
 
         it('should throw error with file path and original error message at stack', async () => {
             parseError = new Error('test error');
-            fs.readFile.withArgs('/filePath').rejects(parseError);
+            fs.promises.readFile.withArgs('/filePath').rejects(parseError);
 
             const error = await assert.isRejected(image.fromFile('/filePath'));
 

test/png/index.js

@@ -1,6 +1,5 @@
 'use strict';
 
-const _ = require('lodash');
 const {validateImages} = require('../lib/validators');
 
 describe('lib/validators', () => {
@@ -27,10 +26,13 @@ describe('lib/validators', () => {
 
             ['left', 'top', 'right', 'bottom'].forEach((field) => {
                 it(`required field "${field}" does not exist in "boundingBox"`, () => {
+                    const boundingBox = {left: 0, top: 0, right: 0, bottom: 0};
+                    delete boundingBox[field];
+
                     assert.throws(() => {
                         return validateImages(
-                            {source: 'image-path', boundingBox: _.omit({left: 0, top: 0, right: 0, bottom: 0}, field)},
-                            {source: 'image-path', boundingBox: _.omit({left: 0, top: 0, right: 0, bottom: 0}, field)}
+                            {source: 'image-path', boundingBox},
+                            {source: 'image-path', boundingBox}
                         );
                     }, TypeError, `Field "${field}" does not exist`);
                 });