JBRULES-3679 drools-compiler should work in any CDI container

ursmma · Nov 6, 2012 · a21b876 · a21b876
1 parent ce68003
commit a21b876
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/drools-compiler/pom.xml b/drools-compiler/pom.xml
@@ -140,10 +140,10 @@
       <artifactId>cdi-api</artifactId>
       <scope>provided</scope><!-- HACK for OSGi: should be <optional>true</optional> instead -->
     </dependency>
-    <dependency>
+    <dependency><!-- It works in any CDI container, but the weld CDI container is used for testing -->
       <groupId>org.jboss.weld.se</groupId>
       <artifactId>weld-se-core</artifactId>
-      <scope>provided</scope><!-- HACK for OSGi: should be <optional>true</optional> instead -->
+      <scope>test</scope>
     </dependency>     
   </dependencies>
 

diff --git a/drools-compiler/src/main/java/org/drools/cdi/KProjectExtension.java b/drools-compiler/src/main/java/org/drools/cdi/KProjectExtension.java
@@ -43,7 +43,6 @@
 import org.drools.kproject.KSessionImpl;
 import org.drools.runtime.StatefulKnowledgeSession;
 import org.drools.runtime.StatelessKnowledgeSession;
-import org.jboss.weld.environment.se.discovery.url.ClasspathScanningException;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -530,8 +529,8 @@ private String fixURL(URL url) {
 
         try {
             urlPath = URLDecoder.decode( urlPath, "UTF-8" );
-        } catch ( UnsupportedEncodingException ex ) {
-            throw new ClasspathScanningException( "Error decoding URL using UTF-8" );
+        } catch ( UnsupportedEncodingException e ) {
+            throw new IllegalArgumentException( "Error decoding URL (" + url + ") using UTF-8", e );
         }
 
         log.debug( "KProject URL Type + URL: " + urlType + ":" + urlPath );