changing tests to knownLimitation per jim.

Marking them makes it easier to summarize them for docs.

2 files changed, 45 insertions, 40 deletions

diff --git a/tests/ajcTests.xml b/tests/ajcTests.xml
index 5f477a19b..c524c7191 100644
--- a/tests/ajcTests.xml
+++ b/tests/ajcTests.xml
@@ -5718,4 +5718,48 @@
 			<message kind="error" line="20"/>
         </compile>
     </ajc-test>
+
+    <ajc-test dir="new" pr="660" title="illegal name binding in around cflow"
+      keywords="from-resolved_104,knownLimitation">
+        <compile files="ArgsInCflowCf.java">
+            <message kind="error" line="19"/>
+            <message kind="error" line="29"/>
+            <message kind="error" line="35"/>
+        </compile>
+    </ajc-test>
+
+    <ajc-test dir="incremental/stringliteral"   
+    	keywords="knownLimitation"
+    	title="incrementally change string size and wire in injar classes">
+        <compile staging="true" options="-incremental" 
+        	files="oneInjar.jar,twoInjar.jar"
+        	sourceroots="src"/>
+		<run class="packageOne.Main"
+			options="in packageOne.Main.main(..),
+					 before main packageOne.Main"/>
+        <inc-compile tag="20">
+        	<dir-changes updated="packageOne.Main"/>
+        </inc-compile>
+        <!-- now failing here.  This step passes in non-injar variant. -->
+		<run class="packageOne.Main"
+			options="in longer packageOne.Main.main(..),
+					 before main packageOne.Main"/>
+        <inc-compile tag="30">
+        	<dir-changes added="RunInjarMains"/>
+        </inc-compile>
+		<run class="packageOne.Main"
+			options="in longer packageOne.Main.main(..),
+					 before main InjarOneMain,
+					 before main InjarTwoMain,
+					 before main packageOne.Main"/>
+    </ajc-test>
+
+    <ajc-test dir="bugs" pr="34206" 
+    	keywords="knownLimitation"
+    	title="before():execution(new(..)) does not throw NoAspectBoundException"
+    	comment="correct behavior of this case needs to be thought through">
+        <compile files="AspectInitError.java"/>
+        <run class="AspectInitError"/>
+    </ajc-test>
+    
 </suite>
diff --git a/tests/ajcTestsFailing.xml b/tests/ajcTestsFailing.xml
index 30dfe9160..50d4ab090 100644
--- a/tests/ajcTestsFailing.xml
+++ b/tests/ajcTestsFailing.xml
@@ -1,46 +1,7 @@
 
 <!DOCTYPE suite SYSTEM "../tests/ajcTestSuite.dtd">
 
+<!-- contains valid tests that the compiler has never passed -->
 <suite>
-    <ajc-test dir="new" pr="660" title="illegal name binding in around cflow"
-      keywords="from-resolved_104">
-        <compile files="ArgsInCflowCf.java">
-            <message kind="error" line="19"/>
-            <message kind="error" line="29"/>
-            <message kind="error" line="35"/>
-        </compile>
-    </ajc-test>
-
-    <ajc-test dir="incremental/stringliteral"  
-    		title="incrementally change string size and wire in injar classes">
-        <compile staging="true" options="-incremental" 
-        	files="oneInjar.jar,twoInjar.jar"
-        	sourceroots="src"/>
-		<run class="packageOne.Main"
-			options="in packageOne.Main.main(..),
-					 before main packageOne.Main"/>
-        <inc-compile tag="20">
-        	<dir-changes updated="packageOne.Main"/>
-        </inc-compile>
-        <!-- now failing here.  This step passes in non-injar variant. -->
-		<run class="packageOne.Main"
-			options="in longer packageOne.Main.main(..),
-					 before main packageOne.Main"/>
-        <inc-compile tag="30">
-        	<dir-changes added="RunInjarMains"/>
-        </inc-compile>
-		<run class="packageOne.Main"
-			options="in longer packageOne.Main.main(..),
-					 before main InjarOneMain,
-					 before main InjarTwoMain,
-					 before main packageOne.Main"/>
-    </ajc-test>
-
-	<!--  The correct behavior of this case needs to be thought through -->
-    <ajc-test dir="bugs" pr="34206" 
-    		title="before():execution(new(..)) does not throw NoAspectBoundException">
-        <compile files="AspectInitError.java"/>
-        <run class="AspectInitError"/>
-    </ajc-test>
 
 </suite>