Merge pull request #2432 from ClickHouse/fix_tests_running

[repo] Make tests run even if group is not set

Author: chernser

clickhouse-http-client/src/test/java/com/clickhouse/client/http/ClickHouseHttpClientTest.java

@@ -233,7 +233,7 @@ public void testPingFailure() {
         }
     }
 
-    @Test // (groups = "integration")
+    @Test(groups = "integration")
     public void testTransaction() throws ClickHouseException {
         if (isCloud()) return; //TODO: testTransaction - Revisit, see: https://github.com/ClickHouse/clickhouse-java/issues/1747
         testAbortTransaction();

clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/AccessManagementTest.java

@@ -17,6 +17,7 @@
 import java.util.Arrays;
 import java.util.Properties;
 
+@Test(groups = { "integration" })
 public class AccessManagementTest extends JdbcIntegrationTest {
     @BeforeMethod(groups = "integration")
     public void setV1() {

clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/ClickHouseStatementTest.java

@@ -60,6 +60,7 @@
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicReference;
 
+@Test(groups = { "integration" })
 public class ClickHouseStatementTest extends JdbcIntegrationTest {
     @BeforeMethod(groups = "integration")
     public void setV1() {

clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/GenericJDBCTest.java

@@ -9,6 +9,7 @@
 
 import static org.testng.Assert.assertThrows;
 
+@Test(groups = { "integration" })
 public class GenericJDBCTest extends JdbcIntegrationTest {
     public Connection getConnection(Properties properties) throws SQLException {
         if (properties == null) {

clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/JdbcIssuesTest.java

@@ -12,6 +12,7 @@
 import java.sql.Statement;
 import java.util.Properties;
 
+@Test(groups = { "integration" })
 public class JdbcIssuesTest extends JdbcIntegrationTest {
     @BeforeMethod(groups = "integration")
     public void setV1() {

clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/internal/ClickHouseConnectionImplTest.java

@@ -12,9 +12,15 @@
 
 import org.testng.Assert;
 import org.testng.annotations.BeforeClass;
+import org.testng.annotations.BeforeMethod;
 import org.testng.annotations.Test;
 
+@Test(groups = "integration")
 public class ClickHouseConnectionImplTest extends JdbcIntegrationTest {
+    @BeforeMethod(groups = "integration")
+    public void setV1() {
+        System.setProperty("clickhouse.jdbc.v1","true");
+    }
     @Test(groups = "integration")
     public void testManualCommit() throws SQLException {
         if (isCloud()) return; //TODO: testManualCommit - Revisit, see: https://github.com/ClickHouse/clickhouse-java/issues/1747

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

@@ -54,6 +54,7 @@
 import static com.github.tomakehurst.wiremock.stubbing.Scenario.STARTED;
 import static org.junit.Assert.fail;
 
+@Test(groups = {"integration"})
 public class HttpTransportTests extends BaseIntegrationTest {
 
     @Test(groups = {"integration"},dataProvider = "testConnectionTTLProvider")

client-v2/src/test/java/com/clickhouse/client/insert/InsertTests.java

@@ -68,6 +68,7 @@
 import static org.testng.Assert.assertTrue;
 import static org.testng.Assert.fail;
 
+@Test(groups = {"integration"})
 public class InsertTests extends BaseIntegrationTest {
     private Client client;
     private InsertSettings settings;

client-v2/src/test/java/com/clickhouse/client/query/QueryTests.java

@@ -90,6 +90,7 @@
 import java.util.stream.Collectors;
 import java.util.stream.IntStream;
 
+@Test(groups = {"integration"})
 public class QueryTests extends BaseIntegrationTest {
 
     private final static Random RANDOM = new Random();
@@ -314,7 +315,8 @@ public void testQueryAll() throws Exception {
     public void testQueryAllSimple() throws Exception {
         testQueryAllSimple(10);
     }
-    public void testQueryAllSimple(int numberOfRecords) throws Exception {
+
+    void testQueryAllSimple(int numberOfRecords) throws Exception {
         GenericRecord record = client.queryAll("SELECT number FROM system.numbers LIMIT " + numberOfRecords).stream().findFirst().get();
         Assert.assertNotNull(record);
     }
@@ -1304,7 +1306,7 @@ private static String sq(String str) {
         return "\'" + str + "\'";
     }
 
-    public void testDataTypes(List<String> columns, List<Supplier<String>> valueGenerators, List<Consumer<ClickHouseBinaryFormatReader>> verifiers) {
+    void testDataTypes(List<String> columns, List<Supplier<String>> valueGenerators, List<Consumer<ClickHouseBinaryFormatReader>> verifiers) {
         final String table = "data_types_test_table";
 
         try {

jdbc-v2/src/test/java/com/clickhouse/jdbc/PreparedStatementTest.java

@@ -39,6 +39,7 @@
 import static org.testng.Assert.assertThrows;
 import static org.testng.Assert.assertTrue;
 
+@Test(groups = { "integration" })
 public class PreparedStatementTest extends JdbcIntegrationTest {
 
     @Test(groups = { "integration" })