fix: remove requirement for operationId (#69)

lib/backends/swagger-client.js

@@ -28,11 +28,17 @@ class SwaggerClient {
       options.qs || options.parameters,
       options.pathnameParameters
     )
+
     const operationId = options.pathItemObject[options.method.toLowerCase()].operationId
-    return this.client.execute({
-      operationId,
-      parameters
-    })
+
+    const operationReference = operationId ? {
+      operationId
+    } : {
+      pathName: options.pathname,
+      method: options.method
+    }
+
+    return this.client.execute(Object.assign({}, operationReference, { parameters }))
   }
 }
 

lib/backends/swagger-client.test.js

@@ -27,7 +27,33 @@ describe('lib.backends.swagger-client', () => {
           }
         })).to.be.equal(true)
         done()
-      })
+      }).catch(done)
+    })
+
+    it('executes operation without operationId', done => {
+      const swagger = { execute: sinon.spy(() => Promise.resolve('some result')) }
+      const client = new SwaggerClient({ client: swagger })
+
+      client.http({
+        method: 'GET',
+        pathname: '/api/v2/foo',
+        pathItemObject: { get: { } },
+        body: { a: 1 },
+        parameters: { b: 2 },
+        pathnameParameters: { c: 3 }
+      }).then(() => {
+        expect(swagger.execute.called).is.equal(true)
+        expect(swagger.execute.getCall(0).args).to.deep.equal([{
+          pathName: '/api/v2/foo',
+          method: 'GET',
+          parameters: {
+            a: 1,
+            b: 2,
+            c: 3
+          }
+        }])
+        done()
+      }).catch(done)
     })
   })
 })

lib/loader.js

@@ -220,7 +220,6 @@ class Component {
     // "Expose" operations by omitting the leading _ from the method name.
     //
     Object.keys(endpoint.pathItem)
-      .filter(key => endpoint.pathItem[key].operationId)
       .forEach(method => {
         component[method] = component['_' + method]
         if (method === 'get') component.getStream = component._getStream

lib/loader.test.js

@@ -88,6 +88,21 @@ describe('list.loader', () => {
       expect(component.foo('loo').bar.getPath()).to.equal('/foo/loo/bar')
     })
 
+    it('does not blow up without operationId', () => {
+      const spec = {
+        paths: {
+          '/apis/some/url': {
+            get: {
+            }
+          }
+        }
+      }
+      const component = new Component()
+      component._addSpec(spec)
+      expect(component.apis.some.url.get).is.a('function')
+      expect(component.apis.some.url.getStream).is.a('function')
+    })
+
     it('tracks parameters correctly', () => {
       const spec = {
         paths: {