cov(@nestjs) increase test coverage

Author: kamilmysliwiec

CHANGELOG.md

@@ -1,4 +1,5 @@
 ## 4.5.2
+- **common**: [feature] rename `modules` to `imports` (`@Module()` decorator)
 - **core**: [feature] exception filters with empty `@Catch()` metadata handle each occurred exception 
 
 ## 4.5.1

src/common/test/decorators/module.decorator.spec.ts

@@ -36,4 +36,26 @@ describe('@Module', () => {
       InvalidModuleConfigException,
     );
   });
+
+  describe(`when "imports" is used`, () => {
+    const imports = ['Imports'];
+    @Module({
+      imports,
+    })
+    class TestModule2 {}
+    it(`should override "modules" metadata when there is no modules`, () => {
+      const modules = Reflect.getMetadata('modules', TestModule2);
+      expect(modules).to.be.eql(imports);
+    });
+
+    @Module({
+      ...moduleProps,
+      imports,
+    })
+    class TestModule3 {}
+    it(`should not override "modules" metadata when there is no modules`, () => {
+      const modules = Reflect.getMetadata('modules', TestModule3);
+      expect(modules).to.be.eql(moduleProps.modules);
+    });
+  });
 });

src/core/router/router-explorer.ts

@@ -156,7 +156,6 @@ export class ExpressRouterExplorer implements RouterExplorer {
       requestMethod,
     );
     const exceptionFilter = this.exceptionsFilter.create(instance, callback);
-
     return this.routerProxy.createProxy(executionContext, exceptionFilter);
   }
 }