remove unused methods

Author: svozza

packages/event-handler/src/rest/RouteHandlerRegistry.ts

@@ -19,7 +19,6 @@ class RouteHandlerRegistry {
   readonly #staticRoutes: Map<string, Route> = new Map();
   readonly #dynamicRoutesSet: Set<string> = new Set();
   readonly #dynamicRoutes: DynamicRoute[] = [];
-  readonly #routesByMethod: Map<string, Route[]> = new Map();
   #shouldSort = true;
 
   readonly #logger: Pick<GenericLogger, 'debug' | 'warn' | 'error'>;
@@ -99,10 +98,6 @@ class RouteHandlerRegistry {
       }
       this.#staticRoutes.set(route.id, route);
     }
-
-    const routesByMethod = this.#routesByMethod.get(route.method) ?? [];
-    routesByMethod.push(route);
-    this.#routesByMethod.set(route.method, routesByMethod);
   }
 
   /**
@@ -165,32 +160,6 @@ class RouteHandlerRegistry {
 
     return null;
   }
-
-  /**
-   * Returns the total number of registered routes.
-   * @returns Total count of registered routes
-   */
-  public getRouteCount(): number {
-    return [...Array.from(this.#staticRoutes.values()), ...this.#dynamicRoutes]
-      .length;
-  }
-
-  /**
-   * Returns all routes registered for a specific HTTP method.
-   * @param method - The HTTP method to filter by
-   * @returns Array of routes for the specified method
-   */
-  public getRoutesByMethod(method: string): Route[] {
-    return this.#routesByMethod.get(method.toUpperCase()) || [];
-  }
-
-  /**
-   * Returns all registered routes (both static and dynamic).
-   * @returns Array of all registered routes
-   */
-  public getAllRoutes(): Route[] {
-    return [...Array.from(this.#staticRoutes.values()), ...this.#dynamicRoutes];
-  }
 }
 
 export { RouteHandlerRegistry };

packages/event-handler/tests/unit/rest/RouteHandlerRegistry.test.ts

@@ -116,118 +116,6 @@ describe('Class: RouteHandlerRegistry', () => {
     expect(postResult?.handler).toBe(postHandler);
   });
 
-  describe('getRouteCount', () => {
-    it('returns 0 for empty registry', () => {
-      // Prepare
-      const registry = new RouteHandlerRegistry({ logger: console });
-
-      // Act & Assess
-      expect(registry.getRouteCount()).toBe(0);
-    });
-
-    it('returns correct count after registering routes', () => {
-      // Prepare
-      const registry = new RouteHandlerRegistry({ logger: console });
-      const handler = () => 'test';
-
-      // Act & Assess
-      registry.register(new Route(HttpVerbs.GET, '/users', handler));
-      expect(registry.getRouteCount()).toBe(1);
-
-      registry.register(new Route(HttpVerbs.POST, '/users', handler));
-      expect(registry.getRouteCount()).toBe(2);
-
-      registry.register(new Route(HttpVerbs.GET, '/posts', handler));
-      expect(registry.getRouteCount()).toBe(3);
-    });
-  });
-
-  describe('getRoutesByMethod', () => {
-    it('returns empty array for non-existent method', () => {
-      // Prepare
-      const registry = new RouteHandlerRegistry({ logger: console });
-
-      // Act & Assess
-      expect(registry.getRoutesByMethod('GET')).toEqual([]);
-      expect(registry.getRoutesByMethod('POST')).toEqual([]);
-    });
-
-    it.each([
-      { method: HttpVerbs.GET },
-      { method: HttpVerbs.POST },
-      { method: HttpVerbs.PUT },
-      { method: HttpVerbs.DELETE },
-    ])('returns routes for $method method', ({ method }) => {
-      // Prepare
-      const registry = new RouteHandlerRegistry({ logger: console });
-      const handler = () => 'test';
-
-      const route1 = new Route(method, '/users', handler);
-      const route2 = new Route(method, '/posts', handler);
-      const otherMethodRoute = new Route(HttpVerbs.PATCH, '/other', handler);
-
-      // Act
-      registry.register(route1);
-      registry.register(route2);
-      registry.register(otherMethodRoute);
-
-      // Assess
-      const routes = registry.getRoutesByMethod(method);
-      expect(routes).toHaveLength(2);
-      expect(routes).toContain(route1);
-      expect(routes).toContain(route2);
-      expect(routes).not.toContain(otherMethodRoute);
-    });
-
-    it('handles case-insensitive method lookup', () => {
-      // Prepare
-      const registry = new RouteHandlerRegistry({ logger: console });
-      const handler = () => 'test';
-
-      const getRoute = new Route(HttpVerbs.GET, '/users', handler);
-
-      // Act
-      registry.register(getRoute);
-
-      // Assess
-      expect(registry.getRoutesByMethod('get')).toContain(getRoute);
-      expect(registry.getRoutesByMethod('GET')).toContain(getRoute);
-      expect(registry.getRoutesByMethod('Get')).toContain(getRoute);
-    });
-  });
-
-  describe('getAllRoutes', () => {
-    it('returns empty array for empty registry', () => {
-      // Prepare
-      const registry = new RouteHandlerRegistry({ logger: console });
-
-      // Act & Assess
-      expect(registry.getAllRoutes()).toEqual([]);
-    });
-
-    it('returns all registered routes', () => {
-      // Prepare
-      const registry = new RouteHandlerRegistry({ logger: console });
-      const handler = () => 'test';
-
-      const route1 = new Route(HttpVerbs.GET, '/users', handler);
-      const route2 = new Route(HttpVerbs.POST, '/users', handler);
-      const route3 = new Route(HttpVerbs.GET, '/posts', handler);
-
-      // Act
-      registry.register(route1);
-      registry.register(route2);
-      registry.register(route3);
-
-      // Assess
-      const allRoutes = registry.getAllRoutes();
-      expect(allRoutes).toHaveLength(3);
-      expect(allRoutes).toContain(route1);
-      expect(allRoutes).toContain(route2);
-      expect(allRoutes).toContain(route3);
-    });
-  });
-
   describe('#compareRouteSpecificity', () => {
     it('handles routes of different specificity', () => {
       // Prepare