added simple invalid configuration test

Author: chernser

client-v2/src/main/java/com/clickhouse/client/api/Client.java

@@ -871,6 +871,11 @@ public Client build() {
                 throw new IllegalArgumentException("Username and password (or access token, or SSL authentication) are required");
             }
 
+            if (this.configuration.containsKey("ssl_authentication") &&
+                    (this.configuration.containsKey("password") || this.configuration.containsKey("access_token"))) {
+                throw new IllegalArgumentException("Only one of password, access token or SSL authentication can be used per client.");
+            }
+
             if (this.configuration.containsKey("ssl_authentication") &&
                 !this.configuration.containsKey(ClickHouseClientOption.SSL_CERTIFICATE.getKey())) {
                 throw new IllegalArgumentException("SSL authentication requires a client certificate");

client-v2/src/main/java/com/clickhouse/client/api/internal/HttpAPIClientHelper.java

@@ -384,13 +384,11 @@ public ClassicHttpResponse executeRequest(ClickHouseNode server, Map<String, Obj
 
     private void addHeaders(HttpPost req, Map<String, String> chConfig, Map<String, Object> requestConfig) {
         req.addHeader(HttpHeaders.CONTENT_TYPE, CONTENT_TYPE.getMimeType());
-        if (requestConfig != null) {
-            if (requestConfig.containsKey(ClickHouseClientOption.FORMAT.getKey())) {
-                req.addHeader(ClickHouseHttpProto.HEADER_FORMAT, requestConfig.get(ClickHouseClientOption.FORMAT.getKey()));
-            }
-            if (requestConfig.containsKey(ClickHouseClientOption.QUERY_ID.getKey())) {
-                req.addHeader(ClickHouseHttpProto.HEADER_QUERY_ID, requestConfig.get(ClickHouseClientOption.QUERY_ID.getKey()).toString());
-            }
+        if (requestConfig.containsKey(ClickHouseClientOption.FORMAT.getKey())) {
+            req.addHeader(ClickHouseHttpProto.HEADER_FORMAT, requestConfig.get(ClickHouseClientOption.FORMAT.getKey()));
+        }
+        if (requestConfig.containsKey(ClickHouseClientOption.QUERY_ID.getKey())) {
+            req.addHeader(ClickHouseHttpProto.HEADER_QUERY_ID, requestConfig.get(ClickHouseClientOption.QUERY_ID.getKey()).toString());
         }
         req.addHeader(ClickHouseHttpProto.HEADER_DATABASE, chConfig.get(ClickHouseClientOption.DATABASE.getKey()));
         req.addHeader(ClickHouseHttpProto.HEADER_DB_USER, chConfig.get(ClickHouseDefaults.USER.getKey()));

client-v2/src/test/java/com/clickhouse/client/HttpTransportTests.java

@@ -43,6 +43,7 @@
 import java.util.concurrent.atomic.AtomicInteger;
 
 import static com.github.tomakehurst.wiremock.stubbing.Scenario.STARTED;
+import static org.junit.Assert.fail;
 
 public class HttpTransportTests extends BaseIntegrationTest {
 
@@ -422,6 +423,9 @@ public void testServerSettings() {
     }
     @Test(groups = { "integration" })
     public void testSSLAuthentication() throws Exception {
+        if (isCloud()) {
+            return; // Current test is working only with local server because of self-signed certificates.
+        }
         ClickHouseNode server = getSecureServer(ClickHouseProtocol.HTTP);
         try (Client client = new Client.Builder().addEndpoint(Protocol.HTTP, "localhost",server.getPort(), true)
                 .setUsername("default")
@@ -449,4 +453,26 @@ public void testSSLAuthentication() throws Exception {
             }
         }
     }
+
+    @Test(groups = { "integration" })
+    public void testSSLAuthentication_invalidConfig() throws Exception {
+        if (isCloud()) {
+            return; // Current test is working only with local server because of self-signed certificates.
+        }
+        ClickHouseNode server = getSecureServer(ClickHouseProtocol.HTTP);
+        try (Client client = new Client.Builder().addEndpoint(Protocol.HTTP, "localhost",server.getPort(), true)
+                .useSSLAuthentication(true)
+                .setUsername("some_user")
+                .setPassword("s3cret")
+                .setRootCertificate("containers/clickhouse-server/certs/localhost.crt")
+                .setClientCertificate("some_user.crt")
+                .setClientKey("some_user.key")
+                .compressServerResponse(false)
+                .build()) {
+            fail("Expected exception");
+        } catch (IllegalArgumentException e) {
+            e.printStackTrace();
+                Assert.assertTrue(e.getMessage().startsWith("Only one of password, access token or SSL authentication"));
+        }
+    }
 }