[HDDS-5758] Speed up TestKeyInputStream and TestChunkInputStream by combining some tests

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.2.0
• Fix Version/s: 1.2.0
• Component/s: test
• Labels: pull-request-available
• Target Version/s: 1.2.0
• Assignee: Stephen O'Donnell
• Reporter: Stephen O'Donnell
• Created: 17/Sep/21 08:36
• Updated: 20/Sep/21 13:25
• Resolved: 20/Sep/21 13:25

Description

TestKeyInputStream takes over 900 seconds to run:

[INFO] Tests run: 16, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 964.773 s - in org.apache.hadoop.ozone.client.rpc.read.TestKeyInputStream
[INFO] Running org.apache.hadoop.ozone.client.rpc.Test2WayCommitInRatis

TestChunkInputStream also takes about:

[INFO] Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 292.226 s - in org.apache.hadoop.ozone.client.rpc.read.TestChunkInputStream
[INFO] Running org.apache.hadoop.ozone.client.rpc.read.TestKeyInputStream

Some of the tests stop and start nodes in the mini-cluster, so its hard to use the same cluster for all tests, but we can combine the "non-destructive" tests into a single test method in each of these class easily saving significant time.

Also in TestInputStreamBase there is a single test, which is executed by both the extending classes, needless repeating it. I have moved in up into one of the extending classes instead.

Issue Links

• links to: GitHub Pull Request #2656