fix(Tags): missing class decorator define (#1703)

* test(Tags): apply tags decorator on controller class * fix(Tags): missing class decorator definition * test(Tags): adjust test cases to fit class decorator
lukeautry · Oct 15, 2024 · 61b01cf · 61b01cf
1 parent 4abc6ee
commit 61b01cf
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/packages/runtime/src/decorators/tags.ts b/packages/runtime/src/decorators/tags.ts
@@ -1,4 +1,4 @@
-export function Tags(...values: string[]): MethodDecorator {
+export function Tags(...values: string[]): ClassDecorator & MethodDecorator {
   return () => {
     return;
   };

diff --git a/tests/fixtures/controllers/methodController.ts b/tests/fixtures/controllers/methodController.ts
@@ -30,6 +30,7 @@ const TEST_SEC = {
 const ATT_KEY9 = 'x-attKey9';
 
 @Route('MethodTest')
+@Tags('MethodTest')
 export class MethodController extends Controller {
   @Options('Options')
   public async optionsMethod(): Promise<TestModel> {

diff --git a/tests/unit/swagger/definitionsGeneration/metadata.spec.ts b/tests/unit/swagger/definitionsGeneration/metadata.spec.ts
@@ -155,7 +155,7 @@ describe('Metadata generation', () => {
         throw new Error('Method tags not defined!');
       }
 
-      expect(method.tags).to.deep.equal(['Tag1', 'Tag2', 'Tag3']);
+      expect(method.tags).to.deep.equal(['Tag1', 'Tag2', 'Tag3', 'MethodTest']);
     });
 
     it('should generate multi response', () => {
@@ -227,7 +227,7 @@ describe('Metadata generation', () => {
         secondSec: ['permission:admin', 'permission:owner'],
       });
 
-      expect(method.tags).to.deep.equal(['EnumTag1']);
+      expect(method.tags).to.deep.equal(['EnumTag1', 'MethodTest']);
     });
 
     it('should generate success response', () => {