Fixed: ServiceLoader wasn't resolving correctly if no service loader …

…classloader was specified.
testng-team · Aug 8, 2011 · f38bbfa · f38bbfa
1 parent 10a202a
commit f38bbfa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -6,6 +6,7 @@ Added: Injectors created by the @Guice annotation are now shared at the <test> l
 Added: IConfigurationListener is now a public listener
 Added: When a method fails, only dependents of the same instance will be skipped
 Added: parallel=instances for factory instance parallel runs
+Fixed: ServiceLoader wasn't resolving correctly if no service loader classloader was specified
 Fixed: Better ordering with mixed priorities and dependencies
 Fixed: Improved detection of graph cycles in parallel runs
 Fixed: @BeforeTest was invoked multiple times if a factory is used

diff --git a/src/main/java/org/testng/TestNG.java b/src/main/java/org/testng/TestNG.java
@@ -920,7 +920,7 @@ private void addServiceLoaderListeners() {
         parameters.add(m_serviceLoaderClassLoader);
         loadMethod = c.getMethod("load", Class.class, ClassLoader.class);
       } else {
-        loadMethod = c.getMethod("load");
+        loadMethod = c.getMethod("load", Class.class);
       }
       Iterable<ITestNGListener> loader =
           (Iterable<ITestNGListener>) loadMethod.invoke(c, parameters.toArray());