0
@@ -18,6 +18,7 @@
0
 package org.apache.buildr;
0
 
0
 import java.lang.reflect.Method;
0
+import java.lang.reflect.Modifier;
0
 import java.io.File;
0
 import java.io.IOException;
0
 import java.net.URL;
0
@@ -70,6 +71,8 @@ public class JavaTestFilter {
0
   }
0
   
0
   private boolean isTest(Class cls) {
0
+    if (Modifier.isAbstract(cls.getModifiers()) || !Modifier.isPublic(cls.getModifiers()))
0
+      return false;
0
     if (interfaces != null) {
0
       for (Iterator it = interfaces.iterator(); it.hasNext(); ) {
0
         Class iface = (Class) it.next();
0
@@ -113,4 +116,4 @@ public class JavaTestFilter {
0
  * indent-tabs-mode: nil
0
  * c-basic-offset: 2
0
  * End:
0
- */
0
\ No newline at end of file
0
+ */

0
@@ -121,6 +121,25 @@ describe Buildr::JUnit do
0
     project('foo').test.tests.should eql(['InnerClassTest'])
0
   end
0
 
0
+  it 'should ignore abstract classes' do
0
+    write 'src/test/java/AbstractClassTest.java', <<-JAVA
0
+      public abstract class AbstractClassTest extends junit.framework.TestCase {
0
+        public void testNothing() { }
0
+      }
0
+    JAVA
0
+    define('foo').test.invoke
0
+    project('foo').test.tests.should be_empty
0
+  end
0
+
0
+  it 'should ignore classes with no tests in them' do
0
+    write 'src/test/java/NoTests.java', <<-JAVA
0
+      public class NoTests {
0
+      }
0
+    JAVA
0
+    define('foo').test.invoke
0
+    project('foo').test.tests.should be_empty
0
+  end
0
+
0
   it 'should pass when JUnit test case passes' do
0
     write 'src/test/java/PassingTest.java', <<-JAVA
0
       public class PassingTest extends junit.framework.TestCase {