[Asunit-svn] SF.net SVN: asunit: [198] branches/ityrrell/framework/as25/asunit

[<i_t...@us...>]

Revision: 198
          http://asunit.svn.sourceforge.net/asunit/?rev=198&view=rev
Author:   i_tyrrell
Date:     2008-07-14 04:17:05 -0700 (Mon, 14 Jul 2008)

Log Message:
-----------
Fixed uninitialised itterator variables - for(var i:Number;...==>for(var i:Number=0;...
Removed extraneous trace() statements

Modified Paths:
--------------
    branches/ityrrell/framework/as25/asunit/framework/TestResult.as
    branches/ityrrell/framework/as25/asunit/framework/TestSuite.as
    branches/ityrrell/framework/as25/asunit/textui/ResultPrinter.as
    branches/ityrrell/framework/as25/asunit/textui/TestRunner.as

Modified: branches/ityrrell/framework/as25/asunit/framework/TestResult.as
===================================================================
--- branches/ityrrell/framework/as25/asunit/framework/TestResult.as	2008-07-13 20:11:33 UTC (rev 197)
+++ branches/ityrrell/framework/as25/asunit/framework/TestResult.as	2008-07-14 11:17:05 UTC (rev 198)
@@ -35,7 +35,7 @@
 		fErrors.push(new TestFailure(test, t));
 		var len:Number = fListeners.length;
 		var item:TestListener;
-		for(var i:Number; i < len; i++) {
+		for(var i:Number=0; i < len; i++) {
 			item = TestListener(fListeners[i]);
 			item.addError(test, t);
 		}
@@ -48,7 +48,7 @@
 		fFailures.push(new TestFailure(test, t));
 		var len:Number = fListeners.length;
 		var item:TestListener;
-		for(var i:Number; i < len; i++) {
+		for(var i:Number=0; i < len; i++) {
 			item = TestListener(fListeners[i]);
 			item.addFailure(test, t);
 		}
@@ -64,7 +64,7 @@
 	 */
 	public function removeListener(listener:TestListener):Void {
 		var len:Number = fListeners.length;
-		for(var i:Number; i < len; i++) {
+		for(var i:Number=0; i < len; i++) {
 			if(fListeners[i] == listener) {
 				fListeners.splice(i, 1);
 				return;
@@ -125,7 +125,7 @@
 
 		var len:Number = fListeners.length;
 		var item:TestListener;
-		for(var i:Number; i < len; i++) {
+		for(var i:Number=0; i < len; i++) {
 			item = TestListener(fListeners[i]);
 			item.startTest(test);
 		}
@@ -134,7 +134,7 @@
 	public function startTestMethod(test:Test, method:String):Void {
 		var len:Number = fListeners.length;
 		var item:TestListener;
-		for(var i:Number; i < len; i++) {
+		for(var i:Number=0; i < len; i++) {
 			item = TestListener(fListeners[i]);
 			item.startTestMethod(test, method);
 		}
@@ -143,7 +143,7 @@
 	public function endTestMethod(test:Test, method:String):Void {
 		var len:Number = fListeners.length;
 		var item:TestListener;
-		for(var i:Number; i < len; i++) {
+		for(var i:Number=0; i < len; i++) {
 			item = TestListener(fListeners[i]);
 			item.endTestMethod(test, method);
 		}
@@ -155,7 +155,7 @@
 	public function endTest(test:Test):Void {
 		var len:Number = fListeners.length;
 		var item:TestListener;
-		for(var i:Number; i < len; i++) {
+		for(var i:Number=0; i < len; i++) {
 			item = TestListener(fListeners[i]);
 			item.endTest(test);
 		}

Modified: branches/ityrrell/framework/as25/asunit/framework/TestSuite.as
===================================================================
--- branches/ityrrell/framework/as25/asunit/framework/TestSuite.as	2008-07-13 20:11:33 UTC (rev 197)
+++ branches/ityrrell/framework/as25/asunit/framework/TestSuite.as	2008-07-14 11:17:05 UTC (rev 198)
@@ -49,13 +49,10 @@
 	 * Counts the number of tests that will be run by this Suite.
 	 */
 	public function countTestCases():Number {
-		trace("countTestCases");
 		var count:Number = 0;
 		ArrayUtil.forEach(fTests,
 			function(test:TestCase):Void{
-				trace("test: "+ test);
 				count = count + test.countTestCases();
-				trace("count: " + count);
 			}
 		);
 		return count;

Modified: branches/ityrrell/framework/as25/asunit/textui/ResultPrinter.as
===================================================================
--- branches/ityrrell/framework/as25/asunit/textui/ResultPrinter.as	2008-07-13 20:11:33 UTC (rev 197)
+++ branches/ityrrell/framework/as25/asunit/textui/ResultPrinter.as	2008-07-14 11:17:05 UTC (rev 198)
@@ -21,12 +21,10 @@
 	private var showTrace:Boolean;
 
 	public function ResultPrinter(showTrace:Boolean) {
-		trace("ResultPrinter::constructor");
 		this.showTrace = showTrace==undefined ? false : showTrace;
 	}
 
 	public function setParent(parent:MovieClip):Void{
-		trace("setParent: " + parent);
 		configureAssets(parent);
 		println();
 	}
@@ -222,7 +220,7 @@
 	 */
 	public function startTest(test:Test):Void {
 		var count:Number = test.countTestCases();
-		for(var i:Number; i < count; i++) {
+		for(var i:Number=0; i < count; i++) {
 			print(".");
 			if (fColumn++ >= 80) {
 				println();

Modified: branches/ityrrell/framework/as25/asunit/textui/TestRunner.as
===================================================================
--- branches/ityrrell/framework/as25/asunit/textui/TestRunner.as	2008-07-13 20:11:33 UTC (rev 197)
+++ branches/ityrrell/framework/as25/asunit/textui/TestRunner.as	2008-07-14 11:17:05 UTC (rev 198)
@@ -67,9 +67,6 @@
 		if(testMethod==undefined) testMethod = null;
 		if(showTrace==undefined) showTrace = false;
 		
-		trace("TestRunner::start");
-		trace("testCase: " + testCase);
-		
 //			fscommand("showmenu", "false");
 		try {
 			var instance:Test;
@@ -77,20 +74,17 @@
 				instance = new testCase(testMethod);
 			}
 			else {
-				trace("==============create test case");
 				instance = new testCase();
 			}
 			return doRun(instance, showTrace);
 		}
 		catch(e:Error) {
-			trace("an error was thrown");
 			throw new Error("Could not create and run test suite: " + e.toString());
 		}
 		return null;
 	}
 
 	public function doRun(test:Test, showTrace:Boolean):TestResult {
-		trace("TestRunner::doRun");
 		
 		if(showTrace==undefined) showTrace=false;
 		result = new TestResult();


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