Finishing previous commit

ponderosaTX · ponderosaTX · commit 12a07e035674 · 2024-11-04T01:45:47.000-06:00
Changes to be committed:
	modified:   ../../junit/JUnitTestRunner.java
	modified:   InterpreterJVM.java
	modified:   MainJVM.java
diff --git a/drjava/src/edu/rice/cs/drjava/model/junit/JUnitTestRunner.java b/drjava/src/edu/rice/cs/drjava/model/junit/JUnitTestRunner.java
@@ -34,9 +34,9 @@
 import edu.rice.cs.util.Log;
 import edu.rice.cs.util.UnexpectedException;
 
-/** DrJava's own testrunner. It updates the document in the JUnit pane as error and failure events are fired.  
-  * These methods run inan auxiliary thread.  Many methods are synchronized for atomicity to maintain the 
-  * consistency of local state.
+/** Runs in the InterpreterJVM. It uses RMI calls to update the the JUnitPanel as test results are reported.  Some methods are
+  * synchronized to maintain the consistency of local state.  These methods run in an auxiliary thread.
+  * GUI actions must be transferred to the event handling thread (DispatchThread).
   *  @version $Id$
   */
 public class JUnitTestRunner extends BaseTestRunner {
diff --git a/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/InterpreterJVM.java b/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/InterpreterJVM.java
@@ -555,7 +555,7 @@ public void classFileError(ClassFileError e) {
     catch (RemoteException re) { error.log(re); }
   }
   
-  /** Notifies that a suite of tests has started running.  Unsynchronized because it contains a remote call and does
+  /** Notifies the Main JVM that a suite of tests has started running.  Unsynchronized because it contains a remote call and does
     * not involve mutable local state.
     * @param numTests The number of tests in the suite to be run.
     */
@@ -564,7 +564,7 @@ public void testSuiteStarted(int numTests) {
     catch (RemoteException re) { error.log(re); }
   }
   
-  /** Notifies that a particular test has started.  Unsynchronized because it contains a remote call and does not
+  /** Notifies the Main JVM that a particular test has started.  Unsynchronized because it contains a remote call and does not
     * involve mutable local state.
     * @param testName The name of the test being started.
     */
diff --git a/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/MainJVM.java b/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/MainJVM.java
@@ -258,7 +258,7 @@ public void testSuiteStarted(int numTests) {
   /** Called when a particular test is started.  Forwards from the slave JVM to the local JUnit model.
    * @param testName The name of the test being started.
    */
-  public void testStarted(String testName) {;
+  public void testStarted(String testName) {
     _junitModel.testStarted(testName);
   }
  

Original file line number	Diff line number	Diff line change
`@@ -258,7 +258,7 @@ public void testSuiteStarted(int numTests) {`
`258`	`258`	`/** Called when a particular test is started. Forwards from the slave JVM to the local JUnit model.`
`259`	`259`	`* @param testName The name of the test being started.`
`260`	`260`	`*/`
`261`		`- public void testStarted(String testName) {;`
	`261`	`+ public void testStarted(String testName) {`
`262`	`262`	`_junitModel.testStarted(testName);`
`263`	`263`	`}`
`264`	`264`