agnes-xinyi-lu commented on code in PR #9955:
URL: https://github.com/apache/gravitino/pull/9955#discussion_r2795472580
##########
iceberg/iceberg-rest-server/src/test/java/org/apache/gravitino/iceberg/service/rest/TestIcebergTableOperations.java:
##########
@@ -694,6 +695,40 @@ void testRemoteSigningNotSupported(Namespace namespace) {
"Error message should mention remote signing: " + errorBody);
}
+ @ParameterizedTest
+
@MethodSource("org.apache.gravitino.iceberg.service.rest.IcebergRestTestUtil#testNamespaces")
Review Comment:
Not sure if I understand the ask. testNamespaces is already being used by
every other method in this class, it will touch a lot of places if we rename
it. Or do you mean to put the list of tables with special characters as a
source in IcebergRestTestUtil as well so other methods could reuse it to test
different table names?
--
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]
