From 6e40d047419a6d44aaa464b6ecee1512ff50a123 Mon Sep 17 00:00:00 2001 From: Kyle Shockey Date: Fri, 12 May 2017 23:12:11 -0700 Subject: [PATCH] take advantage of Swagger-Client opId improvements --- src/core/plugins/spec/actions.js | 15 +++++++++++---- src/core/plugins/swagger-js/index.js | 3 ++- 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/src/core/plugins/spec/actions.js b/src/core/plugins/spec/actions.js index a496576a..761d1612 100644 --- a/src/core/plugins/spec/actions.js +++ b/src/core/plugins/spec/actions.js @@ -186,16 +186,23 @@ export const logRequest = (req) => { // Actually fire the request via fn.execute // (For debugging) and ease of testing export const executeRequest = (req) => ({fn, specActions, specSelectors}) => { - let { pathName, method } = req + let { pathName, method, operation } = req + + let op = operation.toJS() // if url is relative, parseUrl makes it absolute by inferring from `window.location` req.contextUrl = parseUrl(specSelectors.url()).toString() - let parsedRequest = Object.assign({}, req) - if ( pathName && method ) { - parsedRequest.operationId = method.toLowerCase() + "-" + pathName + + if(op && op.operationId) { + req.operationId = op.operationId + } else if(op && pathName && method) { + req.operationId = fn.opId(op, pathName, method) } + + let parsedRequest = Object.assign({}, req) parsedRequest = fn.buildRequest(parsedRequest) + specActions.setRequest(req.pathName, req.method, parsedRequest) return fn.execute(req) diff --git a/src/core/plugins/swagger-js/index.js b/src/core/plugins/swagger-js/index.js index c23147c8..d8744c19 100644 --- a/src/core/plugins/swagger-js/index.js +++ b/src/core/plugins/swagger-js/index.js @@ -7,7 +7,8 @@ module.exports = function({ configs }) { buildRequest: Swagger.buildRequest, execute: Swagger.execute, resolve: Swagger.resolve, - serializeRes: Swagger.serializeRes + serializeRes: Swagger.serializeRes, + opId: Swagger.helpers.opId } } }