[jbpm-commits] JBoss JBPM SVN: r2500 - in jbpm3/trunk/modules/core/src: test/java/org/jbpm/instantiation and 1 other directory.
do-not-reply at jboss.org
do-not-reply at jboss.org
Tue Oct 7 10:20:14 EDT 2008
Author: camunda
Date: 2008-10-07 10:20:14 -0400 (Tue, 07 Oct 2008)
New Revision: 2500
Modified:
jbpm3/trunk/modules/core/src/main/java/org/jbpm/util/ClassLoaderUtil.java
jbpm3/trunk/modules/core/src/test/java/org/jbpm/instantiation/ConfigurableClassloadersTest.java
Log:
fixed correct cleanup in tests (missing jbpmContext.close)
Modified: jbpm3/trunk/modules/core/src/main/java/org/jbpm/util/ClassLoaderUtil.java
===================================================================
--- jbpm3/trunk/modules/core/src/main/java/org/jbpm/util/ClassLoaderUtil.java 2008-10-07 14:08:05 UTC (rev 2499)
+++ jbpm3/trunk/modules/core/src/main/java/org/jbpm/util/ClassLoaderUtil.java 2008-10-07 14:20:14 UTC (rev 2500)
@@ -72,8 +72,12 @@
} else if (jbpmClassloader.equals("context")) {
return Thread.currentThread().getContextClassLoader();
} else if (jbpmClassloader.equals("custom")) {
- String jbpmClassloaderClassname = JbpmConfiguration.Configs.getString("jbpm.customClassLoader.className");
+ String jbpmClassloaderClassname = null;
try {
+ if (!JbpmConfiguration.Configs.hasObject("jbpm.customClassLoader.className")) {
+ throw new JbpmException("'jbpm.classloader' property set to 'custom' but 'jbpm.customClassLoader.className' is empty!");
+ }
+ jbpmClassloaderClassname = JbpmConfiguration.Configs.getString("jbpm.customClassLoader.className");
if (jbpmClassloaderClassname == null) {
throw new JbpmException("'jbpm.classloader' property set to 'custom' but 'jbpm.customClassLoader.className' is empty!");
}
Modified: jbpm3/trunk/modules/core/src/test/java/org/jbpm/instantiation/ConfigurableClassloadersTest.java
===================================================================
--- jbpm3/trunk/modules/core/src/test/java/org/jbpm/instantiation/ConfigurableClassloadersTest.java 2008-10-07 14:08:05 UTC (rev 2499)
+++ jbpm3/trunk/modules/core/src/test/java/org/jbpm/instantiation/ConfigurableClassloadersTest.java 2008-10-07 14:20:14 UTC (rev 2500)
@@ -23,6 +23,7 @@
import org.jbpm.AbstractJbpmTestCase;
import org.jbpm.JbpmConfiguration;
+import org.jbpm.JbpmConfigurationTestHelper;
import org.jbpm.JbpmContext;
import org.jbpm.graph.def.ProcessDefinition;
@@ -35,7 +36,7 @@
* @author bernd.ruecker at camunda.com
*/
public class ConfigurableClassloadersTest extends AbstractJbpmTestCase {
-
+
public void testDefaultBehavior() {
JbpmConfiguration.getInstance();
ClassLoader processClassLoader = JbpmConfiguration.getProcessClassLoader(null);
@@ -107,6 +108,8 @@
fail("we should get an exception because custom class loader class not specified");
}
catch (Exception ex) {}
+
+ jbpmContext.close();
jbpmConfiguration.close();
}
More information about the jbpm-commits
mailing list