@e22m4u/js-repository 0.8.7 → 0.8.8

package/src/definition/model/relations/relations-definition-validator.js

@@ -15,15 +15,14 @@ export class RelationsDefinitionValidator extends Service {
   validate(modelName, relDefs) {
     if (!modelName || typeof modelName !== 'string') {
       throw new InvalidArgumentError(
-        'The first argument of RelationsDefinitionValidator.validate ' +
-          'should be a non-empty String, but %v was given.',
+        'Parameter "modelName" must be a non-empty String, but %v was given.',
         modelName,
       );
     }
     if (!relDefs || typeof relDefs !== 'object' || Array.isArray(relDefs)) {
       throw new InvalidArgumentError(
-        'The provided option "relations" of the model %v ' +
-          'should be an Object, but %v was given.',
+        'Option "relations" of the model %v ' +
+          'must be an Object, but %v was given.',
         modelName,
         relDefs,
       );
@@ -45,14 +44,13 @@ export class RelationsDefinitionValidator extends Service {
   _validateRelation(modelName, relName, relDef) {
     if (!modelName || typeof modelName !== 'string') {
       throw new InvalidArgumentError(
-        'The first argument of RelationsDefinitionValidator._validateRelation ' +
-          'should be a non-empty String, but %v was given.',
+        'Parameter "modelName" must be a non-empty String, but %v was given.',
         modelName,
       );
     }
     if (!relName || typeof relName !== 'string') {
       throw new InvalidArgumentError(
-        'The relation name of the model %v should be ' +
+        'Relation name of the model %v must be ' +
           'a non-empty String, but %v was given.',
         modelName,
         relName,
@@ -60,7 +58,7 @@ export class RelationsDefinitionValidator extends Service {
     }
     if (!relDef || typeof relDef !== 'object' || Array.isArray(relDef)) {
       throw new InvalidArgumentError(
-        'The relation %v of the model %v should be an Object, but %v was given.',
+        'Relation %v of the model %v must be an Object, but %v was given.',
         relName,
         modelName,
         relDef,
@@ -68,7 +66,7 @@ export class RelationsDefinitionValidator extends Service {
     }
     if (!relDef.type || !Object.values(RelationType).includes(relDef.type)) {
       throw new InvalidArgumentError(
-        'The relation %v of the model %v requires the option "type" ' +
+        'Relation %v of the model %v requires the option "type" ' +
           'to have one of relation types: %l, but %v was given.',
         relName,
         modelName,
@@ -117,7 +115,7 @@ export class RelationsDefinitionValidator extends Service {
       // A polymorphic "belongsTo" relation.
       if (typeof relDef.polymorphic !== 'boolean') {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v has the type "belongsTo", ' +
+          'Relation %v of the model %v has the type "belongsTo", ' +
             'so it expects the option "polymorphic" to be a Boolean, ' +
             'but %v was given.',
           relName,
@@ -127,7 +125,7 @@ export class RelationsDefinitionValidator extends Service {
       }
       if (relDef.foreignKey && typeof relDef.foreignKey !== 'string') {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v is a polymorphic "belongsTo" relation, ' +
+          'Relation %v of the model %v is a polymorphic "belongsTo" relation, ' +
             'so it expects the provided option "foreignKey" to be a String, ' +
             'but %v was given.',
           relName,
@@ -137,7 +135,7 @@ export class RelationsDefinitionValidator extends Service {
       }
       if (relDef.discriminator && typeof relDef.discriminator !== 'string') {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v is a polymorphic "belongsTo" relation, ' +
+          'Relation %v of the model %v is a polymorphic "belongsTo" relation, ' +
             'so it expects the provided option "discriminator" to be a String, ' +
             'but %v was given.',
           relName,
@@ -149,7 +147,7 @@ export class RelationsDefinitionValidator extends Service {
       // A regular "belongsTo" relation.
       if (!relDef.model || typeof relDef.model !== 'string') {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v has the type "belongsTo", ' +
+          'Relation %v of the model %v has the type "belongsTo", ' +
             'so it requires the option "model" to be a non-empty String, ' +
             'but %v was given.',
           relName,
@@ -159,7 +157,7 @@ export class RelationsDefinitionValidator extends Service {
       }
       if (relDef.foreignKey && typeof relDef.foreignKey !== 'string') {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v has the type "belongsTo", ' +
+          'Relation %v of the model %v has the type "belongsTo", ' +
             'so it expects the provided option "foreignKey" to be a String, ' +
             'but %v was given.',
           relName,
@@ -169,8 +167,8 @@ export class RelationsDefinitionValidator extends Service {
       }
       if (relDef.discriminator) {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v is a non-polymorphic "belongsTo" relation, ' +
-            'so it should not have the option "discriminator" to be provided.',
+          'Relation %v of the model %v is a non-polymorphic "belongsTo" relation, ' +
+            'so it must not have the option "discriminator" to be provided.',
           relName,
           modelName,
         );
@@ -221,7 +219,7 @@ export class RelationsDefinitionValidator extends Service {
     }
     if (!relDef.model || typeof relDef.model !== 'string') {
       throw new InvalidArgumentError(
-        'The relation %v of the model %v has the type "hasOne", ' +
+        'Relation %v of the model %v has the type "hasOne", ' +
           'so it requires the option "model" to be a non-empty String, ' +
           'but %v was given.',
         relName,
@@ -234,8 +232,8 @@ export class RelationsDefinitionValidator extends Service {
         // A polymorphic "hasOne" relation with a target relation name.
         if (relDef.foreignKey) {
           throw new InvalidArgumentError(
-            'The relation %v of the model %v has the option "polymorphic" with ' +
-              'a String value, so it should not have the option "foreignKey" ' +
+            'Relation %v of the model %v has the option "polymorphic" with ' +
+              'a String value, so it must not have the option "foreignKey" ' +
               'to be provided.',
             relName,
             modelName,
@@ -243,8 +241,8 @@ export class RelationsDefinitionValidator extends Service {
         }
         if (relDef.discriminator) {
           throw new InvalidArgumentError(
-            'The relation %v of the model %v has the option "polymorphic" with ' +
-              'a String value, so it should not have the option "discriminator" ' +
+            'Relation %v of the model %v has the option "polymorphic" with ' +
+              'a String value, so it must not have the option "discriminator" ' +
               'to be provided.',
             relName,
             modelName,
@@ -254,7 +252,7 @@ export class RelationsDefinitionValidator extends Service {
         // A polymorphic "hasOne" relation with target relation keys.
         if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string') {
           throw new InvalidArgumentError(
-            'The relation %v of the model %v has the option "polymorphic" ' +
+            'Relation %v of the model %v has the option "polymorphic" ' +
               'with "true" value, so it requires the option "foreignKey" ' +
               'to be a non-empty String, but %v was given.',
             relName,
@@ -264,7 +262,7 @@ export class RelationsDefinitionValidator extends Service {
         }
         if (!relDef.discriminator || typeof relDef.discriminator !== 'string') {
           throw new InvalidArgumentError(
-            'The relation %v of the model %v has the option "polymorphic" ' +
+            'Relation %v of the model %v has the option "polymorphic" ' +
               'with "true" value, so it requires the option "discriminator" ' +
               'to be a non-empty String, but %v was given.',
             relName,
@@ -274,7 +272,7 @@ export class RelationsDefinitionValidator extends Service {
         }
       } else {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v has the type "hasOne", ' +
+          'Relation %v of the model %v has the type "hasOne", ' +
             'so it expects the provided option "polymorphic" to be ' +
             'a String or a Boolean, but %v was given.',
           relName,
@@ -286,7 +284,7 @@ export class RelationsDefinitionValidator extends Service {
       // A regular "hasOne" relation.
       if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string') {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v has the type "hasOne", ' +
+          'Relation %v of the model %v has the type "hasOne", ' +
             'so it requires the option "foreignKey" to be a non-empty String, ' +
             'but %v was given.',
           relName,
@@ -296,8 +294,8 @@ export class RelationsDefinitionValidator extends Service {
       }
       if (relDef.discriminator) {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v is a non-polymorphic "hasOne" relation, ' +
-            'so it should not have the option "discriminator" to be provided.',
+          'Relation %v of the model %v is a non-polymorphic "hasOne" relation, ' +
+            'so it must not have the option "discriminator" to be provided.',
           relName,
           modelName,
         );
@@ -348,7 +346,7 @@ export class RelationsDefinitionValidator extends Service {
     }
     if (!relDef.model || typeof relDef.model !== 'string') {
       throw new InvalidArgumentError(
-        'The relation %v of the model %v has the type "hasMany", ' +
+        'Relation %v of the model %v has the type "hasMany", ' +
           'so it requires the option "model" to be a non-empty String, ' +
           'but %v was given.',
         relName,
@@ -361,8 +359,8 @@ export class RelationsDefinitionValidator extends Service {
         // A polymorphic "hasMany" relation with a target relation name.
         if (relDef.foreignKey) {
           throw new InvalidArgumentError(
-            'The relation %v of the model %v has the option "polymorphic" with ' +
-              'a String value, so it should not have the option "foreignKey" ' +
+            'Relation %v of the model %v has the option "polymorphic" with ' +
+              'a String value, so it must not have the option "foreignKey" ' +
               'to be provided.',
             relName,
             modelName,
@@ -370,8 +368,8 @@ export class RelationsDefinitionValidator extends Service {
         }
         if (relDef.discriminator) {
           throw new InvalidArgumentError(
-            'The relation %v of the model %v has the option "polymorphic" with ' +
-              'a String value, so it should not have the option "discriminator" ' +
+            'Relation %v of the model %v has the option "polymorphic" with ' +
+              'a String value, so it must not have the option "discriminator" ' +
               'to be provided.',
             relName,
             modelName,
@@ -381,7 +379,7 @@ export class RelationsDefinitionValidator extends Service {
         // A polymorphic "hasMany" relation with target relation keys.
         if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string') {
           throw new InvalidArgumentError(
-            'The relation %v of the model %v has the option "polymorphic" ' +
+            'Relation %v of the model %v has the option "polymorphic" ' +
               'with "true" value, so it requires the option "foreignKey" ' +
               'to be a non-empty String, but %v was given.',
             relName,
@@ -391,7 +389,7 @@ export class RelationsDefinitionValidator extends Service {
         }
         if (!relDef.discriminator || typeof relDef.discriminator !== 'string') {
           throw new InvalidArgumentError(
-            'The relation %v of the model %v has the option "polymorphic" ' +
+            'Relation %v of the model %v has the option "polymorphic" ' +
               'with "true" value, so it requires the option "discriminator" ' +
               'to be a non-empty String, but %v was given.',
             relName,
@@ -401,7 +399,7 @@ export class RelationsDefinitionValidator extends Service {
         }
       } else {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v has the type "hasMany", ' +
+          'Relation %v of the model %v has the type "hasMany", ' +
             'so it expects the provided option "polymorphic" to be ' +
             'a String or a Boolean, but %v was given.',
           relName,
@@ -413,7 +411,7 @@ export class RelationsDefinitionValidator extends Service {
       // A regular "hasMany" relation.
       if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string') {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v has the type "hasMany", ' +
+          'Relation %v of the model %v has the type "hasMany", ' +
             'so it requires the option "foreignKey" to be a non-empty String, ' +
             'but %v was given.',
           relName,
@@ -423,8 +421,8 @@ export class RelationsDefinitionValidator extends Service {
       }
       if (relDef.discriminator) {
         throw new InvalidArgumentError(
-          'The relation %v of the model %v is a non-polymorphic "hasMany" relation, ' +
-            'so it should not have the option "discriminator" to be provided.',
+          'Relation %v of the model %v is a non-polymorphic "hasMany" relation, ' +
+            'so it must not have the option "discriminator" to be provided.',
           relName,
           modelName,
         );
@@ -455,7 +453,7 @@ export class RelationsDefinitionValidator extends Service {
     }
     if (!relDef.model || typeof relDef.model !== 'string') {
       throw new InvalidArgumentError(
-        'The relation %v of the model %v has the type "referencesMany", ' +
+        'Relation %v of the model %v has the type "referencesMany", ' +
           'so it requires the option "model" to be a non-empty String, ' +
           'but %v was given.',
         relName,
@@ -465,7 +463,7 @@ export class RelationsDefinitionValidator extends Service {
     }
     if (relDef.foreignKey && typeof relDef.foreignKey !== 'string') {
       throw new InvalidArgumentError(
-        'The relation %v of the model %v has the type "referencesMany", ' +
+        'Relation %v of the model %v has the type "referencesMany", ' +
           'so it expects the provided option "foreignKey" to be a String, ' +
           'but %v was given.',
         relName,
@@ -475,8 +473,8 @@ export class RelationsDefinitionValidator extends Service {
     }
     if (relDef.discriminator) {
       throw new InvalidArgumentError(
-        'The relation %v of the model %v has the type "referencesMany", ' +
-          'so it should not have the option "discriminator" to be provided.',
+        'Relation %v of the model %v has the type "referencesMany", ' +
+          'so it must not have the option "discriminator" to be provided.',
         relName,
         modelName,
       );

package/src/definition/model/relations/relations-definition-validator.spec.js

@@ -7,12 +7,11 @@ const S = new RelationsDefinitionValidator();
 
 describe('RelationsDefinitionValidator', function () {
   describe('validate', function () {
-    it('requires a first argument to be a non-empty string', function () {
+    it('requires the parameter "modelName" to be a non-empty string', function () {
       const validate = v => () => S.validate(v, {});
       const error = v =>
         format(
-          'The first argument of RelationsDefinitionValidator.validate ' +
-            'should be a non-empty String, but %s was given.',
+          'Parameter "modelName" must be a non-empty String, but %s was given.',
           v,
         );
       expect(validate('')).to.throw(error('""'));
@@ -26,12 +25,12 @@ describe('RelationsDefinitionValidator', function () {
       validate('model')();
     });
 
-    it('requires a second argument to be an object', function () {
+    it('requires the parameter "relDefs" to be an object', function () {
       const validate = v => () => S.validate('model', v);
       const error = v =>
         format(
-          'The provided option "relations" of the model "model" ' +
-            'should be an Object, but %s was given.',
+          'Option "relations" of the model "model" ' +
+            'must be an Object, but %s was given.',
           v,
         );
       expect(validate('str')).to.throw(error('"str"'));
@@ -48,7 +47,7 @@ describe('RelationsDefinitionValidator', function () {
       const validate = v => () => S.validate('model', v);
       const error = v =>
         format(
-          'The relation name of the model "model" should be ' +
+          'Relation name of the model "model" must be ' +
             'a non-empty String, but %s was given.',
           v,
         );
@@ -60,8 +59,8 @@ describe('RelationsDefinitionValidator', function () {
       const validate = v => () => S.validate('model', {foo: v});
       const error = v =>
         format(
-          'The relation "foo" of the model "model" should ' +
-            'be an Object, but %s was given.',
+          'Relation "foo" of the model "model" must be an Object, ' +
+            'but %s was given.',
           v,
         );
       expect(validate('str')).to.throw(error('"str"'));
@@ -81,7 +80,7 @@ describe('RelationsDefinitionValidator', function () {
       };
       const error = v =>
         format(
-          'The relation "foo" of the model "model" requires the option "type" ' +
+          'Relation "foo" of the model "model" requires the option "type" ' +
             'to have one of relation types: %l, but %s was given.',
           Object.values(RelationType),
           v,
@@ -109,7 +108,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "belongsTo", ' +
+              'Relation "foo" of the model "model" has the type "belongsTo", ' +
                 'so it requires the option "model" to be a non-empty String, ' +
                 'but %s was given.',
               v,
@@ -136,7 +135,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "belongsTo", ' +
+              'Relation "foo" of the model "model" has the type "belongsTo", ' +
                 'so it expects the provided option "foreignKey" to be a String, ' +
                 'but %s was given.',
               v,
@@ -162,8 +161,8 @@ describe('RelationsDefinitionValidator', function () {
               },
             });
           expect(throwable).to.throw(
-            'The relation "foo" of the model "model" is a non-polymorphic "belongsTo" relation, ' +
-              'so it should not have the option "discriminator" to be provided.',
+            'Relation "foo" of the model "model" is a non-polymorphic "belongsTo" relation, ' +
+              'so it must not have the option "discriminator" to be provided.',
           );
         });
       });
@@ -179,7 +178,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "belongsTo", ' +
+              'Relation "foo" of the model "model" has the type "belongsTo", ' +
                 'so it expects the option "polymorphic" to be a Boolean, ' +
                 'but %s was given.',
               v,
@@ -202,7 +201,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" is a polymorphic "belongsTo" relation, ' +
+              'Relation "foo" of the model "model" is a polymorphic "belongsTo" relation, ' +
                 'so it expects the provided option "foreignKey" to be a String, ' +
                 'but %s was given.',
               v,
@@ -229,7 +228,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" is a polymorphic "belongsTo" relation, ' +
+              'Relation "foo" of the model "model" is a polymorphic "belongsTo" relation, ' +
                 'so it expects the provided option "discriminator" to be a String, ' +
                 'but %s was given.',
               v,
@@ -260,7 +259,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "hasOne", ' +
+              'Relation "foo" of the model "model" has the type "hasOne", ' +
                 'so it requires the option "model" to be a non-empty String, ' +
                 'but %s was given.',
               v,
@@ -287,7 +286,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "hasOne", ' +
+              'Relation "foo" of the model "model" has the type "hasOne", ' +
                 'so it requires the option "foreignKey" to be a non-empty String, ' +
                 'but %s was given.',
               v,
@@ -314,8 +313,8 @@ describe('RelationsDefinitionValidator', function () {
               },
             });
           expect(throwable).to.throw(
-            'The relation "foo" of the model "model" is a non-polymorphic "hasOne" relation, ' +
-              'so it should not have the option "discriminator" to be provided.',
+            'Relation "foo" of the model "model" is a non-polymorphic "hasOne" relation, ' +
+              'so it must not have the option "discriminator" to be provided.',
           );
         });
       });
@@ -332,7 +331,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "hasOne", ' +
+              'Relation "foo" of the model "model" has the type "hasOne", ' +
                 'so it requires the option "model" to be a non-empty String, ' +
                 'but %s was given.',
               v,
@@ -359,8 +358,8 @@ describe('RelationsDefinitionValidator', function () {
               },
             });
           expect(throwable).to.throw(
-            'The relation "foo" of the model "model" has the option "polymorphic" with ' +
-              'a String value, so it should not have the option "foreignKey" ' +
+            'Relation "foo" of the model "model" has the option "polymorphic" with ' +
+              'a String value, so it must not have the option "foreignKey" ' +
               'to be provided.',
           );
         });
@@ -376,8 +375,8 @@ describe('RelationsDefinitionValidator', function () {
               },
             });
           expect(throwable).to.throw(
-            'The relation "foo" of the model "model" has the option "polymorphic" with ' +
-              'a String value, so it should not have the option "discriminator" ' +
+            'Relation "foo" of the model "model" has the option "polymorphic" with ' +
+              'a String value, so it must not have the option "discriminator" ' +
               'to be provided.',
           );
         });
@@ -397,7 +396,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "hasOne", ' +
+              'Relation "foo" of the model "model" has the type "hasOne", ' +
                 'so it requires the option "model" to be a non-empty String, ' +
                 'but %s was given.',
               v,
@@ -426,7 +425,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the option "polymorphic" ' +
+              'Relation "foo" of the model "model" has the option "polymorphic" ' +
                 'with "true" value, so it requires the option "foreignKey" ' +
                 'to be a non-empty String, but %s was given.',
               v,
@@ -455,7 +454,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the option "polymorphic" ' +
+              'Relation "foo" of the model "model" has the option "polymorphic" ' +
                 'with "true" value, so it requires the option "discriminator" ' +
                 'to be a non-empty String, but %s was given.',
               v,
@@ -486,7 +485,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "hasMany", ' +
+              'Relation "foo" of the model "model" has the type "hasMany", ' +
                 'so it requires the option "model" to be a non-empty String, ' +
                 'but %s was given.',
               v,
@@ -513,7 +512,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "hasMany", ' +
+              'Relation "foo" of the model "model" has the type "hasMany", ' +
                 'so it requires the option "foreignKey" to be a non-empty String, ' +
                 'but %s was given.',
               v,
@@ -540,8 +539,8 @@ describe('RelationsDefinitionValidator', function () {
               },
             });
           expect(throwable).to.throw(
-            'The relation "foo" of the model "model" is a non-polymorphic "hasMany" ' +
-              'relation, so it should not have the option "discriminator" to be provided.',
+            'Relation "foo" of the model "model" is a non-polymorphic "hasMany" ' +
+              'relation, so it must not have the option "discriminator" to be provided.',
           );
         });
       });
@@ -558,7 +557,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "hasMany", ' +
+              'Relation "foo" of the model "model" has the type "hasMany", ' +
                 'so it requires the option "model" to be a non-empty String, ' +
                 'but %s was given.',
               v,
@@ -585,8 +584,8 @@ describe('RelationsDefinitionValidator', function () {
               },
             });
           expect(throwable).to.throw(
-            'The relation "foo" of the model "model" has the option "polymorphic" ' +
-              'with a String value, so it should not have the option "foreignKey" ' +
+            'Relation "foo" of the model "model" has the option "polymorphic" ' +
+              'with a String value, so it must not have the option "foreignKey" ' +
               'to be provided.',
           );
         });
@@ -602,8 +601,8 @@ describe('RelationsDefinitionValidator', function () {
               },
             });
           expect(throwable).to.throw(
-            'The relation "foo" of the model "model" has the option "polymorphic" with ' +
-              'a String value, so it should not have the option "discriminator" ' +
+            'Relation "foo" of the model "model" has the option "polymorphic" with ' +
+              'a String value, so it must not have the option "discriminator" ' +
               'to be provided.',
           );
         });
@@ -623,7 +622,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the type "hasMany", ' +
+              'Relation "foo" of the model "model" has the type "hasMany", ' +
                 'so it requires the option "model" to be a non-empty String, ' +
                 'but %s was given.',
               v,
@@ -652,7 +651,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the option "polymorphic" ' +
+              'Relation "foo" of the model "model" has the option "polymorphic" ' +
                 'with "true" value, so it requires the option "foreignKey" ' +
                 'to be a non-empty String, but %s was given.',
               v,
@@ -681,7 +680,7 @@ describe('RelationsDefinitionValidator', function () {
           };
           const error = v =>
             format(
-              'The relation "foo" of the model "model" has the option "polymorphic" ' +
+              'Relation "foo" of the model "model" has the option "polymorphic" ' +
                 'with "true" value, so it requires the option "discriminator" ' +
                 'to be a non-empty String, but %s was given.',
               v,
@@ -710,7 +709,7 @@ describe('RelationsDefinitionValidator', function () {
         };
         const error = v =>
           format(
-            'The relation "foo" of the model "model" has the type "referencesMany", ' +
+            'Relation "foo" of the model "model" has the type "referencesMany", ' +
               'so it requires the option "model" to be a non-empty String, ' +
               'but %s was given.',
             v,
@@ -737,7 +736,7 @@ describe('RelationsDefinitionValidator', function () {
         };
         const error = v =>
           format(
-            'The relation "foo" of the model "model" has the type "referencesMany", ' +
+            'Relation "foo" of the model "model" has the type "referencesMany", ' +
               'so it expects the provided option "foreignKey" to be a String, ' +
               'but %s was given.',
             v,
@@ -763,8 +762,8 @@ describe('RelationsDefinitionValidator', function () {
             },
           });
         expect(throwable).to.throw(
-          'The relation "foo" of the model "model" has the type "referencesMany", ' +
-            'so it should not have the option "discriminator" to be provided.',
+          'Relation "foo" of the model "model" has the type "referencesMany", ' +
+            'so it must not have the option "discriminator" to be provided.',
         );
       });
     });

package/src/errors/invalid-operator-value-error.js

@@ -14,7 +14,7 @@ export class InvalidOperatorValueError extends Error {
   constructor(operator, expected, value) {
     super(
       format(
-        'Condition of {%s: ...} should have %s, but %v was given.',
+        'Condition of {%s: ...} must have %s, but %v was given.',
         operator,
         expected,
         value,

package/src/errors/invalid-operator-value-error.spec.js

@@ -5,7 +5,7 @@ describe('InvalidOperatorValueError', function () {
   it('sets specific message', function () {
     const error = new InvalidOperatorValueError('exists', 'a boolean', '');
     const message =
-      'Condition of {exists: ...} should have a boolean, but "" was given.';
+      'Condition of {exists: ...} must have a boolean, but "" was given.';
     expect(error.message).to.be.eq(message);
   });
 });