fix: correctly handle 307 response with location header (#23)

Author: Monforton

src/main/kotlin/io/github/guidewire/oss/DataSender.kt

@@ -54,7 +54,7 @@ private fun postTestRun(endpoint: String, fernUrl: String, payload: String): Htt
 
   var response = client.send(request, HttpResponse.BodyHandlers.ofString())
   if (response.statusCode() == 307) {
-    val locationHeader = response.headers().firstValue("location")
+    val locationHeader = response.headers().firstValue("location").orElseThrow { RuntimeException("Location header not found in 307 response") }
     response = postTestRun(fernUrl + locationHeader, fernUrl, payload)
   }
   return response

src/test/kotlin/DataSenderTest.kt

@@ -66,6 +66,44 @@ class DataSenderTest {
     )
   }
 
+ @Test
+   fun `sendTestRun should follow redirects`() {
+     // Setup mock server for initial redirect
+     stubFor(
+       post(urlEqualTo("/api/testrun/"))
+         .willReturn(
+           aResponse()
+             .withStatus(307)
+             .withHeader("Location", "/api/testrun/redirect")
+         )
+     )
+
+     // Setup mock server for the redirected endpoint
+     stubFor(
+       post(urlEqualTo("/api/testrun/redirect"))
+         .withHeader("Content-Type", equalTo("application/json"))
+         .willReturn(
+           aResponse()
+             .withStatus(200)
+             .withBody("{\"status\":\"success\"}")
+         )
+     )
+
+     // Test API call
+     val result = sendTestRun(testRun, "http://localhost:${wireMockServer.port()}", true)
+
+     // Verify
+     assertTrue(result.isSuccess)
+     verify(
+       postRequestedFor(urlEqualTo("/api/testrun/"))
+         .withHeader("Content-Type", equalTo("application/json"))
+     )
+     verify(
+       postRequestedFor(urlEqualTo("/api/testrun/redirect"))
+         .withHeader("Content-Type", equalTo("application/json"))
+     )
+   }
+
   @Test
   fun `sendTestRun should handle server errors`() {
     // Setup mock server to return error