implements two missing unit tests

Author: xinri

exercises/practice/two-bucket/.meta/config.json

@@ -14,7 +14,8 @@
     "muzimuzhi",
     "sjwarner-bp",
     "SleeplessByte",
-    "sshine"
+    "sshine",
+    "Xinri"
   ],
   "files": {
     "solution": [

exercises/practice/two-bucket/src/test/java/TwoBucketTest.java

@@ -1,4 +1,5 @@
 import org.junit.jupiter.api.Disabled;
+import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
@@ -7,6 +8,7 @@
 public class TwoBucketTest {
 
     @Test
+    @DisplayName("Measure using bucket one of size 3 and bucket two of size 5 - start with bucket one")
     public void testBucketOneSizeThreeBucketTwoSizeFiveStartWithOne() {
 
         Result bucketResult = new TwoBucket(3, 5, 1, "one").getResult();
@@ -19,6 +21,7 @@ public void testBucketOneSizeThreeBucketTwoSizeFiveStartWithOne() {
 
     @Disabled("Remove to run test")
     @Test
+    @DisplayName("Measure using bucket one of size 3 and bucket two of size 5 - start with bucket two")
     public void testBucketOneSizeThreeBucketTwoSizeFiveStartWithTwo() {
 
         Result bucketResult = new TwoBucket(3, 5, 1, "two").getResult();
@@ -31,6 +34,7 @@ public void testBucketOneSizeThreeBucketTwoSizeFiveStartWithTwo() {
 
     @Disabled("Remove to run test")
     @Test
+    @DisplayName("Measure using bucket one of size 7 and bucket two of size 11 - start with bucket one")
     public void testBucketOneSizeSevenBucketTwoSizeElevenStartWithOne() {
 
         Result bucketResult = new TwoBucket(7, 11, 2, "one").getResult();
@@ -43,6 +47,7 @@ public void testBucketOneSizeSevenBucketTwoSizeElevenStartWithOne() {
 
     @Disabled("Remove to run test")
     @Test
+    @DisplayName("Measure using bucket one of size 7 and bucket two of size 11 - start with bucket two")
     public void testBucketOneSizeSevenBucketTwoSizeElevenStartWithTwo() {
 
         Result bucketResult = new TwoBucket(7, 11, 2, "two").getResult();
@@ -55,6 +60,7 @@ public void testBucketOneSizeSevenBucketTwoSizeElevenStartWithTwo() {
 
     @Disabled("Remove to run test")
     @Test
+    @DisplayName("Measure one step using bucket one of size 1 and bucket two of size 3 - start with bucket two")
     public void testBucketOneSizeOneBucketTwoSizeThreeStartWithTwo() {
 
         Result bucketResult = new TwoBucket(1, 3, 3, "two").getResult();
@@ -67,6 +73,7 @@ public void testBucketOneSizeOneBucketTwoSizeThreeStartWithTwo() {
 
     @Disabled("Remove to run test")
     @Test
+    @DisplayName("Measure using bucket one of size 2 and bucket two of size 3 - " + "start with bucket one and end with bucket two")
     public void testBucketOneSizeTwoBucketTwoSizeThreeStartWithOne() {
 
         Result bucketResult = new TwoBucket(2, 3, 3, "one").getResult();
@@ -79,15 +86,42 @@ public void testBucketOneSizeTwoBucketTwoSizeThreeStartWithOne() {
 
     @Disabled("Remove to run test")
     @Test
+    @DisplayName("Measure using bucket one much bigger than bucket two")
+    public void testBucketOneMuchBiggerThanBucketTwo() {
+
+        Result bucketResult = new TwoBucket(5, 1, 2, "one").getResult();
+
+        assertThat(bucketResult.getTotalMoves()).isEqualTo(6);
+        assertThat(bucketResult.getFinalBucket()).isEqualTo("one");
+        assertThat(bucketResult.getOtherBucket()).isEqualTo(1);
+
+    }
+
+    @Disabled("Remove to run test")
+    @Test
+    @DisplayName("Measure using bucket one much smaller than bucket two")
+    public void testBucketOneMuchSmallerThanBucketTwo() {
+
+        Result bucketResult = new TwoBucket(3, 15, 9, "one").getResult();
+
+        assertThat(bucketResult.getTotalMoves()).isEqualTo(6);
+        assertThat(bucketResult.getFinalBucket()).isEqualTo("two");
+        assertThat(bucketResult.getOtherBucket()).isEqualTo(0);
+
+    }
+
+    @Disabled("Remove to run test")
+    @Test
+    @DisplayName("Not possible to reach the goal")
     public void testReachingGoalIsImpossible() {
-        
-        assertThatExceptionOfType(UnreachableGoalException.class)
-                .isThrownBy(() -> new TwoBucket(6, 15, 5, "one").getResult());
+
+        assertThatExceptionOfType(UnreachableGoalException.class).isThrownBy(() -> new TwoBucket(6, 15, 5, "one").getResult());
 
     }
 
     @Disabled("Remove to run test")
     @Test
+    @DisplayName("With the same buckets but a different goal, then it is possible")
     public void testBucketOneSizeSixBucketTwoSizeFifteenStartWithOne() {
 
         Result bucketResult = new TwoBucket(6, 15, 9, "one").getResult();
@@ -96,14 +130,14 @@ public void testBucketOneSizeSixBucketTwoSizeFifteenStartWithOne() {
         assertThat(bucketResult.getFinalBucket()).isEqualTo("two");
         assertThat(bucketResult.getOtherBucket()).isEqualTo(0);
 
-    }   
+    }
 
     @Disabled("Remove to run test")
     @Test
+    @DisplayName("Goal larger than both buckets is impossible")
     public void testGoalLargerThanBothBucketsIsImpossible() {
 
-        assertThatExceptionOfType(UnreachableGoalException.class)
-                .isThrownBy(() -> new TwoBucket(5, 7, 8, "one").getResult());
+        assertThatExceptionOfType(UnreachableGoalException.class).isThrownBy(() -> new TwoBucket(5, 7, 8, "one").getResult());
 
     }
 }