pmd: ignoring tests with @ignore, avoiding a little bit of configuration

git-svn-id: https://pmd.svn.sourceforge.net/svnroot/pmd/trunk@7688 51baf565-9d33-0410-a72c-fc3788e3496d
hooperbloob · Jun 2, 2012 · d04e8d3 · d04e8d3
1 parent 9573085
commit d04e8d3
Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/pmd/pom.xml b/pmd/pom.xml
@@ -291,12 +291,6 @@
                 <configuration>
                     <forkMode>once</forkMode>
                     <argLine>-Xmx512m -Dfile.encoding=${project.build.sourceEncoding}</argLine>
-                    <excludes>
-                        <exclude>net/sourceforge/pmd/lang/java/rule/typeresolution/xml/MyTestCase.java</exclude>
-                        <!-- Come on surefire ! be smart, the following class
-                            is not a test to run ! -->
-                        <exclude>net/sourceforge/pmd/testframework/TestDescriptor.java</exclude>
-                    </excludes>
                 </configuration>
             </plugin>
             <plugin>

diff --git a/pmd/src/test/java/net/sourceforge/pmd/lang/java/rule/typeresolution/xml/MyTestCase.java b/pmd/src/test/java/net/sourceforge/pmd/lang/java/rule/typeresolution/xml/MyTestCase.java
@@ -1,5 +1,6 @@
 package net.sourceforge.pmd.lang.java.rule.typeresolution.xml;
 
+import org.junit.Ignore;
 import junit.framework.TestCase;
 
 /**
@@ -10,6 +11,7 @@
  *      
  *      The file is already excluded from maven/surefire.
  */
+@Ignore
 public class MyTestCase extends TestCase {
 
 }
diff --git a/pmd/src/test/java/net/sourceforge/pmd/testframework/TestDescriptor.java b/pmd/src/test/java/net/sourceforge/pmd/testframework/TestDescriptor.java
@@ -8,9 +8,12 @@
 import net.sourceforge.pmd.Rule;
 import net.sourceforge.pmd.lang.LanguageVersion;
 
+import org.junit.Ignore;
+
 /**
  * Stores the information required to run a complete test.
  */
+@Ignore("this is not a unit test")
 public class TestDescriptor {
     private Rule rule;
     private Properties properties;