@e22m4u/js-repository 0.8.4 → 0.8.6

package/src/definition/model/properties/required-property-validator.js

@@ -1,6 +1,5 @@
 import {DataType} from './data-type.js';
 import {Service} from '@e22m4u/js-service';
-import {BlankValuesService} from '@e22m4u/js-empty-values';
 import {InvalidArgumentError} from '../../../errors/index.js';
 import {ModelDefinitionUtils} from '../model-definition-utils.js';
 
@@ -44,7 +43,6 @@ export class RequiredPropertyValidator extends Service {
         ModelDefinitionUtils,
       ).getPropertiesDefinitionInBaseModelHierarchy(modelName);
     const propNames = Object.keys(isPartial ? modelData : propDefs);
-    const blankValuesService = this.getService(BlankValuesService);
     for (const propName of propNames) {
       const propDef = propDefs[propName];
       if (!propDef || typeof propDef !== 'object') {
@@ -52,16 +50,13 @@ export class RequiredPropertyValidator extends Service {
       }
       // проверка основного значения
       const propValue = modelData[propName];
-      if (propDef.required) {
-        const propType = propDef.type || DataType.ANY;
-        if (blankValuesService.isBlankOf(propType, propValue)) {
-          throw new InvalidArgumentError(
-            'Property %v of the model %v is required, but %v was given.',
-            propName,
-            modelName,
-            propValue,
-          );
-        }
+      if (propDef.required && propValue == null) {
+        throw new InvalidArgumentError(
+          'Property %v of the model %v is required, but %v was given.',
+          propName,
+          modelName,
+          propValue,
+        );
       }
       // проверка вложенного объекта
       if (

package/src/definition/model/properties/required-property-validator.spec.js

@@ -1,9 +1,8 @@
 import {expect} from 'chai';
+import {DataType} from './data-type.js';
 import {format} from '@e22m4u/js-format';
 import {DatabaseSchema} from '../../../database-schema.js';
 import {RequiredPropertyValidator} from './required-property-validator.js';
-import {DataType} from './data-type.js';
-import {BlankValuesService} from '@e22m4u/js-empty-values';
 
 describe('RequiredPropertyValidator', function () {
   describe('validate', function () {
@@ -84,11 +83,9 @@ describe('RequiredPropertyValidator', function () {
       S.validate('model', {foo: 'bar'});
     });
 
-    it('should not throw an error if a required property is not blank', function () {
+    it('should not throw an error if a required property is defined', function () {
       const dbs = new DatabaseSchema();
       const S = dbs.getService(RequiredPropertyValidator);
-      const blankValues = S.getService(BlankValuesService);
-      blankValues.setBlankValues([undefined]);
       dbs.defineModel({
         name: 'model',
         properties: {
@@ -101,11 +98,9 @@ describe('RequiredPropertyValidator', function () {
       S.validate('model', {foo: 'bar'});
     });
 
-    it('should throw an error if a required property is blank', function () {
+    it('should throw an error if a required property is undefined', function () {
       const dbs = new DatabaseSchema();
       const S = dbs.getService(RequiredPropertyValidator);
-      const blankValues = S.getService(BlankValuesService);
-      blankValues.setBlankValues([undefined]);
       dbs.defineModel({
         name: 'model',
         properties: {
@@ -126,8 +121,6 @@ describe('RequiredPropertyValidator', function () {
       it('should not throw an error if no data is provided for an embedded model', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -151,8 +144,6 @@ describe('RequiredPropertyValidator', function () {
       it('should throw an error if an embedded model is required but not provided', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -181,8 +172,6 @@ describe('RequiredPropertyValidator', function () {
       it('should allow a model data to have properties without a specified schema', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -206,8 +195,6 @@ describe('RequiredPropertyValidator', function () {
       it('should allow omit a model data when its model has a required property', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -232,8 +219,6 @@ describe('RequiredPropertyValidator', function () {
       it('should allow omit an optional property for an embedded model', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -257,8 +242,6 @@ describe('RequiredPropertyValidator', function () {
       it('should throw an error if a required property is not provided', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -289,8 +272,6 @@ describe('RequiredPropertyValidator', function () {
       it('should allow omit an optional array', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -315,8 +296,6 @@ describe('RequiredPropertyValidator', function () {
       it('should allow a required array to be empty', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -339,11 +318,9 @@ describe('RequiredPropertyValidator', function () {
         S.validate('modelA', {array: []});
       });
 
-      it('should allow omit an optional array even if the item model has a required property', function () {
+      it('should allow omit an optional array even if an item model has a required property', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -366,11 +343,9 @@ describe('RequiredPropertyValidator', function () {
         S.validate('modelA', {});
       });
 
-      it('should allow an empty array even if the item model has a required property', function () {
+      it('should allow an empty array even if an item model has a required property', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -396,8 +371,6 @@ describe('RequiredPropertyValidator', function () {
       it('should throw an error when a required array is not provided', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -427,8 +400,6 @@ describe('RequiredPropertyValidator', function () {
       it('should allow omit an optional property of the item model', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -454,8 +425,6 @@ describe('RequiredPropertyValidator', function () {
       it('should allow an item date to have properties without a specified schema', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -480,11 +449,9 @@ describe('RequiredPropertyValidator', function () {
     });
 
     describe('isPartial', function () {
-      it('should throw an error if a required property is blank', function () {
+      it('should throw an error if a required property is undefined', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'model',
           properties: {
@@ -501,11 +468,9 @@ describe('RequiredPropertyValidator', function () {
         );
       });
 
-      it('should not validate a required but not provided properties', function () {
+      it('should not validate required but not provided properties', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'model',
           properties: {
@@ -521,8 +486,6 @@ describe('RequiredPropertyValidator', function () {
       it('should validate not provided properties of an embedded model', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {
@@ -551,8 +514,6 @@ describe('RequiredPropertyValidator', function () {
       it('should validate not provided properties of an item model', function () {
         const dbs = new DatabaseSchema();
         const S = dbs.getService(RequiredPropertyValidator);
-        const blankValues = S.getService(BlankValuesService);
-        blankValues.setBlankValues([undefined]);
         dbs.defineModel({
           name: 'modelA',
           properties: {

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

@@ -13,19 +13,21 @@ export class RelationsDefinitionValidator extends Service {
    * @param {object} relDefs
    */
   validate(modelName, relDefs) {
-    if (!modelName || typeof modelName !== 'string')
+    if (!modelName || typeof modelName !== 'string') {
       throw new InvalidArgumentError(
         'The first argument of RelationsDefinitionValidator.validate ' +
           'should be a non-empty String, but %v was given.',
         modelName,
       );
-    if (!relDefs || typeof relDefs !== 'object' || Array.isArray(relDefs))
+    }
+    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.',
         modelName,
         relDefs,
       );
+    }
     const relNames = Object.keys(relDefs);
     relNames.forEach(relName => {
       const relDef = relDefs[relName];
@@ -41,27 +43,30 @@ export class RelationsDefinitionValidator extends Service {
    * @param {object} relDef
    */
   _validateRelation(modelName, relName, relDef) {
-    if (!modelName || typeof modelName !== 'string')
+    if (!modelName || typeof modelName !== 'string') {
       throw new InvalidArgumentError(
         'The first argument of RelationsDefinitionValidator._validateRelation ' +
           'should be a non-empty String, but %v was given.',
         modelName,
       );
-    if (!relName || typeof relName !== 'string')
+    }
+    if (!relName || typeof relName !== 'string') {
       throw new InvalidArgumentError(
         'The relation name of the model %v should be ' +
           'a non-empty String, but %v was given.',
         modelName,
         relName,
       );
-    if (!relDef || typeof relDef !== 'object' || Array.isArray(relDef))
+    }
+    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.',
         relName,
         modelName,
         relDef,
       );
-    if (!relDef.type || !Object.values(RelationType).includes(relDef.type))
+    }
+    if (!relDef.type || !Object.values(RelationType).includes(relDef.type)) {
       throw new InvalidArgumentError(
         'The relation %v of the model %v requires the option "type" ' +
           'to have one of relation types: %l, but %v was given.',
@@ -70,6 +75,7 @@ export class RelationsDefinitionValidator extends Service {
         Object.values(RelationType),
         relDef.type,
       );
+    }
     this._validateBelongsTo(modelName, relName, relDef);
     this._validateHasOne(modelName, relName, relDef);
     this._validateHasMany(modelName, relName, relDef);
@@ -104,10 +110,12 @@ export class RelationsDefinitionValidator extends Service {
    * @private
    */
   _validateBelongsTo(modelName, relName, relDef) {
-    if (relDef.type !== RelationType.BELONGS_TO) return;
+    if (relDef.type !== RelationType.BELONGS_TO) {
+      return;
+    }
     if (relDef.polymorphic) {
       // A polymorphic "belongsTo" relation.
-      if (typeof relDef.polymorphic !== 'boolean')
+      if (typeof relDef.polymorphic !== 'boolean') {
         throw new InvalidArgumentError(
           'The relation %v of the model %v has the type "belongsTo", ' +
             'so it expects the option "polymorphic" to be a Boolean, ' +
@@ -116,7 +124,8 @@ export class RelationsDefinitionValidator extends Service {
           modelName,
           relDef.polymorphic,
         );
-      if (relDef.foreignKey && typeof relDef.foreignKey !== 'string')
+      }
+      if (relDef.foreignKey && typeof relDef.foreignKey !== 'string') {
         throw new InvalidArgumentError(
           'The relation %v of the model %v is a polymorphic "belongsTo" relation, ' +
             'so it expects the provided option "foreignKey" to be a String, ' +
@@ -125,7 +134,8 @@ export class RelationsDefinitionValidator extends Service {
           modelName,
           relDef.foreignKey,
         );
-      if (relDef.discriminator && typeof relDef.discriminator !== 'string')
+      }
+      if (relDef.discriminator && typeof relDef.discriminator !== 'string') {
         throw new InvalidArgumentError(
           'The relation %v of the model %v is a polymorphic "belongsTo" relation, ' +
             'so it expects the provided option "discriminator" to be a String, ' +
@@ -134,9 +144,10 @@ export class RelationsDefinitionValidator extends Service {
           modelName,
           relDef.discriminator,
         );
+      }
     } else {
       // A regular "belongsTo" relation.
-      if (!relDef.model || typeof relDef.model !== 'string')
+      if (!relDef.model || typeof relDef.model !== 'string') {
         throw new InvalidArgumentError(
           'The relation %v of the model %v has the type "belongsTo", ' +
             'so it requires the option "model" to be a non-empty String, ' +
@@ -145,7 +156,8 @@ export class RelationsDefinitionValidator extends Service {
           modelName,
           relDef.model,
         );
-      if (relDef.foreignKey && typeof relDef.foreignKey !== 'string')
+      }
+      if (relDef.foreignKey && typeof relDef.foreignKey !== 'string') {
         throw new InvalidArgumentError(
           'The relation %v of the model %v has the type "belongsTo", ' +
             'so it expects the provided option "foreignKey" to be a String, ' +
@@ -154,13 +166,15 @@ export class RelationsDefinitionValidator extends Service {
           modelName,
           relDef.foreignKey,
         );
-      if (relDef.discriminator)
+      }
+      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.',
           relName,
           modelName,
         );
+      }
     }
   }
 
@@ -202,8 +216,10 @@ export class RelationsDefinitionValidator extends Service {
    * @private
    */
   _validateHasOne(modelName, relName, relDef) {
-    if (relDef.type !== RelationType.HAS_ONE) return;
-    if (!relDef.model || typeof relDef.model !== 'string')
+    if (relDef.type !== RelationType.HAS_ONE) {
+      return;
+    }
+    if (!relDef.model || typeof relDef.model !== 'string') {
       throw new InvalidArgumentError(
         'The relation %v of the model %v has the type "hasOne", ' +
           'so it requires the option "model" to be a non-empty String, ' +
@@ -212,10 +228,11 @@ export class RelationsDefinitionValidator extends Service {
         modelName,
         relDef.model,
       );
+    }
     if (relDef.polymorphic) {
       if (typeof relDef.polymorphic === 'string') {
         // A polymorphic "hasOne" relation with a target relation name.
-        if (relDef.foreignKey)
+        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" ' +
@@ -223,7 +240,8 @@ export class RelationsDefinitionValidator extends Service {
             relName,
             modelName,
           );
-        if (relDef.discriminator)
+        }
+        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" ' +
@@ -231,9 +249,10 @@ export class RelationsDefinitionValidator extends Service {
             relName,
             modelName,
           );
+        }
       } else if (typeof relDef.polymorphic === 'boolean') {
         // A polymorphic "hasOne" relation with target relation keys.
-        if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string')
+        if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string') {
           throw new InvalidArgumentError(
             'The relation %v of the model %v has the option "polymorphic" ' +
               'with "true" value, so it requires the option "foreignKey" ' +
@@ -242,7 +261,8 @@ export class RelationsDefinitionValidator extends Service {
             modelName,
             relDef.foreignKey,
           );
-        if (!relDef.discriminator || typeof relDef.discriminator !== 'string')
+        }
+        if (!relDef.discriminator || typeof relDef.discriminator !== 'string') {
           throw new InvalidArgumentError(
             'The relation %v of the model %v has the option "polymorphic" ' +
               'with "true" value, so it requires the option "discriminator" ' +
@@ -251,6 +271,7 @@ export class RelationsDefinitionValidator extends Service {
             modelName,
             relDef.discriminator,
           );
+        }
       } else {
         throw new InvalidArgumentError(
           'The relation %v of the model %v has the type "hasOne", ' +
@@ -263,7 +284,7 @@ export class RelationsDefinitionValidator extends Service {
       }
     } else {
       // A regular "hasOne" relation.
-      if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string')
+      if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string') {
         throw new InvalidArgumentError(
           'The relation %v of the model %v has the type "hasOne", ' +
             'so it requires the option "foreignKey" to be a non-empty String, ' +
@@ -272,13 +293,15 @@ export class RelationsDefinitionValidator extends Service {
           modelName,
           relDef.foreignKey,
         );
-      if (relDef.discriminator)
+      }
+      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.',
           relName,
           modelName,
         );
+      }
     }
   }
 
@@ -320,8 +343,10 @@ export class RelationsDefinitionValidator extends Service {
    * @private
    */
   _validateHasMany(modelName, relName, relDef) {
-    if (relDef.type !== RelationType.HAS_MANY) return;
-    if (!relDef.model || typeof relDef.model !== 'string')
+    if (relDef.type !== RelationType.HAS_MANY) {
+      return;
+    }
+    if (!relDef.model || typeof relDef.model !== 'string') {
       throw new InvalidArgumentError(
         'The relation %v of the model %v has the type "hasMany", ' +
           'so it requires the option "model" to be a non-empty String, ' +
@@ -330,10 +355,11 @@ export class RelationsDefinitionValidator extends Service {
         modelName,
         relDef.model,
       );
+    }
     if (relDef.polymorphic) {
       if (typeof relDef.polymorphic === 'string') {
         // A polymorphic "hasMany" relation with a target relation name.
-        if (relDef.foreignKey)
+        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" ' +
@@ -341,7 +367,8 @@ export class RelationsDefinitionValidator extends Service {
             relName,
             modelName,
           );
-        if (relDef.discriminator)
+        }
+        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" ' +
@@ -349,9 +376,10 @@ export class RelationsDefinitionValidator extends Service {
             relName,
             modelName,
           );
+        }
       } else if (typeof relDef.polymorphic === 'boolean') {
         // A polymorphic "hasMany" relation with target relation keys.
-        if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string')
+        if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string') {
           throw new InvalidArgumentError(
             'The relation %v of the model %v has the option "polymorphic" ' +
               'with "true" value, so it requires the option "foreignKey" ' +
@@ -360,7 +388,8 @@ export class RelationsDefinitionValidator extends Service {
             modelName,
             relDef.foreignKey,
           );
-        if (!relDef.discriminator || typeof relDef.discriminator !== 'string')
+        }
+        if (!relDef.discriminator || typeof relDef.discriminator !== 'string') {
           throw new InvalidArgumentError(
             'The relation %v of the model %v has the option "polymorphic" ' +
               'with "true" value, so it requires the option "discriminator" ' +
@@ -369,6 +398,7 @@ export class RelationsDefinitionValidator extends Service {
             modelName,
             relDef.discriminator,
           );
+        }
       } else {
         throw new InvalidArgumentError(
           'The relation %v of the model %v has the type "hasMany", ' +
@@ -381,7 +411,7 @@ export class RelationsDefinitionValidator extends Service {
       }
     } else {
       // A regular "hasMany" relation.
-      if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string')
+      if (!relDef.foreignKey || typeof relDef.foreignKey !== 'string') {
         throw new InvalidArgumentError(
           'The relation %v of the model %v has the type "hasMany", ' +
             'so it requires the option "foreignKey" to be a non-empty String, ' +
@@ -390,13 +420,15 @@ export class RelationsDefinitionValidator extends Service {
           modelName,
           relDef.foreignKey,
         );
-      if (relDef.discriminator)
+      }
+      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.',
           relName,
           modelName,
         );
+      }
     }
   }
 
@@ -418,8 +450,10 @@ export class RelationsDefinitionValidator extends Service {
    * @private
    */
   _validateReferencesMany(modelName, relName, relDef) {
-    if (relDef.type !== RelationType.REFERENCES_MANY) return;
-    if (!relDef.model || typeof relDef.model !== 'string')
+    if (relDef.type !== RelationType.REFERENCES_MANY) {
+      return;
+    }
+    if (!relDef.model || typeof relDef.model !== 'string') {
       throw new InvalidArgumentError(
         'The relation %v of the model %v has the type "referencesMany", ' +
           'so it requires the option "model" to be a non-empty String, ' +
@@ -428,7 +462,8 @@ export class RelationsDefinitionValidator extends Service {
         modelName,
         relDef.model,
       );
-    if (relDef.foreignKey && typeof relDef.foreignKey !== 'string')
+    }
+    if (relDef.foreignKey && typeof relDef.foreignKey !== 'string') {
       throw new InvalidArgumentError(
         'The relation %v of the model %v has the type "referencesMany", ' +
           'so it expects the provided option "foreignKey" to be a String, ' +
@@ -437,12 +472,14 @@ export class RelationsDefinitionValidator extends Service {
         modelName,
         relDef.foreignKey,
       );
-    if (relDef.discriminator)
+    }
+    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.',
         relName,
         modelName,
       );
+    }
   }
 }