Copilot commented on code in PR #11593:
URL: https://github.com/apache/gravitino/pull/11593#discussion_r3393704766
##########
spark-connector/spark-common/src/test/java/org/apache/gravitino/spark/connector/integration/test/iceberg/SparkIcebergCatalogIT.java:
##########
@@ -125,6 +124,72 @@ protected boolean supportsUpdateColumnPosition() {
return true;
}
+ @Override
+ protected boolean supportsIcebergPartitionTransforms() {
+ return true;
+ }
+
+ @Override
+ protected void testIcebergPartitions() {
+ Map<String, String> partitionPaths = new java.util.HashMap<>();
Review Comment:
`testIcebergPartitions()` overrides the superclass `@Test` method but does
not declare `@Test` (or `@EnabledIf`) itself. JUnit 5 will not treat this
overriding method as a test, so the Iceberg partition-transform coverage can be
silently skipped.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
