bug: Fix to RelationCacheTest failure

2 files changed, 23 insertions, 2 deletions

diff --git a/plugins/org.eclipse.osee.framework.logging/src/org/eclipse/osee/framework/logging/SevereLoggingMonitor.java b/plugins/org.eclipse.osee.framework.logging/src/org/eclipse/osee/framework/logging/SevereLoggingMonitor.java
index 91edbb2..318639d 100644
--- a/plugins/org.eclipse.osee.framework.logging/src/org/eclipse/osee/framework/logging/SevereLoggingMonitor.java
+++ b/plugins/org.eclipse.osee.framework.logging/src/org/eclipse/osee/framework/logging/SevereLoggingMonitor.java
@@ -16,17 +16,28 @@ import java.util.logging.Level;
 
 public class SevereLoggingMonitor implements ILoggerListener {
 
+   private boolean ignore;
    private final List<IHealthStatus> status = new ArrayList<IHealthStatus>();
 
    @Override
    public void log(String loggerName, Level level, String message, Throwable th) {
-      status.add(new BaseStatus(loggerName, level, message, th));
+      if (!ignore) {
+         status.add(new BaseStatus(loggerName, level, message, th));
+      }
    }
 
    public List<IHealthStatus> getAllLogs() {
       return status;
    }
 
+   public void pause() {
+      this.ignore = true;
+   }
+
+   public void resume() {
+      this.ignore = false;
+   }
+
    public List<IHealthStatus> getSevereLogs() {
       List<IHealthStatus> severeStatus = new ArrayList<IHealthStatus>(status.size());
 
diff --git a/plugins/org.eclipse.osee.framework.skynet.core.test/src/org/eclipse/osee/framework/skynet/core/relation/RelationCacheTest.java b/plugins/org.eclipse.osee.framework.skynet.core.test/src/org/eclipse/osee/framework/skynet/core/relation/RelationCacheTest.java
index 6205cc0..2ce27e9 100644
--- a/plugins/org.eclipse.osee.framework.skynet.core.test/src/org/eclipse/osee/framework/skynet/core/relation/RelationCacheTest.java
+++ b/plugins/org.eclipse.osee.framework.skynet.core.test/src/org/eclipse/osee/framework/skynet/core/relation/RelationCacheTest.java
@@ -129,14 +129,24 @@ public class RelationCacheTest {
       Assert.assertEquals(0, relCache.getAll(artfact2).size());
    }
 
+   /**
+    * <p>
+    * When RelationCache.cache() is changed to throw an exception for a duplicate relation, <br/>
+    * then this test should fail and severeLoggingMonitor should not be paused.
+    * </p>
+    * 
+    * @throws Exception
+    */
    @Test
-   public void testCanAddCacheSameRelTwice() {
+   public void testCanAddCacheSameRelTwice() throws Exception {
       RelationCache relCache = new RelationCache();
 
       RelationLink link1 = sourceLinksRelType1.iterator().next();
 
+      severeLoggingMonitor.pause();
       relCache.cache(artfact1, link1);
       relCache.cache(artfact1, link1);
+      severeLoggingMonitor.resume();
 
       Assert.assertEquals(2, relCache.getAll(artfact1).size());
    }