add reset and tests

Author: molly-moen

org-code-javabuilder/validation/src/main/java/org/code/validation/support/NeighborhoodTracker.java

@@ -18,7 +18,7 @@
 import org.code.validation.Position;
 
 public class NeighborhoodTracker {
-  private final Map<String, PainterTracker> painterTrackers;
+  private Map<String, PainterTracker> painterTrackers;
   private String[][] neighborhoodState;
   private boolean isInitialized;
 
@@ -83,6 +83,11 @@ public void trackEvent(ClientMessage message) {
     }
   }
 
+  public void reset() {
+    this.painterTrackers = new HashMap<>();
+    this.isInitialized = false;
+  }
+
   private void initializeGrid() {
     World world = (World) JavabuilderContext.getInstance().get(World.class);
     final int gridSize = world.getGrid().getSize();

org-code-javabuilder/validation/src/main/java/org/code/validation/support/SystemOutTracker.java

@@ -20,6 +20,15 @@ public void trackEvent(ClientMessage message) {
     if (message.getType() != ClientMessageType.SYSTEM_OUT) {
       return;
     }
-    this.messages.add(message.getValue());
+    String value = message.getValue();
+    // Ignore new lines as those are indications of a println vs a print.
+    // For validation we don't care which is used.
+    if (!value.equals("\n")) {
+      this.messages.add(message.getValue());
+    }
+  }
+
+  public void reset() {
+    this.messages = new ArrayList<>();
   }
 }

org-code-javabuilder/validation/src/main/java/org/code/validation/support/ValidationProtocol.java

@@ -38,6 +38,8 @@ public void invokeMainMethod() {
     if (this.mainMethod == null) {
       throw new ValidationRuntimeException(ExceptionKey.NO_MAIN_METHOD_VALIDATION);
     }
+    this.neighborhoodTracker.reset();
+    this.systemOutTracker.reset();
     try {
       this.mainMethod.invoke(null, new Object[] {null});
     } catch (IllegalAccessException e) {

org-code-javabuilder/validation/src/test/java/org/code/validation/support/NeighborhoodTrackerTest.java

@@ -47,7 +47,7 @@ public void testInitializeMessageCreatesNewPainterTracker() {
 
   @Test
   public void testUpdatesPainterTrackerWithPainterMessages() {
-    final String id = "id";
+    final String id = "id1";
     // Initialize
     unitUnderTest.trackEvent(createInitEvent(id, Direction.EAST, 5, 10, 20));
 
@@ -66,7 +66,7 @@ public void testUpdatesPainterTrackerWithPainterMessages() {
 
     final PainterLog painterLog = unitUnderTest.getNeighborhoodLog().getPainterLogs()[0];
 
-    assertEquals(ID, painterLog.getPainterId());
+    assertEquals(id, painterLog.getPainterId());
     assertEquals(4, painterLog.getEndingPosition().getX());
     assertEquals(10, painterLog.getEndingPosition().getY());
     assertEquals(21, painterLog.getEndingPaintCount());
@@ -132,7 +132,7 @@ public void testSelectsCorrectPainterTrackerById() {
 
   @Test
   public void testUpdatesPainterLogWithNonAnimatedMoves() {
-    final String id = "id";
+    final String id = "id1";
     // Initialize
     unitUnderTest.trackEvent(createInitEvent(id, Direction.EAST, 5, 10, 20));
 
@@ -144,11 +144,38 @@ public void testUpdatesPainterLogWithNonAnimatedMoves() {
 
     final PainterLog painterLog = unitUnderTest.getNeighborhoodLog().getPainterLogs()[0];
 
-    assertEquals(ID, painterLog.getPainterId());
+    assertEquals(id, painterLog.getPainterId());
     assertTrue(painterLog.didActionOnce(NeighborhoodActionType.IS_ON_BUCKET));
     assertFalse(painterLog.didActionAtLeast(NeighborhoodActionType.CAN_MOVE, 1));
   }
 
+  @Test
+  public void resetsSuccessfully() {
+    final String id = "id1";
+    // Initialize
+    unitUnderTest.trackEvent(createInitEvent(id, Direction.EAST, 5, 10, 20));
+    // create paint event
+    final HashMap<String, String> paintDetails = new HashMap<>();
+    paintDetails.put(COLOR, "orange");
+    paintDetails.put(ID, id);
+    unitUnderTest.trackEvent(
+        new NeighborhoodSignalMessage(NeighborhoodSignalKey.PAINT, paintDetails));
+    unitUnderTest.reset();
+    // re-initialize
+    unitUnderTest.trackEvent(createInitEvent(id, Direction.EAST, 5, 10, 20));
+    // create move event
+    final HashMap<String, String> moveDetails = new HashMap<>();
+    moveDetails.put(DIRECTION, Direction.WEST.getDirectionString());
+    moveDetails.put(ID, id);
+    unitUnderTest.trackEvent(
+        new NeighborhoodSignalMessage(NeighborhoodSignalKey.MOVE, moveDetails));
+
+    final PainterLog painterLog = unitUnderTest.getNeighborhoodLog().getPainterLogs()[0];
+
+    assertEquals(false, painterLog.didActionOnce(NeighborhoodActionType.PAINT));
+    assertEquals(true, painterLog.didActionOnce(NeighborhoodActionType.MOVE));
+  }
+
   private NeighborhoodSignalMessage createInitEvent(
       String id, Direction direction, int x, int y, int paint) {
     final HashMap<String, String> details = new HashMap<>();

org-code-javabuilder/validation/src/test/java/org/code/validation/support/SystemOutTrackerTest.java

@@ -0,0 +1,44 @@
+package org.code.validation.support;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+import java.util.List;
+import org.code.protocol.ClientMessageType;
+import org.code.validation.ClientMessageHelper;
+import org.junit.jupiter.api.Test;
+
+public class SystemOutTrackerTest {
+  private SystemOutTracker unitUnderTest;
+
+  @Test
+  public void tracksAllNonNewlineMessages() {
+    unitUnderTest = new SystemOutTracker();
+    // should be tracked
+    unitUnderTest.trackEvent(new ClientMessageHelper(ClientMessageType.SYSTEM_OUT, "hello"));
+    // should be ignored
+    unitUnderTest.trackEvent(new ClientMessageHelper(ClientMessageType.SYSTEM_OUT, "\n"));
+    // should be tracked
+    unitUnderTest.trackEvent(new ClientMessageHelper(ClientMessageType.SYSTEM_OUT, "hello world"));
+
+    List<String> messages = unitUnderTest.getSystemOutMessages();
+    assertEquals(2, messages.size());
+    assertEquals("hello world", messages.get(1));
+  }
+
+  @Test
+  public void resetsSuccessfully() {
+    unitUnderTest = new SystemOutTracker();
+    unitUnderTest.trackEvent(new ClientMessageHelper(ClientMessageType.SYSTEM_OUT, "hello"));
+    // should reset message list
+    unitUnderTest.reset();
+    unitUnderTest.trackEvent(
+        new ClientMessageHelper(ClientMessageType.SYSTEM_OUT, "a new message"));
+    unitUnderTest.trackEvent(new ClientMessageHelper(ClientMessageType.SYSTEM_OUT, "hello world"));
+    unitUnderTest.trackEvent(
+        new ClientMessageHelper(ClientMessageType.SYSTEM_OUT, "a third message"));
+
+    List<String> messages = unitUnderTest.getSystemOutMessages();
+    assertEquals(3, messages.size());
+    assertEquals("a new message", messages.get(0));
+  }
+}