Merge pull request kittinunf#89 from kittinunf/head_support

head support

Author: yoavst

README.md

@@ -6,7 +6,7 @@ The easiest HTTP networking library for Kotlin/Android.
 
 ## Features
 
-- [x] Support basic HTTP GET/POST/PUT/DELETE in a fluent style interface
+- [x] Support basic HTTP GET/POST/PUT/DELETE/HEAD in a fluent style interface
 - [x] Support both asynchronous and blocking requests
 - [x] Download file
 - [x] Upload file (multipart/form-data)
@@ -195,6 +195,14 @@ Fuel.delete("http://httpbin.org/delete").response { request, response, result ->
 }
 ```
 
+### HEAD
+
+``` Kotlin
+Fuel.head("http://httpbin.org/get").response { request, response, result ->
+   // request body should be empty.
+}
+```
+
 ### Debug Logging
 * Use `toString()` method to Log (request|response)
 

fuel/src/main/kotlin/com/github/kittinunf/fuel/Fuel.kt

@@ -92,6 +92,17 @@ class Fuel {
             return upload(convertible.path, method, parameters)
         }
 
+        //head
+        @JvmStatic @JvmOverloads
+        fun head(path: String, parameters: List<Pair<String, Any?>>? = null): Request {
+            return request(Method.HEAD, path, parameters)
+        }
+
+        @JvmStatic @JvmOverloads
+        fun head(convertible: PathStringConvertible, parameters: List<Pair<String, Any?>>? = null): Request {
+            return request(Method.HEAD, convertible, parameters)
+        }
+
         //request
         private fun request(method: Method, path: String, parameters: List<Pair<String, Any?>>? = null): Request {
             return FuelManager.instance.request(method, path, parameters)
@@ -169,3 +180,13 @@ fun String.httpUpload(method: Method = Method.POST, parameters: List<Pair<String
 fun Fuel.PathStringConvertible.httpUpload(method: Method = Method.POST, parameters: List<Pair<String, Any?>>? = null): Request {
     return Fuel.upload(this, method, parameters)
 }
+
+@JvmOverloads
+fun Fuel.PathStringConvertible.httpHead(parameter: List<Pair<String, Any?>>? = null): Request {
+    return Fuel.head(this, parameter)
+}
+
+@JvmOverloads
+fun String.httpHead(parameters: List<Pair<String, Any?>>? = null): Request {
+    return Fuel.head(this, parameters)
+}

fuel/src/main/kotlin/com/github/kittinunf/fuel/core/Encoding.kt

@@ -67,7 +67,7 @@ class Encoding : Fuel.RequestConvertible {
 
     private fun encodeParameterInUrl(method: Method): Boolean {
         when (method) {
-            Method.GET, Method.DELETE -> return true
+            Method.GET, Method.DELETE, Method.HEAD -> return true
             else -> return false
         }
     }

fuel/src/main/kotlin/com/github/kittinunf/fuel/core/Method.kt

@@ -5,6 +5,7 @@ enum class Method(val value: String) {
     POST("POST"),
     PUT("PUT"),
     DELETE("DELETE"),
-    PATCH("PATCH")
+    PATCH("PATCH"),
+    HEAD("HEAD")
 }
 

fuel/src/main/kotlin/com/github/kittinunf/fuel/toolbox/HttpClient.kt

@@ -98,7 +98,7 @@ class HttpClient : Client {
 
     private fun setDoOutput(connection: HttpURLConnection, method: Method) {
         when (method) {
-            Method.GET, Method.DELETE -> connection.doOutput = false
+            Method.GET, Method.DELETE, Method.HEAD -> connection.doOutput = false
             Method.POST, Method.PUT, Method.PATCH -> connection.doOutput = true
         }
     }

fuel/src/test/kotlin/com/github/kittinunf/fuel/RequestTest.kt

@@ -249,6 +249,38 @@ class RequestTest : BaseTestCase() {
         assertThat(string, containsString(paramValue))
     }
 
+    @Test
+    fun httpHeadRequest() {
+        var request: Request? = null
+        var response: Response? = null
+        var data: Any? = null
+        var error: FuelError? = null
+
+        val paramKey = "foo"
+        val paramValue = "bar"
+
+        manager.request(Method.HEAD, "http://httpbin.org/get", listOf(paramKey to paramValue)).responseString { req, res, result ->
+            request = req
+            response = res
+
+            val (d, err) = result
+            data = d
+            error = err
+        }
+
+        val string = data as String
+
+        assertThat(request, notNullValue())
+        assertThat(response, notNullValue())
+        assertThat(error, nullValue())
+        assertThat(data, notNullValue())
+
+        val statusCode = HttpURLConnection.HTTP_OK
+        assertThat(response?.httpStatusCode, isEqualTo(statusCode))
+
+        assertThat(string, equalTo(""))
+    }
+
     @Test
     fun httpGetRequestWithPathStringConvertible() {
         var request: Request? = null