okhttp post need body or null to be passed

Shashi Ranjan · Shashi Ranjan · commit 79c49b9db456 · 2015-03-28T20:41:05.000-07:00
diff --git a/src/targets/java/okhttp.js b/src/targets/java/okhttp.js
@@ -20,7 +20,9 @@ module.exports = function (source, options) {
 
   var code = new CodeBuilder(opts.indent)
 
-  var methods = [ 'GET', 'POST', 'PUT', 'DELETE', 'PATCH', 'HEAD', 'OPTIONS' ]
+  var methods = [ 'GET', 'POST', 'PUT', 'DELETE', 'PATCH', 'HEAD']
+
+  var methodsWithBody = [ 'POST', 'PUT', 'DELETE', 'PATCH']
 
   code.push('OkHttpClient client = new OkHttpClient();')
       .blank()
@@ -34,6 +36,8 @@ module.exports = function (source, options) {
     code.push('RequestBody body = RequestBody.create(mediaType, %s);', JSON.stringify(source.postData.text))
   }
 
+  console.log(source)
+
   code.push('Request request = new Request.Builder()')
   code.push(1, '.url("%s")', source.fullUrl)
   if (methods.indexOf(source.method.toUpperCase()) === -1) {
@@ -46,7 +50,12 @@ module.exports = function (source, options) {
     if (source.postData.text) {
       code.push(1, '.%s(body)', source.method.toLowerCase())
     }else {
-      code.push(1, '.%s()', source.method.toLowerCase())
+      console.log(methodsWithBody.indexOf(source.method.toUpperCase()))
+      if (methodsWithBody.indexOf(source.method.toUpperCase()) >= 0) {
+        code.push(1, '.%s(null)', source.method.toLowerCase())
+      }else {
+        code.push(1, '.%s()', source.method.toLowerCase())
+      }
     }
   }
 
diff --git a/test/fixtures/output/java/okhttp/cookies.java b/test/fixtures/output/java/okhttp/cookies.java
@@ -2,7 +2,7 @@
 
 Request request = new Request.Builder()
   .url("http://mockbin.com/har")
-  .post()
+  .post(null)
   .addHeader("cookie", "foo=bar; bar=baz")
   .build();
 
