nevzheng commented on code in PR #13517:
URL: https://github.com/apache/gravitino/pull/13517#discussion_r4120245387
##########
core/src/test/java/org/apache/gravitino/storage/relational/BackendTestExtension.java:
##########
@@ -121,19 +130,22 @@ public Stream<TestTemplateInvocationContext>
provideTestTemplateInvocationContex
"Running tests with H2 backend only. Set env var 'dockerTest=true'
to include all backends.");
}
- return backendsToTest.stream().map(BackendInvocationContext::new);
+ return backendsToTest.stream()
+ .map(backendType -> new BackendInvocationContext(testMethodName,
backendType));
}
private static class BackendInvocationContext implements
TestTemplateInvocationContext {
+ private final String testMethodName;
private final String backendType;
- public BackendInvocationContext(String backendType) {
+ public BackendInvocationContext(String testMethodName, String backendType)
{
+ this.testMethodName = testMethodName;
this.backendType = backendType;
}
@Override
public String getDisplayName(int invocationIndex) {
- return String.format("[%s Backend]", backendType.toUpperCase());
+ return String.format("%s()[%s Backend]", testMethodName,
backendType.toUpperCase());
Review Comment:
Done — dropped the hardcoded () in 654cfe33a.
--
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]
