You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildr.apache.org by bo...@apache.org on 2009/12/04 21:16:25 UTC

svn commit: r887340 - /buildr/trunk/lib/buildr/java/org/apache/buildr/JavaTestFilter.java

Author: boisvert
Date: Fri Dec  4 20:16:24 2009
New Revision: 887340

URL: http://svn.apache.org/viewvc?rev=887340&view=rev
Log:
Improve error reporting if a class (or its dependencies) can't be loaded

Modified:
    buildr/trunk/lib/buildr/java/org/apache/buildr/JavaTestFilter.java

Modified: buildr/trunk/lib/buildr/java/org/apache/buildr/JavaTestFilter.java
URL: http://svn.apache.org/viewvc/buildr/trunk/lib/buildr/java/org/apache/buildr/JavaTestFilter.java?rev=887340&r1=887339&r2=887340&view=diff
==============================================================================
--- buildr/trunk/lib/buildr/java/org/apache/buildr/JavaTestFilter.java (original)
+++ buildr/trunk/lib/buildr/java/org/apache/buildr/JavaTestFilter.java Fri Dec  4 20:16:24 2009
@@ -116,11 +116,16 @@
   }
 
 
-  public String[] filter(String[] names) throws ClassNotFoundException {
+  public String[] filter(String[] names) throws Throwable {
     Vector testCases = new Vector();
     for (int i = names.length ; i-- > 0 ;) {
-      Class cls = _loader.loadClass(names[i]);
-      if (isTest(cls)) { testCases.add(names[i]); }
+      try {
+        Class cls = _loader.loadClass(names[i]);
+        if (isTest(cls)) { testCases.add(names[i]); }
+      } catch (Throwable e) {
+        System.err.println("JavaTestFilter: Unable to load class "+names[i]+" to dertermine testing ability");
+        throw e;
+      }
     }
     String[] result = new String[testCases.size()];
     testCases.toArray(result);