Skip to content

HDDS-15813. Rename TestDataUtil to DataTestUtil#10841

Merged
adoroszlai merged 5 commits into
apache:masterfrom
prathmesh12-coder:HDDS-15813
Jul 25, 2026
Merged

HDDS-15813. Rename TestDataUtil to DataTestUtil#10841
adoroszlai merged 5 commits into
apache:masterfrom
prathmesh12-coder:HDDS-15813

Conversation

@prathmesh12-coder

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

  • Test helpers should not be named Test...
  • Renamed the test helper TestDataUtil.java -> DataTestUtil.java
  • Updated all import references.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-15813

How was this patch tested?

  • Confirmed no remaining TestDataUtil references.
psapate@CWJC2KR9DP ozone % grep -r 'TestDataUtil' ./*   

@@ -56,9 +56,9 @@
/**
* Utility to help to generate test data.

@sravani-revuri sravani-revuri Jul 23, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @prathmesh12-coder .
nit : would it be better to update this javadoc to "Utility class with helper methods for creating test data in integration tests." similar to NodeTestUtil ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @sravani-revuri for review !
Done..updated with "creating and managing test data" since the class also has get* and cleanup* helpers.

@adoroszlai
adoroszlai merged commit 92557fb into apache:master Jul 25, 2026
30 checks passed
@adoroszlai

adoroszlai commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Thanks @prathmesh12-coder for the patch, @sravani-revuri for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants