Revert "final thing, query param method"

This reverts commit 7061f4c.

Author: SentryMan

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

@@ -237,17 +237,6 @@ public interface HttpClientRequest {
    */
   HttpClientRequest queryParam(String name, Object value);
 
-  /**
-   * Add a collection query parameters associated with a single key
-   *
-   * @param name The name of the query parameter
-   * @param values The values of the query parameter which can be null
-   * @return The request being built
-   */
-  default HttpClientRequest queryParam(String name, Collection<Object> value) {
-    return queryParam(name, (Object) value);
-  }
-
   /**
    * Add a multiple query parameters as name value map.
    *

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

@@ -31,8 +31,6 @@ class HelloControllerTest extends BaseWebTest {
   private static final ObjectMapper objectMapper = new ObjectMapper();
 
   final HttpClientContext clientContext = client();
-  
-  String nullString = null;
 
   @Test
   void newClientTest() {
@@ -803,7 +801,7 @@ void async_list_as() throws ExecutionException, InterruptedException {
   void get_withPathParamAndQueryParam_returningBean() {
 
     final HelloDto dto = clientContext.request()
-      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", nullString)
+      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", null)
       .GET()
       .bean(HelloDto.class);
 
@@ -815,7 +813,7 @@ void get_withPathParamAndQueryParam_returningBean() {
   @Test
   void callBean() {
     final HelloDto dto = clientContext.request()
-      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", (String) null)
+      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", null)
       .GET()
       .call().bean(HelloDto.class).execute();
 
@@ -827,7 +825,7 @@ void callBean() {
   @Test
   void callBeanAsync() throws ExecutionException, InterruptedException {
     final CompletableFuture<HelloDto> future = clientContext.request()
-      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", (String) null)
+      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", null)
       .GET()
       .call().bean(HelloDto.class).async();
 
@@ -844,7 +842,7 @@ void async_whenComplete_returningBean() throws ExecutionException, InterruptedEx
     final AtomicReference<HelloDto> ref = new AtomicReference<>();
 
     final CompletableFuture<HelloDto> future = clientContext.request()
-      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", nullString)
+      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", null)
       .GET()
       .async().bean(HelloDto.class);
 
@@ -874,7 +872,7 @@ void async_whenComplete_returningBeanWithHeaders() throws ExecutionException, In
     final AtomicReference<HttpResponse<HelloDto>> ref = new AtomicReference<>();
 
     final CompletableFuture<HttpResponse<HelloDto>> future = clientContext.request()
-      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", nullString)
+      .path("hello/43/2020-03-05").queryParam("otherParam", "other").queryParam("foo", null)
       .GET()
       .async().as(HelloDto.class);