commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ohe...@apache.org
Subject svn commit: r751493 - /commons/proper/configuration/trunk/src/test/org/apache/commons/configuration/TestSubnodeConfiguration.java
Date Sun, 08 Mar 2009 18:26:32 GMT
Author: oheger
Date: Sun Mar  8 18:26:32 2009
New Revision: 751493

URL: http://svn.apache.org/viewvc?rev=751493&view=rev
Log:
Fixed two findbugs warnings.

Modified:
    commons/proper/configuration/trunk/src/test/org/apache/commons/configuration/TestSubnodeConfiguration.java

Modified: commons/proper/configuration/trunk/src/test/org/apache/commons/configuration/TestSubnodeConfiguration.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/trunk/src/test/org/apache/commons/configuration/TestSubnodeConfiguration.java?rev=751493&r1=751492&r2=751493&view=diff
==============================================================================
--- commons/proper/configuration/trunk/src/test/org/apache/commons/configuration/TestSubnodeConfiguration.java
(original)
+++ commons/proper/configuration/trunk/src/test/org/apache/commons/configuration/TestSubnodeConfiguration.java
Sun Mar  8 18:26:32 2009
@@ -22,13 +22,13 @@
 import java.util.NoSuchElementException;
 import java.util.Set;
 
+import junit.framework.TestCase;
+
 import org.apache.commons.collections.CollectionUtils;
 import org.apache.commons.configuration.reloading.FileAlwaysReloadingStrategy;
 import org.apache.commons.configuration.tree.ConfigurationNode;
 import org.apache.commons.configuration.tree.xpath.XPathExpressionEngine;
 
-import junit.framework.TestCase;
-
 /**
  * Test case for SubnodeConfiguration.
  *
@@ -59,9 +59,6 @@
     /** The subnode configuration to be tested. */
     SubnodeConfiguration config;
 
-    /** Stores the root node of the subnode config. */
-    ConfigurationNode subnode;
-
     /** Stores a counter for the created nodes. */
     int nodeCounter;
 
@@ -77,7 +74,7 @@
         // remove the test output file if necessary
         if (TEST_FILE.exists())
         {
-            TEST_FILE.delete();
+            assertTrue("Could not remove test file", TEST_FILE.delete());
         }
     }
 



Mime
View raw message