mhansonp commented on a change in pull request #7124:
URL: https://github.com/apache/geode/pull/7124#discussion_r780594583
##########
File path:
geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/rebalance/PartitionedRegionLoadModelJUnitTest.java
##########
@@ -902,19 +927,555 @@ public void testMoveBuckets() throws
UnknownHostException {
buildDetails(member2, 500, 500, new long[] {0, 0, 0, 0}, new long[]
{0, 0, 0, 0});
model.addRegion("a", Arrays.asList(details1, details2), new
FakeOfflineDetails(), true);
- assertEquals(2, doMoves(new CompositeDirector(false, false, true, true),
model));
+ assertThat(doMoves(new CompositeDirector(false, false, true, true),
model)).isEqualTo(2);
- assertEquals(Collections.emptyList(), bucketOperator.creates);
- assertEquals(Collections.emptyList(), bucketOperator.primaryMoves);
+ assertThat(bucketOperator.creates).isEqualTo(Collections.emptyList());
+ assertThat(bucketOperator.primaryMoves).isEqualTo(Collections.emptyList());
// Two of the buckets should move to member2
List<Move> expectedMoves = new ArrayList<>();
expectedMoves.add(new Move(member1, member2));
expectedMoves.add(new Move(member1, member2));
- assertEquals(expectedMoves, bucketOperator.bucketMoves);
+ assertThat(bucketOperator.bucketMoves).isEqualTo(expectedMoves);
+ }
+
+ @Test
+ public void testRemoveBuckets() throws UnknownHostException {
Review comment:
I think it would be best to work together to get to more meaningful
names for you. I will ping you.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
