#323 followup - tidy only and add some extra tests

Author: rbygrave

http-client/src/main/java/io/avaje/http/client/DUrlBuilder.java

@@ -15,11 +15,9 @@ final class DUrlBuilder implements UrlBuilder {
 
   @Override
   public UrlBuilder url(String url) {
-
     if (url.startsWith("http") && url.contains("://")) {
       buffer.setLength(0);
     }
-
     buffer.append(url);
     return this;
   }

http-client/src/test/java/io/avaje/http/client/DUrlBuilderTest.java

@@ -0,0 +1,46 @@
+package io.avaje.http.client;
+
+import org.junit.jupiter.api.Test;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+class DUrlBuilderTest {
+
+  @Test
+  void url() {
+    var uri = UrlBuilder.of("http://foo")
+      .path("more")
+      .build();
+
+    assertThat(uri).isEqualTo("http://foo/more");
+  }
+
+  @Test
+  void url_when_httpPrefix() {
+    var uri = UrlBuilder.of("http://foo")
+      .url("http://bar")
+      .path("more")
+      .build();
+
+    assertThat(uri).isEqualTo("http://bar/more");
+  }
+
+  @Test
+  void path_basic() {
+    var uri = UrlBuilder.of("http://foo")
+      .path("more")
+      .build();
+
+    assertThat(uri).isEqualTo("http://foo/more");
+  }
+
+  @Test
+  void path_when_httpPrefix() {
+    var uri = UrlBuilder.of("http://foo")
+      .path("http://bar")
+      .path("more")
+      .build();
+
+    assertThat(uri).isEqualTo("http://bar/more");
+  }
+}