[HDDS-10934] Refactor TestOzoneRpcClient hierarchy

• Type: Sub-task
• Status: Resolved
• Priority: Minor
• Resolution: Implemented
• Fix Version/s: 1.5.0
• Component/s: test
• Labels: pull-request-available
• Assignee: Attila Doroszlai
• Reporter: Attila Doroszlai
• Created: 29/May/24 07:32
• Updated: 31/May/24 20:07
• Resolved: 31/May/24 20:07

Description

1. TestOzoneRpcClient and TestOzoneRpcClientWithRatis are subclasses of TestOzoneRpcClientAbstract. They used to execute the same test cases with OM Ratis disabled (relying on default setting) and enabled (explicitly), respectively. HDDS-4498 enabled OM Ratis by default, so now both classes test with OM Ratis enabled.
2. TestSecureOzoneRpcClient should directly extend TestOzoneRpcClientAbstract to avoid confusion caused by shadowed lifecycle methods.
3. Allow passing MiniOzoneCluster.Builder to startCluster to avoid code duplication in TestSecureOzoneRpcClient.
4. TestOzoneRpcClientAbstract should be renamed to OzoneRpcClientTests to follow naming conventions.
5. Move test cases from TestOzoneRpcClientWithRatis to the base class to extend coverage for other configs.

Issue Links

• links to: GitHub Pull Request #6747