feat(http_api): allow patch method

Co-authored-by: Mike Nishizawa <michael@fortitude-solutions.com>
ekino · Aug 6, 2021 · a6b3423 · a6b3423
1 parent 824c435
commit a6b3423
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -740,7 +740,7 @@ Given:
 
 When:
   - /^(?:I )?reset http client$/
-  - /^(?:I )?(GET|POST|PUT|DELETE) (.+)$/
+  - /^(?:I )?(GET|POST|PUT|DELETE|PATCH) (.+)$/
   - /^(?:I )?dump response body$/
   - /^(?:I )?dump response headers$/
   - /^(?:I )?dump response cookies$/

diff --git a/src/extensions/http_api/client.js b/src/extensions/http_api/client.js
@@ -14,7 +14,7 @@ const BODY_TYPE_JSON = 'json'
 const BODY_TYPE_FORM = 'form'
 const BODY_TYPE_MULTIPART = 'form-data'
 
-const verbsAcceptingBody = ['POST', 'PUT', 'DELETE']
+const verbsAcceptingBody = ['POST', 'PUT', 'DELETE', 'PATCH']
 
 /**
  * Http Api Client extension.

diff --git a/src/extensions/http_api/definitions.js b/src/extensions/http_api/definitions.js
@@ -171,7 +171,7 @@ exports.install = ({ baseUrl = '' } = {}) => {
     /**
      * Performing a request
      */
-    When(/^(?:I )?(GET|POST|PUT|DELETE) (.+)$/, function (method, path) {
+    When(/^(?:I )?(GET|POST|PUT|DELETE|PATCH) (.+)$/, function (method, path) {
         return this.httpApiClient.makeRequest(method, this.state.populate(path), baseUrl)
     })
 

diff --git a/tests/extensions/http_api/definitions.test.js b/tests/extensions/http_api/definitions.test.js
@@ -466,16 +466,18 @@ test('reset http client', () => {
 test('perform a request', () => {
     const context = helper.getContext() // Extension context
 
-    const def = context.getDefinitionByMatcher('GET|POST|PUT|DELETE')
+    const def = context.getDefinitionByMatcher('GET|POST|PUT|DELETE|PATCH')
     def.shouldNotMatch('I GET ')
     def.shouldMatch('I GET /', ['GET', '/'])
     def.shouldMatch('I POST /create', ['POST', '/create'])
     def.shouldMatch('I PUT /update', ['PUT', '/update'])
     def.shouldMatch('I DELETE /delete', ['DELETE', '/delete'])
+    def.shouldMatch('I PATCH /updatePartial', ['PATCH', '/updatePartial'])
     def.shouldMatch('GET /', ['GET', '/'])
     def.shouldMatch('POST /create', ['POST', '/create'])
     def.shouldMatch('PUT /update', ['PUT', '/update'])
     def.shouldMatch('DELETE /delete', ['DELETE', '/delete'])
+    def.shouldMatch('PATCH /updatePartial', ['PATCH', '/updatePartial'])
 })
 
 test('dump response body', () => {