first pass at system out api

Author: molly-moen

org-code-javabuilder/lib/src/main/java/org/code/javabuilder/ValidationRunner.java

@@ -6,6 +6,7 @@
 import org.code.javabuilder.util.ProjectLoadUtils;
 import org.code.protocol.*;
 import org.code.validation.support.NeighborhoodTracker;
+import org.code.validation.support.SystemOutTracker;
 import org.code.validation.support.ValidationProtocol;
 
 public class ValidationRunner extends BaseTestRunner {
@@ -49,6 +50,6 @@ private void setUpForValidation(URLClassLoader urlClassLoader) throws UserInitia
     JavabuilderContext.getInstance()
         .register(
             ValidationProtocol.class,
-            new ValidationProtocol(mainMethod, new NeighborhoodTracker()));
+            new ValidationProtocol(mainMethod, new NeighborhoodTracker(), new SystemOutTracker()));
   }
 }

org-code-javabuilder/validation/src/main/java/org/code/validation/SystemOutTestRunner.java

@@ -0,0 +1,14 @@
+package org.code.validation;
+
+import java.util.List;
+import org.code.protocol.JavabuilderContext;
+import org.code.validation.support.ValidationProtocol;
+
+public class SystemOutTestRunner {
+  public static List<String> run() {
+    ValidationProtocol protocolInstance =
+        (ValidationProtocol) JavabuilderContext.getInstance().get(ValidationProtocol.class);
+    protocolInstance.invokeMainMethod();
+    return protocolInstance.getSystemOutMessages();
+  }
+}

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

@@ -0,0 +1,25 @@
+package org.code.validation.support;
+
+import java.util.ArrayList;
+import java.util.List;
+import org.code.protocol.ClientMessage;
+import org.code.protocol.ClientMessageType;
+
+public class SystemOutTracker {
+  private List<String> messages;
+
+  public SystemOutTracker() {
+    this.messages = new ArrayList<>();
+  }
+
+  public List<String> getSystemOutMessages() {
+    return this.messages;
+  }
+
+  public void trackEvent(ClientMessage message) {
+    if (message.getType() != ClientMessageType.SYSTEM_OUT) {
+      return;
+    }
+    this.messages.add(message.getValue());
+  }
+}

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

@@ -29,16 +29,16 @@ public void sendMessage(ClientMessage message) {
           delegateOutputAdapter.sendMessage(message);
         }
         break;
-      case NEIGHBORHOOD:
-        if (this.isValidation) {
-          ValidationProtocol protocolInstance =
-              (ValidationProtocol) JavabuilderContext.getInstance().get(ValidationProtocol.class);
-          protocolInstance.trackEvent(message);
-        }
-        break;
       default:
         break;
     }
+    // If this is validation, send all messages to the validation protocol, which will decide
+    // whether or not to track them.
+    if (this.isValidation) {
+      ValidationProtocol protocolInstance =
+          (ValidationProtocol) JavabuilderContext.getInstance().get(ValidationProtocol.class);
+      protocolInstance.trackEvent(message);
+    }
   }
 
   public void setIsValidation(boolean isValidation) {

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

@@ -2,25 +2,36 @@
 
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
+import java.util.List;
 import org.code.protocol.ClientMessage;
 import org.code.protocol.JavabuilderSharedObject;
 import org.code.validation.NeighborhoodLog;
 
 public class ValidationProtocol extends JavabuilderSharedObject {
   private final Method mainMethod;
   private final NeighborhoodTracker neighborhoodTracker;
+  private final SystemOutTracker systemOutTracker;
 
-  public ValidationProtocol(Method mainMethod, NeighborhoodTracker neighborhoodTracker) {
+  public ValidationProtocol(
+      Method mainMethod,
+      NeighborhoodTracker neighborhoodTracker,
+      SystemOutTracker systemOutTracker) {
     this.mainMethod = mainMethod;
     this.neighborhoodTracker = neighborhoodTracker;
+    this.systemOutTracker = systemOutTracker;
   }
 
   public NeighborhoodLog getNeighborhoodLog() {
     return this.neighborhoodTracker.getNeighborhoodLog();
   }
 
+  public List<String> getSystemOutMessages() {
+    return this.systemOutTracker.getSystemOutMessages();
+  }
+
   public void trackEvent(ClientMessage message) {
     this.neighborhoodTracker.trackEvent(message);
+    this.systemOutTracker.trackEvent(message);
   }
 
   public void invokeMainMethod() {

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

@@ -14,7 +14,10 @@ public class UserTestOutputAdapterTest {
 
   @BeforeEach
   public void setUp() {
+
     testOutputAdapter = new UserTestOutputAdapter(delegateOutputAdapter);
+    JavabuilderContext.getInstance()
+        .register(ValidationProtocol.class, mock(ValidationProtocol.class));
   }
 
   @Test