[Mulgara-svn] r1806 - trunk/src/jar/resolver-store/java/org/mulgara/store/statement/xa

[pag at mulgara.org]

Author: pag
Date: 2009-10-05 23:38:39 -0700 (Mon, 05 Oct 2009)
New Revision: 1806

Modified:
   trunk/src/jar/resolver-store/java/org/mulgara/store/statement/xa/XAStatementStoreImplUnitTest.java
Log:
added generics and refactored a fixed IF statement to remove warnings

Modified: trunk/src/jar/resolver-store/java/org/mulgara/store/statement/xa/XAStatementStoreImplUnitTest.java
===================================================================
--- trunk/src/jar/resolver-store/java/org/mulgara/store/statement/xa/XAStatementStoreImplUnitTest.java	2009-10-06 06:37:07 UTC (rev 1805)
+++ trunk/src/jar/resolver-store/java/org/mulgara/store/statement/xa/XAStatementStoreImplUnitTest.java	2009-10-06 06:38:39 UTC (rev 1806)
@@ -67,9 +67,8 @@
 public class XAStatementStoreImplUnitTest
     extends StatementStoreAbstractUnitTest {
 
-  /**
-   * Logger.
-   */
+  /** Logger. */
+  @SuppressWarnings("unused")
   private final static Logger logger = Logger.getLogger(XAStatementStoreImplUnitTest.class);
 
   /**
@@ -85,7 +84,7 @@
   /**
    * Description of the Field
    */
-  private Set releasedNodes = new HashSet();
+  private Set<Long> releasedNodes = new HashSet<Long>();
 
 
   /**
@@ -134,7 +133,7 @@
    * @throws Exception EXCEPTION TO DO
    */
   public void testListeners() throws Exception {
-    if (!XAStatementStoreImpl.RELEASE_NODE_LISTENERS_ENABLED) return;
+    if (!checkIfListenersEnabled()) return;
 
     xaStore.addReleaseNodeListener(
       new ReleaseNodeListener() {
@@ -155,7 +154,17 @@
     assertEquals(Collections.singleton(new Long(5)), releasedNodes);
   }
 
+
   /**
+   * This access is hidden in a method to convince the compiler to look at it.
+   * @return <code>true</code> when listeners are enabled at build time.
+   */
+  private static boolean checkIfListenersEnabled() {
+    return XAStatementStoreImpl.RELEASE_NODE_LISTENERS_ENABLED;
+  }
+
+
+  /**
    * A method to call for each graph before running tests on it.
    *
    * @throws Exception EXCEPTION TO DO