changing tests to knownLimitation per jim.

Marking them makes it easier to summarize them for docs.

diff --git a/tests/ajcTests.xml b/tests/ajcTests.xml
index 5f477a19bd6476826c593093dde5085ee07f3552..c524c71911f98e3ff9bbc4c4389a69bf20fb5943 100644 (file)
--- 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 30dfe916044fe74c3ba62afbc4f6a7d1e9eaed3c..50d4ab09007d7b625d36166b55e0027a3dc06c5f 100644 (file)
--- 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>