ringhash: improve test coverage

[easwars]

I just noticed that the amount of tests that we have for the ringhash LB policy is woefully short compared to what C-core and Java have. We should add more tests, which exercise different scenarios.

C-core tests are here: https://github.com/grpc/grpc/blob/master/test/cpp/end2end/xds/xds_ring_hash_end2end_test.cc Java tests are here: https://github.com/grpc/grpc-java/blob/master/xds/src/test/java/io/grpc/xds/RingHashLoadBalancerTest.java

[avinilcodes]

I would like to contribute, can you assign this to me.

[dfawley]

@avinilcodes - I can gladly assign this to you, but right now it seems very ambiguous. Do you think you'll be able to assess the tests in our repo and compare it to the tests in the other repos, or would you need us to help identify the missing things more concretely?

[shashank-priyadarshi]

Hi @dfawley , I can work on this.

[ginayeh]

@shashank-priyadarshi as @dfawley mentioned earlier, do you think you'll be able to assess the tests in our repo and compare it to the tests in other repos, or would you need us help identify the missing things more concretely?

[arvindbr8]

@shashank-priyadarshi / @avinilcodes -- friendly ping! We are almost half way through "Hacktoberfest" 😄.

[shashank-priyadarshi]

Sorry, been busy with work lately, couldn't squeeze this in! Hopefull about the weekend!

[atollena]

This lines up well with some work I have in progress on improving ring hash, and such tests would actually help me, so I'm assigning to me.

[easwars]

There are more tests to be written :)

[atollena]

This work uncovered https://github.com/grpc/grpc-go/issues/7363.