You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by us...@apache.org on 2010/10/18 17:55:42 UTC

svn commit: r1023870 - /lucene/dev/trunk/solr/src/test/org/apache/solr/SolrInfoMBeanTest.java

Author: uschindler
Date: Mon Oct 18 15:55:41 2010
New Revision: 1023870

URL: http://svn.apache.org/viewvc?rev=1023870&view=rev
Log:
fix last commit

Modified:
    lucene/dev/trunk/solr/src/test/org/apache/solr/SolrInfoMBeanTest.java

Modified: lucene/dev/trunk/solr/src/test/org/apache/solr/SolrInfoMBeanTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/src/test/org/apache/solr/SolrInfoMBeanTest.java?rev=1023870&r1=1023869&r2=1023870&view=diff
==============================================================================
--- lucene/dev/trunk/solr/src/test/org/apache/solr/SolrInfoMBeanTest.java (original)
+++ lucene/dev/trunk/solr/src/test/org/apache/solr/SolrInfoMBeanTest.java Mon Oct 18 15:55:41 2010
@@ -83,6 +83,8 @@ public class SolrInfoMBeanTest extends L
     }
     assertTrue( "there are at least 10 SolrInfoMBean that should be found in the classpath, found " + checked, checked > 10 );
   }
+  
+  static final String FOLDER = File.separator + "build" + File.separator + "solr" + File.separator + "org" + File.separator + "apache" + File.separator + "solr" + File.separator;
 
   private static List<Class> getClassesForPackage(String pckgname) throws Exception {
     ArrayList<File> directories = new ArrayList<File>();
@@ -91,11 +93,8 @@ public class SolrInfoMBeanTest extends L
     Enumeration<URL> resources = cld.getResources(path);
     while (resources.hasMoreElements()) {
       final File f = new File(resources.nextElement().toURI());
-      // only iterate classes from the core, not the tests
-      if (!f.toString().contains("build" + File.separator + "solr"))
-        continue;
-      // extra security :-)
-      if (f.toString().contains("tests"))
+      // only iterate classes from the core, not the tests (must be in dir "/build/solr/org"
+      if (!f.toString().contains(FOLDER))
         continue;
       directories.add(f);
     }