It works in the runtime (I tested without problem).

I'm updating the tests.

Regards
JB

On 03/27/2012 12:08 PM, Jamie G. wrote:
It appears adding the " featuresService.removeRepository(uri);" has
broken several KarArtifactInstallerTest test cases' mocking.

Should be a minor tweak to get them working again.

Cheers,
Jamie

On Tue, Mar 27, 2012 at 5:29 AM,<[email protected]>  wrote:
Author: jbonofre
Date: Tue Mar 27 07:59:27 2012
New Revision: 1305766

URL: http://svn.apache.org/viewvc?rev=1305766&view=rev
Log:
[KARAF-1139] Refresh the feature repository URI in the kar deployer

Modified:
    
karaf/branches/karaf-2.2.x/deployer/kar/src/main/java/org/apache/karaf/deployer/kar/KarArtifactInstaller.java

Modified: 
karaf/branches/karaf-2.2.x/deployer/kar/src/main/java/org/apache/karaf/deployer/kar/KarArtifactInstaller.java
URL: 
http://svn.apache.org/viewvc/karaf/branches/karaf-2.2.x/deployer/kar/src/main/java/org/apache/karaf/deployer/kar/KarArtifactInstaller.java?rev=1305766&r1=1305765&r2=1305766&view=diff
==============================================================================
--- 
karaf/branches/karaf-2.2.x/deployer/kar/src/main/java/org/apache/karaf/deployer/kar/KarArtifactInstaller.java
 (original)
+++ 
karaf/branches/karaf-2.2.x/deployer/kar/src/main/java/org/apache/karaf/deployer/kar/KarArtifactInstaller.java
 Tue Mar 27 07:59:27 2012
@@ -18,7 +18,19 @@
  */
  package org.apache.karaf.deployer.kar;

-import java.io.BufferedOutputStream;
+import org.apache.felix.fileinstall.ArtifactInstaller;
+import org.apache.karaf.features.Feature;
+import org.apache.karaf.features.FeaturesService;
+import org.apache.karaf.features.Repository;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.w3c.dom.Document;
+import org.xml.sax.ErrorHandler;
+import org.xml.sax.SAXException;
+import org.xml.sax.SAXParseException;
+
+import javax.xml.parsers.DocumentBuilder;
+import javax.xml.parsers.DocumentBuilderFactory;
  import java.io.File;
  import java.io.FileOutputStream;
  import java.io.IOException;
@@ -31,20 +43,6 @@ import java.util.List;
  import java.util.zip.ZipEntry;
  import java.util.zip.ZipFile;

-import javax.xml.parsers.DocumentBuilder;
-import javax.xml.parsers.DocumentBuilderFactory;
-
-import org.apache.felix.fileinstall.ArtifactInstaller;
-import org.apache.karaf.features.Feature;
-import org.apache.karaf.features.FeaturesService;
-import org.apache.karaf.features.Repository;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.w3c.dom.Document;
-import org.xml.sax.ErrorHandler;
-import org.xml.sax.SAXException;
-import org.xml.sax.SAXParseException;
-
  public class KarArtifactInstaller implements ArtifactInstaller {

     public static final String FEATURES_CLASSIFIER = "features";
@@ -262,6 +260,7 @@ public class KarArtifactInstaller implem
        private void addToFeaturesRepositories(URI uri)  {
         // URI mvnUri = pathToMvnUri(path);
                try {
+            featuresService.removeRepository(uri);
                        featuresService.addRepository(uri);
             logger.info("Added feature repository '{}'.", uri);
                } catch (Exception e) {



--
Jean-Baptiste Onofré
[email protected]
http://blog.nanthrax.net
Talend - http://www.talend.com

Reply via email to