[P-unit-devel] SF.net SVN: p-unit: [83] trunk/punit

[<zha...@us...>]

Revision: 83
          http://p-unit.svn.sourceforge.net/p-unit/?rev=83&view=rev
Author:   zhanghuangzhu
Date:     2007-05-15 15:06:08 -0700 (Tue, 15 May 2007)

Log Message:
-----------
Andrew Zhang: Refactored "VisitorUtil" to "TraverserUtil"

Modified Paths:
--------------
    trunk/punit/src/org/punit/exception/ConcurrentException.java
    trunk/punit/src/org/punit/reporter/stream/StreamLoggerListener.java
    trunk/punit/src/org/punit/runner/PUnitAbstractRunner.java
    trunk/punit/src/org/punit/runner/method/AbstractTestMethodRunner.java
    trunk/punit.extension/src/org/punit/reporter/chart/image/OverviewImageReporter.java

Added Paths:
-----------
    trunk/punit/src/org/punit/util/TraverserUtil.java

Removed Paths:
-------------
    trunk/punit/src/org/punit/util/VisitorUtil.java

Modified: trunk/punit/src/org/punit/exception/ConcurrentException.java
===================================================================
--- trunk/punit/src/org/punit/exception/ConcurrentException.java	2007-05-15 22:01:02 UTC (rev 82)
+++ trunk/punit/src/org/punit/exception/ConcurrentException.java	2007-05-15 22:06:08 UTC (rev 83)
@@ -20,7 +20,7 @@
 
 	public void printStackTrace() {
 		System.err.println(ConcurrentException.class.getName()+":"); //$NON-NLS-1$
-		VisitorUtil.traverse(_throwables.iterator(), new Traverser() {
+		TraverserUtil.traverse(_throwables.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((Throwable) obj).printStackTrace();
 			}			
@@ -29,7 +29,7 @@
 	
 	public void printStackTrace(final PrintStream ps) {
 		ps.println(ConcurrentException.class.getName() + ":"); //$NON-NLS-1$
-		VisitorUtil.traverse(_throwables.iterator(), new Traverser() {
+		TraverserUtil.traverse(_throwables.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((Throwable) obj).printStackTrace(ps);
 			}
@@ -38,7 +38,7 @@
 	
 	public void printStackTrace(final PrintWriter pw) {
 		pw.println(ConcurrentException.class.getName() + ":"); //$NON-NLS-1$
-		VisitorUtil.traverse(_throwables.iterator(), new Traverser() {
+		TraverserUtil.traverse(_throwables.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((Throwable) obj).printStackTrace(pw);
 			}
@@ -47,7 +47,7 @@
 	
 	public String getMessage() {
 		GetMessageVisitor visitor = new GetMessageVisitor();
-		VisitorUtil.traverse(_throwables.iterator(), visitor);
+		TraverserUtil.traverse(_throwables.iterator(), visitor);
 		return visitor.sb.toString();
 	}
 

Modified: trunk/punit/src/org/punit/reporter/stream/StreamLoggerListener.java
===================================================================
--- trunk/punit/src/org/punit/reporter/stream/StreamLoggerListener.java	2007-05-15 22:01:02 UTC (rev 82)
+++ trunk/punit/src/org/punit/reporter/stream/StreamLoggerListener.java	2007-05-15 22:06:08 UTC (rev 83)
@@ -166,7 +166,7 @@
 			sb.append(_timeWatcher.stringValue());
 			sb.append(LoggerUtil.LINE_SEPERATOR);
 			logger.log(sb.toString(), Level.INFO);
-			VisitorUtil.traverse(_throwableList.iterator(), new Traverser() {
+			TraverserUtil.traverse(_throwableList.iterator(), new Traverser() {
 				int count = 0;
 				public void traverse(Object obj) {
 					logger.log(++count + ")", Level.INFO); //$NON-NLS-1$

Modified: trunk/punit/src/org/punit/runner/PUnitAbstractRunner.java
===================================================================
--- trunk/punit/src/org/punit/runner/PUnitAbstractRunner.java	2007-05-15 22:01:02 UTC (rev 82)
+++ trunk/punit/src/org/punit/runner/PUnitAbstractRunner.java	2007-05-15 22:06:08 UTC (rev 83)
@@ -56,7 +56,7 @@
 			return;
 		}
 		_eventListeners.clear();
-		VisitorUtil.traverse(eventListners.iterator(), new Traverser() {
+		TraverserUtil.traverse(eventListners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				String eventListenerName = (String) obj;
 				PUnitEventListener listener = (PUnitEventListener) ReflectionUtil.newInstance(ReflectionUtil.newClass(eventListenerName));
@@ -109,7 +109,7 @@
 		sb.append(" true "); //$NON-NLS-1$
 		sb.append(vm.punitName());
 		
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				PUnitEventListener eventListener = (PUnitEventListener) obj;
 				sb.append(" "); //$NON-NLS-1$
@@ -120,7 +120,7 @@
 	}
 
 	private void onRunnerStart(final Class clazz) {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onRunnerStart(clazz,
 						PUnitAbstractRunner.this);
@@ -129,7 +129,7 @@
 	}
 
 	private void onRunnerEnd(final Class clazz) {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onRunnerEnd(clazz,
 						PUnitAbstractRunner.this);
@@ -151,7 +151,7 @@
 	}
 
 	private void onClassStart(final Class clazz) {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onClassStart(clazz);
 			}
@@ -159,7 +159,7 @@
 	}
 
 	private void onClassEnd(final Class clazz) {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onClassEnd(clazz);
 			}

Modified: trunk/punit/src/org/punit/runner/method/AbstractTestMethodRunner.java
===================================================================
--- trunk/punit/src/org/punit/runner/method/AbstractTestMethodRunner.java	2007-05-15 22:01:02 UTC (rev 82)
+++ trunk/punit/src/org/punit/runner/method/AbstractTestMethodRunner.java	2007-05-15 22:06:08 UTC (rev 83)
@@ -68,7 +68,7 @@
 	}
 
 	private void onMethodStart(final Method method) {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onMethodStart(method);
 			}
@@ -76,7 +76,7 @@
 	}
 
 	private void onMethodEnd(final Method method, final Throwable t) {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onMethodEnd(method, t);
 			}
@@ -161,7 +161,7 @@
 	}
 
 	private void onWatchersStart() {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onWatchersStart(_watchers);
 			}
@@ -169,7 +169,7 @@
 	}
 	
 	private void onWatcherStart(final Watcher watcher) {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onWatcherStart(watcher);
 			}
@@ -185,7 +185,7 @@
 	}
 
 	private void onWatcherEnd(final Watcher watcher) {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onWatcherEnd(watcher);
 			}
@@ -193,7 +193,7 @@
 	}
 	
 	private void onWatchersEnd() {
-		VisitorUtil.traverse(_eventListeners.iterator(), new Traverser() {
+		TraverserUtil.traverse(_eventListeners.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				((PUnitEventListener) obj).onWatchersEnd(_watchers);
 			}

Copied: trunk/punit/src/org/punit/util/TraverserUtil.java (from rev 67, trunk/punit/src/org/punit/util/VisitorUtil.java)
===================================================================
--- trunk/punit/src/org/punit/util/TraverserUtil.java	                        (rev 0)
+++ trunk/punit/src/org/punit/util/TraverserUtil.java	2007-05-15 22:06:08 UTC (rev 83)
@@ -0,0 +1,11 @@
+package org.punit.util;
+
+import java.util.*;
+
+public class TraverserUtil {
+	public static void traverse(Iterator iter, Traverser traverser) {
+		while (iter.hasNext()) {
+			traverser.traverse(iter.next());
+		}
+	}
+}

Deleted: trunk/punit/src/org/punit/util/VisitorUtil.java
===================================================================
--- trunk/punit/src/org/punit/util/VisitorUtil.java	2007-05-15 22:01:02 UTC (rev 82)
+++ trunk/punit/src/org/punit/util/VisitorUtil.java	2007-05-15 22:06:08 UTC (rev 83)
@@ -1,11 +0,0 @@
-package org.punit.util;
-
-import java.util.*;
-
-public class VisitorUtil {
-	public static void traverse(Iterator iter, Traverser traverser) {
-		while (iter.hasNext()) {
-			traverser.traverse(iter.next());
-		}
-	}
-}

Modified: trunk/punit.extension/src/org/punit/reporter/chart/image/OverviewImageReporter.java
===================================================================
--- trunk/punit.extension/src/org/punit/reporter/chart/image/OverviewImageReporter.java	2007-05-15 22:01:02 UTC (rev 82)
+++ trunk/punit.extension/src/org/punit/reporter/chart/image/OverviewImageReporter.java	2007-05-15 22:06:08 UTC (rev 83)
@@ -70,7 +70,7 @@
 	public void onRunnerEnd(final Class clazz, final Runner runner) {
 		Set entrySet = _overviewDatasets.entrySet();
 		final VM[] vms = _pargs.attachedVMs();
-		VisitorUtil.traverse(entrySet.iterator(), new Traverser() {
+		TraverserUtil.traverse(entrySet.iterator(), new Traverser() {
 			public void traverse(Object obj) {
 				Entry entry = (Entry) obj;
 				Watcher watcher = (Watcher) entry.getKey();


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.