Temporarily commenting out unreliable section of the test.

1 files changed, 20 insertions, 14 deletions

diff --git a/ajde/testsrc/org/aspectj/ajde/ReweavableTestCase.java b/ajde/testsrc/org/aspectj/ajde/ReweavableTestCase.java
index 087b7889e..4b6d8e012 100644
--- a/ajde/testsrc/org/aspectj/ajde/ReweavableTestCase.java
+++ b/ajde/testsrc/org/aspectj/ajde/ReweavableTestCase.java
@@ -136,10 +136,12 @@ public class ReweavableTestCase extends AjdeTestCase {
 		assertTrue("bin/Logger.class should exist?!?",fLog.exists());
 		System.out.println("CalculatePI.class is of size: "+fCalc.length());
 		System.out.println("Logger.class is of size: "+fLog.length());
-		assertTrue("Reweavable version should be larger than non-reweavable version of CalculatePI",
-		  fCalc.length()>nonreweavesize_CalculatePI);
-		assertTrue("Reweavable version should be larger than non-reweavable version of Logger",
-		  fLog.length()>nonreweavesize_Logger);
+		// Temporarily remove these tests - it seems the order in which the testXXX methods are run cannot be relied upon
+		// so reweavablesize_XXX fields might not have been set yet.
+//		assertTrue("Reweavable version should be larger than non-reweavable version of CalculatePI",
+//		  fCalc.length()>nonreweavesize_CalculatePI);
+//		assertTrue("Reweavable version should be larger than non-reweavable version of Logger",
+//		  fLog.length()>nonreweavesize_Logger);
 
 		reweavablesize_CalculatePI = (int)fCalc.length();
 		reweavablesize_Logger = (int)fLog.length();
@@ -183,16 +185,20 @@ public class ReweavableTestCase extends AjdeTestCase {
 		int loglen = (int)fLog.length();
 		System.out.println("CalculatePI.class is of size: "+calclen);
 		System.out.println("Logger.class is of size: "+loglen);
-		assertTrue("Reweavable version should be larger than non-reweavable version of CalculatePI",
-		  calclen>nonreweavesize_CalculatePI);
-		assertTrue("Reweavable version should be larger than non-reweavable version of Logger",
-		  loglen>nonreweavesize_Logger);
-		
-		assertTrue("Reweavable (with compression) version should be smaller than reweavable (without compression) version of CalculatePI:" +
			"  Compressed version:"+calclen+"bytes   Non-compressed version:"+reweavablesize_CalculatePI+"bytes",
-		  calclen<reweavablesize_CalculatePI);
-		assertTrue("Reweavable (with compression) version should be smaller than reweavable (without compression) version of Logger"+
-		"  Compressed version:"+loglen+"bytes   Non-compressed version:"+reweavablesize_Logger+"bytes",
-		  loglen<reweavablesize_Logger);  
+		// Temporarily remove these tests - it seems the order in which the testXXX methods are run cannot be relied upon
+		// so reweavablesize_XXX fields might not have been set yet.	
+//		assertTrue("Reweavable version should be larger than non-reweavable version of CalculatePI",
+//		  calclen>nonreweavesize_CalculatePI);
+//		assertTrue("Reweavable version should be larger than non-reweavable version of Logger",
+//		  loglen>nonreweavesize_Logger);
+
+	// Temporarily remove these tests - it seems the order in which the testXXX methods are run cannot be relied upon
+	// so reweavablesize_XXX fields might not have been set yet.	
+//		assertTrue("Reweavable (with compression) version should be smaller than reweavable (without compression) version of CalculatePI:" +
//			"  Compressed version:"+calclen+"bytes   Non-compressed version:"+reweavablesize_CalculatePI+"bytes",
+//		  calclen<reweavablesize_CalculatePI);
+//		assertTrue("Reweavable (with compression) version should be smaller than reweavable (without compression) version of Logger"+
+//		"  Compressed version:"+loglen+"bytes   Non-compressed version:"+reweavablesize_Logger+"bytes",
+//		  loglen<reweavablesize_Logger);  
 		 
 		System.out.println("\n\n\n");
 	}