reversed changes to Main that were used for testing, updated tests for off by...

reversed changes to Main that were used for testing, updated tests for off by one error/higher error coverage

src/test/java/dotsandboxes/DotsAndBoxesGridTest.java

@@ -27,28 +27,50 @@ public class DotsAndBoxesGridTest {
 
     @Test
     public void boxCompleteDetectsCompletedBoxes() {
-        DotsAndBoxesGrid case1 = new DotsAndBoxesGrid(5, 5, 2);
+        DotsAndBoxesGrid case1 = new DotsAndBoxesGrid(2, 2, 2);
         case1.drawHorizontal(0, 0, 0);
-        case1.drawHorizontal(0, 0, 0);
-        case1.drawVertical(0, 0, 1);
         case1.drawHorizontal(0, 1, 0);
+        case1.drawVertical(0, 0, 1);
         case1.drawVertical(1, 0, 1);
         assertTrue(case1.boxComplete(0, 0));
     }
 
     @Test
     public void boxCompleteDetectsIncompleteBoxes() {
-        DotsAndBoxesGrid case1 = new DotsAndBoxesGrid(5, 5, 2);
-        case1.drawHorizontal(0, 0, 0);
-        case1.drawVertical(0, 0, 1);
-        case1.drawHorizontal(0, 1, 0);
-        case1.drawVertical(1, 0, 1);
-        assertFalse(case1.boxComplete(2, 2));
+        DotsAndBoxesGrid topHorizontal = new DotsAndBoxesGrid(2, 2, 2);
+        DotsAndBoxesGrid botHorizontal = new DotsAndBoxesGrid(2, 2, 2);
+        DotsAndBoxesGrid leftVertical = new DotsAndBoxesGrid(2, 2, 2);
+        DotsAndBoxesGrid rightVertical = new DotsAndBoxesGrid(2, 2, 2);
+
+        // top horizontal not drawn
+        topHorizontal.drawHorizontal(0, 1, 0);
+        topHorizontal.drawVertical(0, 0, 1);
+        topHorizontal.drawVertical(1, 0, 1);
+
+        // bot horizontal not drawn
+        botHorizontal.drawHorizontal(0, 0, 0);
+        botHorizontal.drawVertical(0, 0, 1);
+        botHorizontal.drawVertical(1, 0, 1);
+
+        // left vertical not drawn
+        leftVertical.drawHorizontal(0, 0, 0);
+        leftVertical.drawHorizontal(0, 1, 0);
+        leftVertical.drawVertical(1, 0, 1);
+
+        // right vertical not drawn
+        rightVertical.drawHorizontal(0, 0, 0);
+        rightVertical.drawHorizontal(0, 1, 0);
+        rightVertical.drawVertical(0, 0, 1);
+
+        assertFalse(topHorizontal.boxComplete(0, 0));
+        assertFalse(botHorizontal.boxComplete(0, 0));
+        assertFalse(leftVertical.boxComplete(0, 0));
+        assertFalse(rightVertical.boxComplete(0, 0));
     }
 
     @Test
     public void drawMethodsDetectRedrawnLines() {
-        DotsAndBoxesGrid case2 = new DotsAndBoxesGrid(5, 5, 2);
+        DotsAndBoxesGrid case2 = new DotsAndBoxesGrid(2, 2, 2);
         case2.drawHorizontal(0, 0, 0);
         case2.drawVertical(0, 0, 1);
         assertThrows(RuntimeException.class, () -> case2.drawHorizontal(0, 0, 0));

[Rohan Gene Isaac Vance]

Didn't add Main.java, needed to comment out bugfixes. Attempting in next commit.