fix(server): changes per review

pmcelhaney · Apr 10, 2024 · d9fa14f · d9fa14f
1 parent d0f6f5a
commit d9fa14f
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 8 deletions.
diff --git a/src/server/dispatcher.ts b/src/server/dispatcher.ts
@@ -101,13 +101,14 @@ export class Dispatcher {
     path: string,
     method: HttpMethods,
   ): OpenApiOperation | undefined {
-    for (const key in this.openApiDocument?.paths) {
-      if (key.toLowerCase() === path.toLowerCase()) {
-        return this.openApiDocument.paths[key]?.[
-          // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-          method.toLowerCase() as Lowercase<HttpMethods>
-        ];
-        break;
+    if (this.openApiDocument) {
+      for (const key in this.openApiDocument.paths) {
+        if (key.toLowerCase() === path.toLowerCase()) {
+          return this.openApiDocument.paths[key]?.[
+            // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
+            method.toLowerCase() as Lowercase<HttpMethods>
+          ];
+        }
       }
     }
 

diff --git a/test/server/dispatcher.test.ts b/test/server/dispatcher.test.ts
@@ -852,7 +852,7 @@ describe("given a request that contains the differently cased path", () => {
       },
     });
 
-    const response = await dispatcher.request({
+    let response = await dispatcher.request({
       body: "",
       headers: {},
       method: "POST",
@@ -864,5 +864,18 @@ describe("given a request that contains the differently cased path", () => {
     expect(response.status).toBe(200);
 
     expect(response.body).toBe("ok");
+
+    response = await dispatcher.request({
+      body: "",
+      headers: {},
+      method: "POST",
+      path: "/ABC",
+      query: {},
+      req: { path: "/ABC" },
+    });
+
+    expect(response.status).toBe(200);
+
+    expect(response.body).toBe("ok");
   });
 });