Gargi-jais11 commented on code in PR #9457:
URL: https://github.com/apache/ozone/pull/9457#discussion_r2626479350
##########
hadoop-ozone/integration-test-s3/src/test/java/org/apache/hadoop/ozone/s3/awssdk/v1/AbstractS3SDKV1Tests.java:
##########
@@ -143,7 +146,7 @@
*
*/
@TestMethodOrder(MethodOrderer.MethodName.class)
-public abstract class AbstractS3SDKV1Tests extends OzoneTestBase {
Review Comment:
I believe `AbstractS3SDKV1Tests` should extend `OzoneTestBase` .
`OzoneTestBase.getTestName()` returns the actual test method name (e.g.,
`"testCreateBucket"`), ensuring unique resource names per test
- Without it, tests may interfere with each other and debugging becomes
difficult
- This is a standard pattern used across all Ozone test classes
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
