From aeb17588b329ea9f88d07d4663596d3bed5c54ae Mon Sep 17 00:00:00 2001
From: translatenix <119817707+translatenix@users.noreply.github.com>
Date: Mon, 6 May 2024 03:02:45 -0700
Subject: [PATCH] Simplify implementation of RequestCapturingClient (#473)

---
 .../pkl/core/http/RequestCapturingClient.kt   | 29 ++-----------------
 1 file changed, 3 insertions(+), 26 deletions(-)

diff --git a/pkl-core/src/test/kotlin/org/pkl/core/http/RequestCapturingClient.kt b/pkl-core/src/test/kotlin/org/pkl/core/http/RequestCapturingClient.kt
index 165e2aba..55f5d2a1 100644
--- a/pkl-core/src/test/kotlin/org/pkl/core/http/RequestCapturingClient.kt
+++ b/pkl-core/src/test/kotlin/org/pkl/core/http/RequestCapturingClient.kt
@@ -1,41 +1,18 @@
 package org.pkl.core.http
 
-import java.net.URI
-import java.net.http.HttpHeaders
+import org.pkl.commons.test.FakeHttpResponse
 import java.net.http.HttpRequest
 import java.net.http.HttpResponse
-import java.util.*
-import javax.net.ssl.SSLSession
 
 class RequestCapturingClient : HttpClient {
   lateinit var request: HttpRequest
 
-  private fun <T: Any?> dummyResponse(request: HttpRequest) : HttpResponse<T> {
-    return object : HttpResponse<T> {
-      override fun statusCode(): Int = throw NotImplementedError()
-
-      override fun request(): HttpRequest = request
-
-      override fun previousResponse(): Optional<HttpResponse<T>> = throw NotImplementedError()
-
-      override fun headers(): HttpHeaders = throw NotImplementedError()
-
-      override fun body(): T = throw NotImplementedError()
-
-      override fun sslSession(): Optional<SSLSession> = throw NotImplementedError()
-
-      override fun uri(): URI = throw NotImplementedError()
-
-      override fun version(): java.net.http.HttpClient.Version = throw NotImplementedError()
-    }
-  }
-
-  override fun <T : Any?> send(
+  override fun <T : Any> send(
     request: HttpRequest,
     responseBodyHandler: HttpResponse.BodyHandler<T>
   ): HttpResponse<T> {
     this.request = request
-    return dummyResponse(request)
+    return FakeHttpResponse()
   }
 
   override fun close() {}