test() update unit tests, fix issue

Author: kamilmysliwiec

packages/core/services/reflector.service.ts

@@ -43,13 +43,13 @@ export class Reflector {
   }
 
   /**
-   * Retrieve metadata for a specified key for a specified set of targets and concat results.
+   * Retrieve metadata for a specified key for a specified set of targets and merge results.
    *
    * @param metadataKey lookup key for metadata to retrieve
    * @param targets context (decorated objects) to retrieve metadata from
    *
    */
-  public getAllAndConcat<TResult extends any[] = any[], TKey = any>(
+  public getAllAndMerge<TResult extends any[] = any[], TKey = any>(
     metadataKey: TKey,
     targets: (Type<any> | Function)[],
   ): TResult {
@@ -64,7 +64,7 @@ export class Reflector {
           ...b,
         };
       }
-      return b;
+      return [a, b];
     });
   }
 

packages/core/test/services/reflector.service.spec.ts

@@ -25,16 +25,33 @@ describe('Reflector', () => {
     });
   });
 
-  describe('getAllAndConcat', () => {
+  describe('getAllAndMerge', () => {
     it('should reflect metadata of all targets and concat arrays', () => {
+      const key = 'key';
+      const value = 'value';
+      Reflect.defineMetadata(key, [value], Test);
+      expect(reflector.getAllAndMerge(key, [Test, Test])).to.eql([
+        value,
+        value,
+      ]);
+    });
+    it('should reflect metadata of all targets and create an array', () => {
       const key = 'key';
       const value = 'value';
       Reflect.defineMetadata(key, value, Test);
-      expect(reflector.getAllAndConcat(key, [Test, Test])).to.eql([
+      expect(reflector.getAllAndMerge(key, [Test, Test])).to.eql([
         value,
         value,
       ]);
     });
+    it('should reflect metadata of all targets and merge an object', () => {
+      const key = 'key';
+      const value = { test: 'test' };
+      Reflect.defineMetadata(key, value, Test);
+      expect(reflector.getAllAndMerge(key, [Test, Test])).to.eql({
+        ...value,
+      });
+    });
   });
 
   describe('getAllAndOverride', () => {