Enable TestAltinityConnectorSmokeTest in ClickHouse connector

The server version it was upgraded in 0196018 doesn't cause flaky issues with new driver package.
trinodb · Feb 18, 2022 · 3177d31 · 3177d31
1 parent 698ff6c
commit 3177d31
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 25 deletions.
diff --git a/plugin/trino-clickhouse/pom.xml b/plugin/trino-clickhouse/pom.xml
@@ -180,27 +180,4 @@
             <scope>test</scope>
         </dependency>
     </dependencies>
-
-    <profiles>
-        <profile>
-            <id>default</id>
-            <activation>
-                <activeByDefault>true</activeByDefault>
-            </activation>
-            <build>
-                <plugins>
-                    <plugin>
-                        <groupId>org.apache.maven.plugins</groupId>
-                        <artifactId>maven-surefire-plugin</artifactId>
-                        <configuration>
-                            <excludes>
-                                <!-- TODO (https://github.com/trinodb/trino/issues/10752) Enable after resolving flaky test issues -->
-                                <exclude>**/TestAltinityConnectorSmokeTest.java</exclude>
-                            </excludes>
-                        </configuration>
-                    </plugin>
-                </plugins>
-            </build>
-        </profile>
-    </profiles>
 </project>
diff --git a/...o-clickhouse/src/test/java/io/trino/plugin/clickhouse/TestAltinityConnectorSmokeTest.java b/...o-clickhouse/src/test/java/io/trino/plugin/clickhouse/TestAltinityConnectorSmokeTest.java
@@ -32,7 +32,6 @@ protected QueryRunner createQueryRunner()
                 ImmutableMap.of(),
                 ImmutableMap.<String, String>builder()
                         .put("clickhouse.map-string-as-varchar", "true") // To handle string types in TPCH tables as varchar instead of varbinary
-                        .put("clickhouse.legacy-driver", "true")
                         .buildOrThrow(),
                 REQUIRED_TPCH_TABLES);
     }
@@ -42,6 +41,6 @@ public void testRenameSchema()
     {
         // Override because RENAME DATABASE statement isn't supported in Altinity 20.8
         assertThatThrownBy(super::testRenameSchema)
-                .hasMessageMatching("ClickHouse exception, code: 48,.* RENAME DATABASE is not supported.*\\n");
+                .hasMessageContaining("RENAME DATABASE is not supported");
     }
 }