Fix filterDataMappingSmokeTestData in ClickHouse

trinodb · Jun 14, 2023 · a221b13 · a221b13
1 parent f79196f
commit a221b13
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/...rino-clickhouse/src/test/java/io/trino/plugin/clickhouse/TestClickHouseConnectorTest.java b/...rino-clickhouse/src/test/java/io/trino/plugin/clickhouse/TestClickHouseConnectorTest.java
@@ -580,8 +580,14 @@ protected Optional<DataMappingTestSetup> filterDataMappingSmokeTestData(DataMapp
                 return Optional.empty();
 
             case "date":
-                // TODO (https://github.com/trinodb/trino/issues/7101) enable the test
-                return Optional.empty();
+                // The connector supports date type, but these values are unsupported in ClickHouse
+                // See BaseClickHouseTypeMapping for additional test coverage
+                if (dataMappingTestSetup.getSampleValueLiteral().equals("DATE '0001-01-01'") ||
+                        dataMappingTestSetup.getSampleValueLiteral().equals("DATE '1582-10-05'") ||
+                        dataMappingTestSetup.getHighValueLiteral().equals("DATE '9999-12-31'")) {
+                    return Optional.empty();
+                }
+                return Optional.of(dataMappingTestSetup);
 
             case "time":
             case "time(6)":