KAFKA-14589 [2/3] Tests of ConsoleGroupCommand rewritten in java

[nizhikov]

This PR is part of #14471 Is contains some of ConsoleGroupCommand tests rewritten in java. Intention of separate PR is to reduce changes and simplify review.

Committer Checklist (excluded from commit message)

• [ ] Verify design and implementation
• [ ] Verify test coverage and CI build status
• [ ] Verify documentation (including upgrade notes)

[nizhikov]

Hello @showuon

Can you, please, take a look? I've checked CI results and it seems OK for me.

[nizhikov]

Hello @mimaison @jolshan Can you, please, take a look?

[nizhikov]

Hello @showuon @jolshan @mimaison

Can you, please, take a look?

[nizhikov]

Hello, @rreddy-22 @dajac

Can you, please, take a look at these changes? It PR moves DescribeConsumerGroupTest and ResetConsumerGroupOffsetTest as part of moving ConsoleGroupCommand to tools.

Having it in trunk will reduce your work while improving ConsoleGroupCommand (no need to duplicate test changes in java and scala) and help moving command to tools.

Big PR where all command code rewritten in java - #14471

Right now, changes in ConsoleGroupCommand leads to conflicts in my PR which is hard to resolve.

[nizhikov]

Hello @chia7712

Please, take a look. If this patch to big to review I can split it to two.

WDYT?

[chia7712]

If this patch to big to review I can split it to two.

please :)

[nizhikov]

@chia7712 Done.

[nizhikov]

@chia7712

Other PR is #15465

Seems, PR's are reviewable now. Please, take a look.

[nizhikov]

@chia7712 This PR ready for review.

Please, take a look.

[nizhikov]

@chia7712 Thanks for the review. All your comments fixed. Please, take a look one more time.

[chia7712]

the failed tests pass on my machine.

./gradlew cleanTest core:test --tests ReplicaManagerTest --tests LogDirFailureTest

will merge it

[chia7712]

@nizhikov thanks for this nice patch

[nizhikov]

@chia7712 Thanks for the review and merge!