@e22m4u/js-repository 0.8.5 → 0.8.6

package/src/relations/has-many-resolver.js

@@ -27,42 +27,48 @@ export class HasManyResolver extends Service {
     foreignKey,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of HasManyResolver.includeTo requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of HasManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of HasManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of HasManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (!foreignKey || typeof foreignKey !== 'string')
+    }
+    if (!foreignKey || typeof foreignKey !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "foreignKey" of HasManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         foreignKey,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of HasManyResolver.includeTo ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
 
     const sourcePkPropName =
       this.getService(ModelDefinitionUtils).getPrimaryKeyAsPropertyName(
@@ -70,14 +76,17 @@ export class HasManyResolver extends Service {
       );
     const sourceIds = [];
     entities.forEach(entity => {
-      if (!entity || typeof entity !== 'object' || Array.isArray(entity))
+      if (!entity || typeof entity !== 'object' || Array.isArray(entity)) {
         throw new InvalidArgumentError(
           'The parameter "entities" of HasManyResolver.includeTo requires ' +
             'an Array of Object, but %v was given.',
           entity,
         );
+      }
       const sourceId = entity[sourcePkPropName];
-      if (sourceIds.includes(sourceId)) return;
+      if (sourceIds.includes(sourceId)) {
+        return;
+      }
       sourceIds.push(sourceId);
     });
 
@@ -130,48 +139,55 @@ export class HasManyResolver extends Service {
     discriminator,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of HasManyResolver.includePolymorphicTo requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of HasManyResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of HasManyResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of HasManyResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (!foreignKey || typeof foreignKey !== 'string')
+    }
+    if (!foreignKey || typeof foreignKey !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "foreignKey" of HasManyResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         foreignKey,
       );
-    if (!discriminator || typeof discriminator !== 'string')
+    }
+    if (!discriminator || typeof discriminator !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "discriminator" of HasManyResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         discriminator,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of HasManyResolver.includePolymorphicTo ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
 
     const sourcePkPropName =
       this.getService(ModelDefinitionUtils).getPrimaryKeyAsPropertyName(
@@ -179,14 +195,17 @@ export class HasManyResolver extends Service {
       );
     const sourceIds = [];
     entities.forEach(entity => {
-      if (!entity || typeof entity !== 'object' || Array.isArray(entity))
+      if (!entity || typeof entity !== 'object' || Array.isArray(entity)) {
         throw new InvalidArgumentError(
           'The parameter "entities" of HasManyResolver.includePolymorphicTo requires ' +
             'an Array of Object, but %v was given.',
           entity,
         );
+      }
       const sourceId = entity[sourcePkPropName];
-      if (sourceIds.includes(sourceId)) return;
+      if (sourceIds.includes(sourceId)) {
+        return;
+      }
       sourceIds.push(sourceId);
     });
 
@@ -240,47 +259,53 @@ export class HasManyResolver extends Service {
     targetRelationName,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of HasManyResolver.includePolymorphicByRelationName requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of HasManyResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of HasManyResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of HasManyResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (!targetRelationName || typeof targetRelationName !== 'string')
+    }
+    if (!targetRelationName || typeof targetRelationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetRelationName" of HasManyResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         targetRelationName,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of HasManyResolver.includePolymorphicByRelationName ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
 
     const targetRelationDef = this.getService(
       ModelDefinitionUtils,
     ).getRelationDefinitionByName(targetName, targetRelationName);
-    if (targetRelationDef.type !== RelationType.BELONGS_TO)
+    if (targetRelationDef.type !== RelationType.BELONGS_TO) {
       throw new InvalidArgumentError(
         'The relation %v of the model %v is a polymorphic "hasMany" relation, ' +
           'so it requires the target relation %v to be a polymorphic "belongsTo", ' +
@@ -290,7 +315,8 @@ export class HasManyResolver extends Service {
         targetRelationName,
         targetRelationDef.type,
       );
-    if (!targetRelationDef.polymorphic)
+    }
+    if (!targetRelationDef.polymorphic) {
       throw new InvalidArgumentError(
         'The relation %v of the model %v is a polymorphic "hasMany" relation, ' +
           'so it requires the target relation %v to be a polymorphic too.',
@@ -298,6 +324,7 @@ export class HasManyResolver extends Service {
         sourceName,
         targetRelationName,
       );
+    }
     const foreignKey =
       targetRelationDef.foreignKey || `${targetRelationName}Id`;
     const discriminator =

package/src/relations/has-one-resolver.js

@@ -27,42 +27,48 @@ export class HasOneResolver extends Service {
     foreignKey,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of HasOneResolver.includeTo requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of HasOneResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of HasOneResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of HasOneResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (!foreignKey || typeof foreignKey !== 'string')
+    }
+    if (!foreignKey || typeof foreignKey !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "foreignKey" of HasOneResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         foreignKey,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of HasOneResolver.includeTo ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
 
     const sourcePkPropName =
       this.getService(ModelDefinitionUtils).getPrimaryKeyAsPropertyName(
@@ -70,14 +76,17 @@ export class HasOneResolver extends Service {
       );
     const sourceIds = [];
     entities.forEach(entity => {
-      if (!entity || typeof entity !== 'object' || Array.isArray(entity))
+      if (!entity || typeof entity !== 'object' || Array.isArray(entity)) {
         throw new InvalidArgumentError(
           'The parameter "entities" of HasOneResolver.includeTo requires ' +
             'an Array of Object, but %v was given.',
           entity,
         );
+      }
       const sourceId = entity[sourcePkPropName];
-      if (sourceIds.includes(sourceId)) return;
+      if (sourceIds.includes(sourceId)) {
+        return;
+      }
       sourceIds.push(sourceId);
     });
 
@@ -94,7 +103,9 @@ export class HasOneResolver extends Service {
       filter.limit = 1;
       promises.push(
         targetRepository.find(filter).then(result => {
-          if (result.length) targetBySourceId.set(sourceId, result[0]);
+          if (result.length) {
+            targetBySourceId.set(sourceId, result[0]);
+          }
         }),
       );
     });
@@ -127,48 +138,55 @@ export class HasOneResolver extends Service {
     discriminator,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of HasOneResolver.includePolymorphicTo requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (!foreignKey || typeof foreignKey !== 'string')
+    }
+    if (!foreignKey || typeof foreignKey !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "foreignKey" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         foreignKey,
       );
-    if (!discriminator || typeof discriminator !== 'string')
+    }
+    if (!discriminator || typeof discriminator !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "discriminator" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         discriminator,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of HasOneResolver.includePolymorphicTo ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
 
     const sourcePkPropName =
       this.getService(ModelDefinitionUtils).getPrimaryKeyAsPropertyName(
@@ -176,14 +194,17 @@ export class HasOneResolver extends Service {
       );
     const sourceIds = [];
     entities.forEach(entity => {
-      if (!entity || typeof entity !== 'object' || Array.isArray(entity))
+      if (!entity || typeof entity !== 'object' || Array.isArray(entity)) {
         throw new InvalidArgumentError(
           'The parameter "entities" of HasOneResolver.includePolymorphicTo requires ' +
             'an Array of Object, but %v was given.',
           entity,
         );
+      }
       const sourceId = entity[sourcePkPropName];
-      if (sourceIds.includes(sourceId)) return;
+      if (sourceIds.includes(sourceId)) {
+        return;
+      }
       sourceIds.push(sourceId);
     });
 
@@ -203,7 +224,9 @@ export class HasOneResolver extends Service {
       filter.limit = 1;
       promises.push(
         targetRepository.find(filter).then(result => {
-          if (result.length) targetBySourceId.set(sourceId, result[0]);
+          if (result.length) {
+            targetBySourceId.set(sourceId, result[0]);
+          }
         }),
       );
     });
@@ -234,47 +257,53 @@ export class HasOneResolver extends Service {
     targetRelationName,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (!targetRelationName || typeof targetRelationName !== 'string')
+    }
+    if (!targetRelationName || typeof targetRelationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetRelationName" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         targetRelationName,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of HasOneResolver.includePolymorphicByRelationName ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
 
     const targetRelationDef = this.getService(
       ModelDefinitionUtils,
     ).getRelationDefinitionByName(targetName, targetRelationName);
-    if (targetRelationDef.type !== RelationType.BELONGS_TO)
+    if (targetRelationDef.type !== RelationType.BELONGS_TO) {
       throw new InvalidArgumentError(
         'The relation %v of the model %v is a polymorphic "hasOne" relation, ' +
           'so it requires the target relation %v to be a polymorphic "belongsTo", ' +
@@ -284,7 +313,8 @@ export class HasOneResolver extends Service {
         targetRelationName,
         targetRelationDef.type,
       );
-    if (!targetRelationDef.polymorphic)
+    }
+    if (!targetRelationDef.polymorphic) {
       throw new InvalidArgumentError(
         'The relation %v of the model %v is a polymorphic "hasOne" relation, ' +
           'so it requires the target relation %v to be a polymorphic too.',
@@ -292,6 +322,7 @@ export class HasOneResolver extends Service {
         sourceName,
         targetRelationName,
       );
+    }
     const foreignKey =
       targetRelationDef.foreignKey || `${targetRelationName}Id`;
     const discriminator =

package/src/relations/references-many-resolver.js

@@ -27,59 +27,69 @@ export class ReferencesManyResolver extends Service {
     foreignKey = undefined,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of ReferencesManyResolver.includeTo requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of ReferencesManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of ReferencesManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of ReferencesManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (foreignKey && typeof foreignKey !== 'string')
+    }
+    if (foreignKey && typeof foreignKey !== 'string') {
       throw new InvalidArgumentError(
         'The provided parameter "foreignKey" of ReferencesManyResolver.includeTo ' +
           'should be a String, but %v was given.',
         foreignKey,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of ReferencesManyResolver.includeTo ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
     if (foreignKey == null) {
       const singularRelationName = singularize(relationName);
       foreignKey = `${singularRelationName}Ids`;
     }
     const targetIds = entities.reduce((acc, entity) => {
-      if (!entity || typeof entity !== 'object' || Array.isArray(entity))
+      if (!entity || typeof entity !== 'object' || Array.isArray(entity)) {
         throw new InvalidArgumentError(
           'The parameter "entities" of ReferencesManyResolver.includeTo requires ' +
             'an Array of Object, but %v was given.',
           entity,
         );
+      }
       const ids = entity[foreignKey];
-      if (Array.isArray(ids))
+      if (Array.isArray(ids)) {
         ids.forEach(id => {
-          if (id == null || acc.includes(id)) return;
+          if (id == null || acc.includes(id)) {
+            return;
+          }
           acc.push(id);
         });
+      }
       return acc;
     }, []);
 
@@ -102,11 +112,14 @@ export class ReferencesManyResolver extends Service {
     entities.forEach(entity => {
       const ids = entity[foreignKey];
       entity[relationName] = [];
-      if (Array.isArray(ids))
+      if (Array.isArray(ids)) {
         targets.forEach(target => {
           const targetId = target[targetPkPropName];
-          if (ids.includes(targetId)) entity[relationName].push(target);
+          if (ids.includes(targetId)) {
+            entity[relationName].push(target);
+          }
         });
+      }
     });
   }
 }

package/src/repository/repository-registry.js

@@ -51,7 +51,9 @@ export class RepositoryRegistry extends Service {
   getRepository(modelName) {
     const modelKey = modelNameToModelKey(modelName);
     let repository = this._repositories[modelKey];
-    if (repository) return repository;
+    if (repository) {
+      return repository;
+    }
     repository = new this._repositoryCtor(this.container, modelName);
     this._repositories[modelKey] = repository;
     return repository;

package/src/repository/repository.js

@@ -51,11 +51,12 @@ export class Repository extends Service {
     this._modelName = modelName;
     const modelDef = this.getService(DefinitionRegistry).getModel(modelName);
     const datasourceName = modelDef.datasource;
-    if (!datasourceName)
+    if (!datasourceName) {
       throw new InvalidArgumentError(
         'The model %v does not have a specified datasource.',
         modelName,
       );
+    }
     this._datasourceName = datasourceName;
   }
 

package/src/utils/capitalize.js

@@ -5,6 +5,8 @@
  * @returns {string}
  */
 export function capitalize(string) {
-  if (!string || typeof string !== 'string') return string;
+  if (!string || typeof string !== 'string') {
+    return string;
+  }
   return string.charAt(0).toUpperCase() + string.slice(1);
 }

package/src/utils/clone-deep.js

@@ -6,7 +6,9 @@
  * @returns {*}
  */
 export function cloneDeep(value) {
-  if (!value) return value; // null, undefined values check
+  if (!value) {
+    return value;
+  } // null, undefined values check
 
   const types = [Number, String, Boolean];
   let result;
@@ -14,7 +16,9 @@ export function cloneDeep(value) {
   // normalizing primitives if someone did new String('aaa'),
   // or new Number('444');
   types.forEach(type => {
-    if (value instanceof type) result = type(value);
+    if (value instanceof type) {
+      result = type(value);
+    }
   });
 
   if (result === undefined) {

package/src/utils/exclude-object-keys.js

@@ -8,11 +8,12 @@ import {InvalidArgumentError} from '../errors/index.js';
  * @returns {object}
  */
 export function excludeObjectKeys(obj, keys) {
-  if (typeof obj !== 'object' || !obj || Array.isArray(obj))
+  if (typeof obj !== 'object' || !obj || Array.isArray(obj)) {
     throw new InvalidArgumentError(
       'Cannot exclude keys from a non-Object value, %v was given.',
       obj,
     );
+  }
   const result = {...obj};
   keys = Array.isArray(keys) ? keys : [keys];
   keys.forEach(key => delete result[key]);