fix: findOperation issues with "/" paths (#223)

kanadgupta · web-flow · commit ca1cad2da6c6 · 2020-07-15T18:13:01.000-07:00
* fix: findOperation issues with "/" paths * test: add check for res.operation feedback: #223 (comment)
diff --git a/packages/tooling/__tests__/oas.test.js b/packages/tooling/__tests__/oas.test.js
@@ -185,6 +185,48 @@ describe('#findOperation()', () => {
     });
   });
 
+  it('should return result if path is slash', () => {
+    const oas = new Oas({
+      openapi: '3.0.0',
+      servers: [
+        {
+          url: 'https://example.com',
+        },
+      ],
+      paths: {
+        '/': {
+          get: {
+            responses: {
+              '200': {
+                description: 'OK',
+              },
+            },
+          },
+        },
+      },
+    });
+
+    const uri = 'https://example.com';
+    const method = 'get';
+
+    const res = oas.findOperation(uri, method);
+    expect(res.url).toStrictEqual({
+      origin: 'https://example.com',
+      path: '/',
+      nonNormalizedPath: '/',
+      slugs: {},
+      method: 'GET',
+    });
+
+    expect(res.operation).toStrictEqual({
+      responses: {
+        '200': {
+          description: 'OK',
+        },
+      },
+    });
+  });
+
   it('should return result if in server variable defaults', () => {
     const oas = new Oas(serverVariables);
     const uri = 'https://demo.example.com:443/v2/post';
diff --git a/packages/tooling/src/oas.js b/packages/tooling/src/oas.js
@@ -167,8 +167,9 @@ class Oas {
     if (!targetServer) return undefined;
     targetServer.url = this.replaceUrl(targetServer.url, targetServer.variables || {});
 
-    const [, pathName] = url.split(targetServer.url);
+    let [, pathName] = url.split(targetServer.url);
     if (pathName === undefined) return undefined;
+    if (pathName === '') pathName = '/';
     const annotatedPaths = generatePathMatches(paths, pathName, targetServer.url);
     if (!annotatedPaths.length) return undefined;
 
