@e22m4u/js-repository 0.0.34 → 0.0.35

package/src/filter/operator-clause-tool.js

@@ -58,7 +58,7 @@ export class OperatorClauseTool extends Service {
   testAll(clause, value) {
     if (!clause || typeof clause !== 'object' || Array.isArray(clause))
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testAll ' +
+        'The first argument of OperatorUtils.testAll ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -136,7 +136,7 @@ export class OperatorClauseTool extends Service {
   testEqNeq(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testEqNeq ' +
+        'The first argument of OperatorUtils.testEqNeq ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -182,7 +182,7 @@ export class OperatorClauseTool extends Service {
   testGtLt(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testGtLt ' +
+        'The first argument of OperatorUtils.testGtLt ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -209,7 +209,7 @@ export class OperatorClauseTool extends Service {
   testInq(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testInq ' +
+        'The first argument of OperatorUtils.testInq ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -245,7 +245,7 @@ export class OperatorClauseTool extends Service {
   testNin(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testNin ' +
+        'The first argument of OperatorUtils.testNin ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -281,7 +281,7 @@ export class OperatorClauseTool extends Service {
   testBetween(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testBetween ' +
+        'The first argument of OperatorUtils.testBetween ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -317,7 +317,7 @@ export class OperatorClauseTool extends Service {
   testExists(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testExists ' +
+        'The first argument of OperatorUtils.testExists ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -350,7 +350,7 @@ export class OperatorClauseTool extends Service {
   testLike(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testLike ' +
+        'The first argument of OperatorUtils.testLike ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -378,7 +378,7 @@ export class OperatorClauseTool extends Service {
   testNlike(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testNlike ' +
+        'The first argument of OperatorUtils.testNlike ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -410,7 +410,7 @@ export class OperatorClauseTool extends Service {
   testIlike(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testIlike ' +
+        'The first argument of OperatorUtils.testIlike ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -442,7 +442,7 @@ export class OperatorClauseTool extends Service {
   testNilike(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testNilike ' +
+        'The first argument of OperatorUtils.testNilike ' +
           'should be an Object, but %v given.',
         clause,
       );
@@ -486,7 +486,7 @@ export class OperatorClauseTool extends Service {
   testRegexp(clause, value) {
     if (!clause || typeof clause !== 'object')
       throw new InvalidArgumentError(
-        'A first argument of OperatorUtils.testRegexp ' +
+        'The first argument of OperatorUtils.testRegexp ' +
           'should be an Object, but %v given.',
         clause,
       );

package/src/filter/operator-clause-tool.spec.js

@@ -115,7 +115,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testAll(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testAll ' +
+        'The first argument of OperatorUtils.testAll ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -130,7 +130,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testEqNeq(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testEqNeq ' +
+        'The first argument of OperatorUtils.testEqNeq ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -215,7 +215,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testGtLt(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testGtLt ' +
+        'The first argument of OperatorUtils.testGtLt ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -390,7 +390,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testInq(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testInq ' +
+        'The first argument of OperatorUtils.testInq ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -445,7 +445,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testNin(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testNin ' +
+        'The first argument of OperatorUtils.testNin ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -514,7 +514,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testBetween(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testBetween ' +
+        'The first argument of OperatorUtils.testBetween ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -571,7 +571,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testExists(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testExists ' +
+        'The first argument of OperatorUtils.testExists ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -694,7 +694,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testLike(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testLike ' +
+        'The first argument of OperatorUtils.testLike ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -773,7 +773,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testNlike(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testNlike ' +
+        'The first argument of OperatorUtils.testNlike ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -852,7 +852,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testIlike(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testIlike ' +
+        'The first argument of OperatorUtils.testIlike ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -931,7 +931,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testNilike(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testNilike ' +
+        'The first argument of OperatorUtils.testNilike ' +
           'should be an Object, but 10 given.',
       );
     });
@@ -1022,7 +1022,7 @@ describe('OperatorClauseTool', function () {
     it('throws an error if a first argument is not an object', function () {
       const throwable = () => S.testRegexp(10);
       expect(throwable).to.throw(
-        'A first argument of OperatorUtils.testRegexp ' +
+        'The first argument of OperatorUtils.testRegexp ' +
           'should be an Object, but 10 given.',
       );
     });

package/src/filter/slice-clause-tool.js

@@ -16,16 +16,16 @@ export class SliceClauseTool extends Service {
   slice(entities, skip = undefined, limit = undefined) {
     if (!Array.isArray(entities))
       throw new InvalidArgumentError(
-        'A first argument of SliceClauseTool.slice ' +
+        'The first argument of SliceClauseTool.slice ' +
           'should be an Array, but %v given.',
         entities,
       );
-    if (skip && typeof skip !== 'number')
+    if (skip != null && typeof skip !== 'number')
       throw new InvalidArgumentError(
         'The provided option "skip" should be a Number, but %v given.',
         skip,
       );
-    if (limit && typeof limit !== 'number')
+    if (limit != null && typeof limit !== 'number')
       throw new InvalidArgumentError(
         'The provided option "limit" should be a Number, but %v given.',
         limit,
@@ -41,7 +41,7 @@ export class SliceClauseTool extends Service {
    * @param {number|undefined} skip
    */
   static validateSkipClause(skip) {
-    if (!skip) return;
+    if (skip == null) return;
     if (typeof skip !== 'number')
       throw new InvalidArgumentError(
         'The provided option "skip" should be a Number, but %v given.',
@@ -55,7 +55,7 @@ export class SliceClauseTool extends Service {
    * @param {number|undefined} limit
    */
   static validateLimitClause(limit) {
-    if (!limit) return;
+    if (limit == null) return;
     if (typeof limit !== 'number')
       throw new InvalidArgumentError(
         'The provided option "limit" should be a Number, but %v given.',

package/src/filter/slice-clause-tool.spec.js

@@ -5,39 +5,100 @@ import {SliceClauseTool} from './slice-clause-tool.js';
 const S = new SliceClauseTool();
 
 describe('SliceClauseTool', function () {
-  describe('filter', function () {
-    it('does nothing if no clauses provided', function () {
+  describe('slice', function () {
+    it('requires the first argument to be an array', function () {
+      const throwable = v => () => S.slice(v);
+      const error = v =>
+        format(
+          'The first argument of SliceClauseTool.slice ' +
+            'should be an Array, but %s given.',
+          v,
+        );
+      expect(throwable('str')).to.throw(error('"str"'));
+      expect(throwable('')).to.throw(error('""'));
+      expect(throwable(10)).to.throw(error('10'));
+      expect(throwable(0)).to.throw(error('0'));
+      expect(throwable(true)).to.throw(error('true'));
+      expect(throwable(false)).to.throw(error('false'));
+      expect(throwable({})).to.throw(error('Object'));
+      expect(throwable(undefined)).to.throw(error('undefined'));
+      expect(throwable(null)).to.throw(error('null'));
+      expect(throwable([{foo: 'bar'}])()).to.be.eql([{foo: 'bar'}]);
+      expect(throwable([])()).to.be.eql([]);
+    });
+
+    it('requires the provided second argument to be a number', function () {
+      const items = [{foo: 'bar'}];
+      const throwable = v => () => S.slice(items, v);
+      const error = v =>
+        format(
+          'The provided option "skip" should be a Number, but %s given.',
+          v,
+        );
+      expect(throwable('str')).to.throw(error('"str"'));
+      expect(throwable('')).to.throw(error('""'));
+      expect(throwable(true)).to.throw(error('true'));
+      expect(throwable(false)).to.throw(error('false'));
+      expect(throwable({})).to.throw(error('Object'));
+      expect(throwable([])).to.throw(error('Array'));
+      expect(throwable(10)()).to.be.eql([]);
+      expect(throwable(0)()).to.be.eql(items);
+      expect(throwable(undefined)()).to.be.eql(items);
+      expect(throwable(null)()).to.be.eql(items);
+    });
+
+    it('requires the provided third argument to be a number', function () {
+      const items = [{foo: 'bar'}];
+      const throwable = v => () => S.slice(items, undefined, v);
+      const error = v =>
+        format(
+          'The provided option "limit" should be a Number, but %s given.',
+          v,
+        );
+      expect(throwable('str')).to.throw(error('"str"'));
+      expect(throwable('')).to.throw(error('""'));
+      expect(throwable(true)).to.throw(error('true'));
+      expect(throwable(false)).to.throw(error('false'));
+      expect(throwable({})).to.throw(error('Object'));
+      expect(throwable([])).to.throw(error('Array'));
+      expect(throwable(10)()).to.be.eql(items);
+      expect(throwable(0)()).to.be.eql(items);
+      expect(throwable(undefined)()).to.be.eql(items);
+      expect(throwable(null)()).to.be.eql(items);
+    });
+
+    it('does nothing if no "skip" and "limit" options provided', function () {
       const objects = [{id: 1}, {id: 2}, {id: 3}];
       const result = S.slice(objects);
       expect(result).to.be.eql(objects);
     });
 
-    it('does nothing if a given skip is zero', function () {
+    it('does nothing if the given "skip" option is zero', function () {
       const objects = [{id: 1}, {id: 2}, {id: 3}];
       const result = S.slice(objects, 0);
       expect(result).to.be.eql(objects);
     });
 
-    it('uses a given skip to exclude array elements from start', function () {
+    it('uses the given "skip" option to exclude array elements from start', function () {
       const objects = [{id: 1}, {id: 2}, {id: 3}];
       const result = S.slice(objects, 2);
       expect(result).to.have.length(1);
       expect(result[0]).to.be.eql(objects[2]);
     });
 
-    it('returns an empty array if skipping too much', function () {
+    it('returns an empty array when "skip" option overflows a size of the given array ', function () {
       const objects = [{id: 1}, {id: 2}, {id: 3}];
       const result = S.slice(objects, 10);
       expect(result).to.have.length(0);
     });
 
-    it('does nothing if a given limit is zero', function () {
+    it('does nothing if the given "limit" option is zero', function () {
       const objects = [{id: 1}, {id: 2}, {id: 3}];
       const result = S.slice(objects, undefined, 0);
       expect(result).to.be.eql(objects);
     });
 
-    it('uses a given limit to trim a given array', function () {
+    it('uses the given "limit" option to trim the given array', function () {
       const objects = [{id: 1}, {id: 2}, {id: 3}];
       const result = S.slice(objects, undefined, 2);
       expect(result).to.have.length(2);
@@ -45,73 +106,53 @@ describe('SliceClauseTool', function () {
       expect(result[1]).to.be.eql(objects[1]);
     });
 
-    it('able to combine a skip and a limit option together', function () {
+    it('uses the "skip" and "limit" options to slice the given array', function () {
       const objects = [{id: 1}, {id: 2}, {id: 3}];
       const result = S.slice(objects, 1, 1);
       expect(result).to.have.length(1);
       expect(result[0]).to.be.eql(objects[1]);
     });
-
-    it('throws an error if a first argument is not an array', function () {
-      const throwable = () => S.slice(10);
-      expect(throwable).to.throw(
-        'A first argument of SliceClauseTool.slice ' +
-          'should be an Array, but 10 given.',
-      );
-    });
-
-    it('throws an error if the given "skip" option is not a number', function () {
-      const throwable = () => S.slice([], 'invalid');
-      expect(throwable).to.throw(
-        'The provided option "skip" should be a Number, but "invalid" given.',
-      );
-    });
-
-    it('throws an error if the given "limit" option is not a number', function () {
-      const throwable = () => S.slice([], undefined, 'invalid');
-      expect(throwable).to.throw(
-        'The provided option "limit" should be a Number, but "invalid" given.',
-      );
-    });
   });
 
   describe('validateSkipClause', function () {
-    it('requires a number value or a falsy value', function () {
-      const validate = v => () => SliceClauseTool.validateSkipClause(v);
+    it('requires a number value', function () {
+      const throwable = v => () => SliceClauseTool.validateSkipClause(v);
       const error = v =>
         format(
           'The provided option "skip" should be a Number, but %s given.',
           v,
         );
-      expect(validate('str')).to.throw(error('"str"'));
-      expect(validate(true)).to.throw(error('true'));
-      expect(validate([])).to.throw(error('Array'));
-      validate('')();
-      validate(false)();
-      validate(undefined)();
-      validate(null)();
-      validate(10)();
-      validate(0)();
+      expect(throwable('str')).to.throw(error('"str"'));
+      expect(throwable('')).to.throw(error('""'));
+      expect(throwable(true)).to.throw(error('true'));
+      expect(throwable(false)).to.throw(error('false'));
+      expect(throwable({})).to.throw(error('Object'));
+      expect(throwable([])).to.throw(error('Array'));
+      throwable(10)();
+      throwable(0)();
+      throwable(undefined)();
+      throwable(null)();
     });
   });
 
   describe('validateLimitClause', function () {
     it('requires a number value or a falsy value', function () {
-      const validate = v => () => SliceClauseTool.validateLimitClause(v);
+      const throwable = v => () => SliceClauseTool.validateLimitClause(v);
       const error = v =>
         format(
           'The provided option "limit" should be a Number, but %s given.',
           v,
         );
-      expect(validate('str')).to.throw(error('"str"'));
-      expect(validate(true)).to.throw(error('true'));
-      expect(validate([])).to.throw(error('Array'));
-      validate('')();
-      validate(false)();
-      validate(undefined)();
-      validate(null)();
-      validate(10)();
-      validate(0)();
+      expect(throwable('str')).to.throw(error('"str"'));
+      expect(throwable('')).to.throw(error('""'));
+      expect(throwable(true)).to.throw(error('true'));
+      expect(throwable(false)).to.throw(error('false'));
+      expect(throwable({})).to.throw(error('Object'));
+      expect(throwable([])).to.throw(error('Array'));
+      throwable(10)();
+      throwable(0)();
+      throwable(undefined)();
+      throwable(null)();
     });
   });
 });

package/src/filter/where-clause-tool.js

@@ -41,11 +41,11 @@ export class WhereClauseTool extends Service {
   filter(entities, where = undefined) {
     if (!Array.isArray(entities))
       throw new InvalidArgumentError(
-        'A first argument of WhereUtils.filter ' +
-          'should be an Array of Objects, but %v given.',
+        'The first argument of WhereClauseTool.filter should be ' +
+          'an Array of Object, but %v given.',
         entities,
       );
-    if (!where) return entities;
+    if (where == null) return entities;
     return entities.filter(this._createFilter(where));
   }
 
@@ -57,7 +57,7 @@ export class WhereClauseTool extends Service {
    */
   _createFilter(whereClause) {
     if (typeof whereClause === 'function') return whereClause;
-    if (typeof whereClause !== 'object')
+    if (typeof whereClause !== 'object' || Array.isArray(whereClause))
       throw new InvalidArgumentError(
         'The provided option "where" should be an Object, but %v given.',
         whereClause,
@@ -66,8 +66,8 @@ export class WhereClauseTool extends Service {
     return data => {
       if (typeof data !== 'object')
         throw new InvalidArgumentError(
-          'A first argument of WhereUtils.filter ' +
-            'should be an Array of Objects, but %v given.',
+          'The first argument of WhereClauseTool.filter should be ' +
+            'an Array of Object, but %v given.',
           data,
         );
       return keys.every(key => {
@@ -155,7 +155,7 @@ export class WhereClauseTool extends Service {
    * @param {WhereClause|undefined} clause
    */
   static validateWhereClause(clause) {
-    if (!clause) return;
+    if (clause == null || typeof clause === 'function') return;
     if (typeof clause !== 'object' || Array.isArray(clause))
       throw new InvalidArgumentError(
         'The provided option "where" should be an Object, but %v given.',