Use Python Requests built-in method methods where possible

Author: pimterry

src/targets/python/requests.js

@@ -14,6 +14,16 @@ const util = require('util')
 const CodeBuilder = require('../../helpers/code-builder')
 const helpers = require('./helpers')
 
+const builtInMethods = [
+  'HEAD',
+  'GET',
+  'POST',
+  'PUT',
+  'PATCH',
+  'DELETE',
+  'OPTIONS'
+]
+
 module.exports = function (source, options) {
   const opts = Object.assign({
     indent: '    ',
@@ -89,7 +99,10 @@ module.exports = function (source, options) {
 
   // Construct request
   const method = source.method
-  let request = util.format('response = requests.request("%s", url', method)
+
+  let request = builtInMethods.includes(method)
+    ? util.format('response = requests.%s(url', method.toLowerCase())
+    : util.format('response = requests.request("%s", url', method)
 
   if (hasPayload) {
     if (jsonPayload) {

test/fixtures/output/python/requests/application-form-encoded.py

@@ -5,6 +5,6 @@
 payload = "foo=bar&hello=world"
 headers = { "content-type": "application/x-www-form-urlencoded" }
 
-response = requests.request("POST", url, data=payload, headers=headers)
+response = requests.post(url, data=payload, headers=headers)
 
 print(response.text)

test/fixtures/output/python/requests/application-json.py

@@ -12,6 +12,6 @@
 }
 headers = { "content-type": "application/json" }
 
-response = requests.request("POST", url, json=payload, headers=headers)
+response = requests.post(url, json=payload, headers=headers)
 
 print(response.text)

test/fixtures/output/python/requests/cookies.py

@@ -4,6 +4,6 @@
 
 headers = { "cookie": "foo=bar; bar=baz" }
 
-response = requests.request("POST", url, headers=headers)
+response = requests.post(url, headers=headers)
 
 print(response.text)

test/fixtures/output/python/requests/full.py

@@ -15,6 +15,6 @@
     "content-type": "application/x-www-form-urlencoded"
 }
 
-response = requests.request("POST", url, data=payload, headers=headers, params=querystring)
+response = requests.post(url, data=payload, headers=headers, params=querystring)
 
 print(response.text)

test/fixtures/output/python/requests/headers.py

@@ -7,6 +7,6 @@
     "x-foo": "Bar"
 }
 
-response = requests.request("GET", url, headers=headers)
+response = requests.get(url, headers=headers)
 
 print(response.text)

test/fixtures/output/python/requests/https.py

@@ -2,6 +2,6 @@
 
 url = "https://mockbin.com/har"
 
-response = requests.request("GET", url)
+response = requests.get(url)
 
 print(response.text)

test/fixtures/output/python/requests/jsonObj-multiline.py

@@ -5,6 +5,6 @@
 payload = { "foo": "bar" }
 headers = { "content-type": "application/json" }
 
-response = requests.request("POST", url, json=payload, headers=headers)
+response = requests.post(url, json=payload, headers=headers)
 
 print(response.text)

test/fixtures/output/python/requests/jsonObj-null-value.py

@@ -5,6 +5,6 @@
 payload = { "foo": None }
 headers = { "content-type": "application/json" }
 
-response = requests.request("POST", url, json=payload, headers=headers)
+response = requests.post(url, json=payload, headers=headers)
 
 print(response.text)

test/fixtures/output/python/requests/multipart-data.py

@@ -5,6 +5,6 @@
 payload = "-----011000010111000001101001\r\nContent-Disposition: form-data; name=\"foo\"; filename=\"hello.txt\"\r\nContent-Type: text/plain\r\n\r\nHello World\r\n-----011000010111000001101001--\r\n"
 headers = { "content-type": "multipart/form-data; boundary=---011000010111000001101001" }
 
-response = requests.request("POST", url, data=payload, headers=headers)
+response = requests.post(url, data=payload, headers=headers)
 
 print(response.text)