chore: remove unused express decorators

BREAKING CHANGE: Remove CaseSensitive, MergeParams, RouterSettings and Strict decorators
tsedio · Jul 25, 2022 · ceec965 · ceec965
1 parent 5251249
commit ceec965
Show file tree

Hide file tree

Showing 11 changed files with 1 addition and 169 deletions.
diff --git a/packages/di/src/decorators/controller.ts b/packages/di/src/decorators/controller.ts
@@ -14,7 +14,6 @@ export interface ControllerMiddlewares {
 export interface ControllerOptions extends Partial<ProviderOpts<any>> {
   path?: PathType;
   children?: Type<any>[];
-  routerOptions?: any;
   middlewares?: Partial<ControllerMiddlewares>;
 }
 

diff --git a/packages/di/src/domain/ControllerProvider.spec.ts b/packages/di/src/domain/ControllerProvider.spec.ts
@@ -33,10 +33,6 @@ describe("ControllerProvider", () => {
     expect(controllerProvider.scope).toEqual("request");
   });
 
-  it("should get routerOptions", () => {
-    expect(controllerProvider.routerOptions).toBeInstanceOf(Object);
-  });
-
   it("should have children", () => {
     expect(controllerProvider.hasChildren()).toEqual(false);
   });

diff --git a/packages/platform/common/src/services/FakeAdapter.ts b/packages/platform/common/src/services/FakeAdapter.ts
@@ -44,16 +44,6 @@ export class FakeAdapter implements PlatformAdapter<any> {
     return {} as any;
   }
 
-  router(routerOptions: any): {router: any; callback(): any} {
-    const router = FakeAdapter.createFakeRawDriver();
-    return {
-      router,
-      callback() {
-        return router;
-      }
-    };
-  }
-
   statics(endpoint: string, options: PlatformStaticsOptions): any {
     return {};
   }

diff --git a/packages/platform/platform-express/src/decorators/caseSensitive.ts b/packages/platform/platform-express/src/decorators/caseSensitive.ts
diff --git a/packages/platform/platform-express/src/decorators/mergeParams.ts b/packages/platform/platform-express/src/decorators/mergeParams.ts
diff --git a/packages/platform/platform-express/src/decorators/routerSettings.spec.ts b/packages/platform/platform-express/src/decorators/routerSettings.spec.ts
diff --git a/packages/platform/platform-express/src/decorators/routerSettings.ts b/packages/platform/platform-express/src/decorators/routerSettings.ts
diff --git a/packages/platform/platform-express/src/decorators/strict.ts b/packages/platform/platform-express/src/decorators/strict.ts
diff --git a/packages/platform/platform-express/src/index.ts b/packages/platform/platform-express/src/index.ts
@@ -4,10 +4,6 @@
 
 export * from "./exports";
 export * from "./components/PlatformExpress";
-export * from "./decorators/caseSensitive";
-export * from "./decorators/mergeParams";
-export * from "./decorators/routerSettings";
-export * from "./decorators/strict";
 export * from "./interfaces/PlatformExpressSettings";
 export * from "./interfaces/PlatformExpressStaticsOptions";
 export * from "./interfaces/interfaces";

diff --git a/packages/platform/platform-express/test/merge-params.spec.ts b/packages/platform/platform-express/test/merge-params.spec.ts
@@ -1,5 +1,5 @@
 import {Controller, Get, PathParams, PlatformTest} from "@tsed/common";
-import {MergeParams, PlatformExpress} from "@tsed/platform-express";
+import {PlatformExpress} from "@tsed/platform-express";
 import {PlatformTestUtils} from "@tsed/platform-test-utils";
 import SuperTest from "supertest";
 import {rootDir, Server} from "./app/Server";
@@ -11,7 +11,6 @@ const utils = PlatformTestUtils.create({
 });
 
 @Controller("/merge-params/:parentId")
-@MergeParams(true)
 class TestMergeParamsCtrl {
   @Get("/:id")
   get(@PathParams("parentId") parentId: string, @PathParams("id") id: string) {

diff --git a/packages/platform/platform-test-utils/src/tests/testStatics.ts b/packages/platform/platform-test-utils/src/tests/testStatics.ts
@@ -20,9 +20,6 @@ export function testStatics(options: PlatformTestOptions) {
   beforeEach(
     PlatformTest.bootstrap(options.server, {
       ...options,
-      logger: {
-        level: "info"
-      },
       mount: {
         "/rest": [CustomStaticsCtrl]
       }