feat: add curl() method test cases to RequestTest for generating cURL commands

Author: hamza-badar

okhttp/src/commonJvmAndroid/kotlin/okhttp3/Request.kt

@@ -423,7 +423,7 @@ class Request internal constructor(
 
     open fun build(): Request = Request(this)
   }
-  
+
   /**
    * Returns a cURL command equivalent to this request, useful for debugging and reproducing requests.
    *
@@ -442,20 +442,39 @@ class Request internal constructor(
     if (method != "GET") {
       curl.append(" -X ").append(method)
     }
-    
+
     for ((name, value) in headers) {
-      curl.append(" -H ")
-        .append("\"").append(name).append(": ").append(value).append("\"")
+      curl
+        .append(" -H ")
+        .append("\"")
+        .append(name)
+        .append(": ")
+        .append(value)
+        .append("\"")
     }
-    
+
     body?.let { requestBody ->
       val buffer = okio.Buffer()
       requestBody.writeTo(buffer)
-      val bodyString = buffer.readUtf8()
-      curl.append(" --data ")
-        .append("\"").append(bodyString.replace("\"", "\\\"")).append("\"")
+
+      val contentType = requestBody.contentType()?.toString() ?: ""
+
+      if (contentType.startsWith("text/") ||
+        contentType.contains("json") ||
+        contentType.contains("xml") ||
+        contentType.contains("form-urlencoded")
+      ) {
+        val bodyString = buffer.readUtf8()
+        curl
+          .append(" --data ")
+          .append("\"")
+          .append(bodyString.replace("\"", "\\\""))
+          .append("\"")
+      } else {
+        curl.append(" --data-binary \"[binary body omitted]\"")
+      }
     }
-    
+
     curl.append(" \"").append(url).append("\"")
     return curl.toString()
   }

okhttp/src/jvmTest/kotlin/okhttp3/RequestTest.kt

@@ -643,6 +643,97 @@ class RequestTest {
     }
   }
 
+  @Test
+  fun curlGet() {
+    val request =
+      Request
+        .Builder()
+        .url("https://example.com")
+        .header("Authorization", "Bearer abc123")
+        .build()
+    val curl = request.curl()
+    assertThat(curl).isEqualTo("curl -H \"Authorization: Bearer abc123\" \"https://example.com/\"")
+  }
+
+  @Test
+  fun curlPostWithBody() {
+    val mediaType = "application/json".toMediaType()
+    val body = "{\"key\":\"value\"}".toRequestBody(mediaType)
+    val request =
+      Request
+        .Builder()
+        .url("https://api.example.com/data")
+        .post(body)
+        .addHeader("Content-Type", "application/json")
+        .addHeader("Authorization", "Bearer abc123")
+        .build()
+    val curl = request.curl()
+    assertThat(
+      curl,
+    ).isEqualTo(
+      "curl -X POST -H \"Content-Type: application/json\" -H \"Authorization: Bearer abc123\" --data \"{\\\"key\\\":\\\"value\\\"}\" \"https://api.example.com/data\"",
+    )
+  }
+
+  @Test
+  fun curlPostWithComplexBody() {
+    val mediaType = "application/json".toMediaType()
+    val jsonBody =
+      """
+      {
+        "user": {
+          "id": 123,
+          "name": "John Doe"
+        },
+        "roles": ["admin", "editor"],
+        "active": true
+      }
+      """.trimIndent()
+    val body = jsonBody.toRequestBody(mediaType)
+    val request =
+      Request
+        .Builder()
+        .url("https://api.example.com/users")
+        .post(body)
+        .addHeader("Content-Type", "application/json")
+        .addHeader("Authorization", "Bearer xyz789")
+        .build()
+    val curl = request.curl()
+    assertThat(curl).isEqualTo(
+      "curl -X POST -H \"Content-Type: application/json\" -H \"Authorization: Bearer xyz789\" --data \"{\n" +
+        "  \\\"user\\\": {\n" +
+        "    \\\"id\\\": 123,\n" +
+        "    \\\"name\\\": \\\"John Doe\\\"\n" +
+        "  },\n" +
+        "  \\\"roles\\\": [\\\"admin\\\", \\\"editor\\\"],\n" +
+        "  \\\"active\\\": true\n" +
+        "}\" \"https://api.example.com/users\"",
+    )
+  }
+
+  @Test
+  fun curlPostWithBinaryBody() {
+    val mediaType = "application/octet-stream".toMediaType()
+    val binaryData = byteArrayOf(0x00, 0x01, 0x02, 0x03, 0x04)
+
+    val body = RequestBody.create(mediaType, binaryData)
+
+    val request =
+      Request
+        .Builder()
+        .url("https://api.example.com/upload")
+        .post(body)
+        .addHeader("Content-Type", "application/octet-stream")
+        .build()
+
+    val curl = request.curl()
+    assertThat(
+      curl,
+    ).isEqualTo(
+      "curl -X POST -H \"Content-Type: application/octet-stream\" --data-binary \"[binary body omitted]\" \"https://api.example.com/upload\"",
+    )
+  }
+
   private fun bodyToHex(body: RequestBody): String {
     val buffer = Buffer()
     body.writeTo(buffer)