FORGE-1968: Fixed ExceptionFuture awareness

forge · Aug 13, 2014 · 846f4e6 · 846f4e6
1 parent 88aa328
commit 846f4e6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/container/src/main/java/org/jboss/forge/furnace/impl/addons/AddonImpl.java b/container/src/main/java/org/jboss/forge/furnace/impl/addons/AddonImpl.java
@@ -14,6 +14,7 @@
 import org.jboss.forge.furnace.addons.AddonId;
 import org.jboss.forge.furnace.addons.AddonStatus;
 import org.jboss.forge.furnace.event.EventManager;
+import org.jboss.forge.furnace.impl.util.ExceptionFuture;
 import org.jboss.forge.furnace.impl.util.NullFuture;
 import org.jboss.forge.furnace.repositories.AddonRepository;
 import org.jboss.forge.furnace.spi.ServiceRegistry;
@@ -93,7 +94,7 @@ public AddonStatus getStatus()
             if (getFuture().isDone())
                result = AddonStatus.STARTED;
 
-            if (getFuture().isCancelled())
+            if (getFuture() instanceof ExceptionFuture || getFuture().isCancelled())
                result = AddonStatus.FAILED;
          }
       }

diff --git a/container/src/main/java/org/jboss/forge/furnace/impl/util/ExceptionFuture.java b/container/src/main/java/org/jboss/forge/furnace/impl/util/ExceptionFuture.java
@@ -42,7 +42,7 @@ public boolean isCancelled()
    @Override
    public boolean isDone()
    {
-      return false;
+      return true;
    }
 
    @Override