Merge pull request #410 from Fox0117/fix-ClickHouseValueFormatter-needsQuoting-Boolean

Bugfix for Boolean type formatting

Author: alex-krash

src/main/java/ru/yandex/clickhouse/util/ClickHouseValueFormatter.java

@@ -168,6 +168,9 @@ public static boolean needsQuoting(Object o) {
         if (o instanceof Number) {
             return false;
         }
+        if (o instanceof Boolean) {
+            return false;
+        }
         if (o.getClass().isArray()) {
             return false;
         }

src/test/java/ru/yandex/clickhouse/ClickHousePreparedStatementParameterTest.java

@@ -33,4 +33,25 @@ public void testArrayAndCollectionParam() {
         assertEquals(p0.getBatchValue(), p1.getBatchValue());
     }
 
+    @Test
+    public void testBooleanParam() {
+        assertEquals(ClickHousePreparedStatementParameter.fromObject(Boolean.TRUE,
+            TimeZone.getDefault(), TimeZone.getDefault()).getRegularValue(), "1");
+        assertEquals(ClickHousePreparedStatementParameter.fromObject(Boolean.FALSE,
+            TimeZone.getDefault(), TimeZone.getDefault()).getRegularValue(), "0");
+    }
+
+    @Test
+    public void testNumberParam() {
+        assertEquals(ClickHousePreparedStatementParameter.fromObject(10,
+            TimeZone.getDefault(), TimeZone.getDefault()).getRegularValue(), "10");
+        assertEquals(ClickHousePreparedStatementParameter.fromObject(10.5,
+            TimeZone.getDefault(), TimeZone.getDefault()).getRegularValue(), "10.5");
+    }
+
+    @Test
+    public void testStringParam() {
+        assertEquals(ClickHousePreparedStatementParameter.fromObject("someString",
+            TimeZone.getDefault(), TimeZone.getDefault()).getRegularValue(), "'someString'");
+    }
 }