Details
-
Sub-task
-
Status: Resolved
-
Minor
-
Resolution: Done
-
None
Description
TestStorageContainerManager has some items to be cleaned up:
- Disabled test cases related to non-HA to HA change. This used to be not supported scenario, but was recently implemented in
HDDS-9897. Since that task also added a new test case, testSCMReinitializationWithHAEnabled and testSCMReinitializationWithHAUpgrade can be removed now. - Test cases can now use MiniOzoneCluster wrapped in try-with-resources blocks to ensure the cluster is shut down even if waitForClusterToBeReady() and other operations fail.
- Thread.currentThread().sleep() can be replaced with Thread.sleep().
- Convert try-catch blocks to assertThrows() where applicable (where exception is expected and try block calls fail().
- Replace usage of GenericTestUtils.getTempPath with @TempDir.
Attachments
Issue Links
- links to