svn commit: r646514 - /commons/proper/configuration/branches/configuration2_experimental/src/test/java/org/apache/commons/configuration2/TestPropertiesConfiguration.java

[oh...@apache.org]

Author: oheger
Date: Wed Apr  9 13:12:35 2008
New Revision: 646514

URL: http://svn.apache.org/viewvc?rev=646514&view=rev
Log:
Improved type safety

Modified:
    commons/proper/configuration/branches/configuration2_experimental/src/test/java/org/apache/commons/configuration2/TestPropertiesConfiguration.java

Modified: commons/proper/configuration/branches/configuration2_experimental/src/test/java/org/apache/commons/configuration2/TestPropertiesConfiguration.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/configuration2_experimental/src/test/java/org/apache/commons/configuration2/TestPropertiesConfiguration.java?rev=646514&r1=646513&r2=646514&view=diff
==============================================================================
--- commons/proper/configuration/branches/configuration2_experimental/src/test/java/org/apache/commons/configuration2/TestPropertiesConfiguration.java (original)
+++ commons/proper/configuration/branches/configuration2_experimental/src/test/java/org/apache/commons/configuration2/TestPropertiesConfiguration.java Wed Apr  9 13:12:35 2008
@@ -133,7 +133,7 @@
      */
     public void testList() throws Exception
     {
-        List packages = conf.getList("packages");
+        List<?> packages = conf.getList("packages");
         // we should get 3 packages here
         assertEquals(3, packages.size());
     }
@@ -159,7 +159,7 @@
         PropertiesConfiguration checkConfig = new PropertiesConfiguration(filename);
         ConfigurationAssert.assertEquals(conf, checkConfig);
 
-        // Save it again, verifing a save with a filename works.
+        // Save it again, verifying a save with a filename works.
         checkConfig.save();
     }
 
@@ -253,7 +253,7 @@
             throws ConfigurationException
     {
         conf.addProperty("test.dirs", "C:\\Temp\\\\,D:\\Data\\\\,E:\\Test\\");
-        List dirs = conf.getList("test.dirs");
+        List<?> dirs = conf.getList("test.dirs");
         assertEquals("Wrong number of list elements", 3, dirs.size());
         conf.save(testSavePropertiesFile);
 
@@ -417,9 +417,9 @@
             throws ConfigurationException
     {
         PropertiesConfiguration checkConfig = new PropertiesConfiguration(testSavePropertiesFile);
-        for (Iterator i = conf.getKeys(); i.hasNext();)
+        for (Iterator<String> i = conf.getKeys(); i.hasNext();)
         {
-            String key = (String) i.next();
+            String key = i.next();
             assertTrue("The saved configuration doesn't contain the key '" + key + "'", checkConfig.containsKey(key));
             assertEquals("Value of the '" + key + "' property", conf.getProperty(key), checkConfig.getProperty(key));
         }
@@ -513,7 +513,7 @@
      */
     public void testNewLineEscaping()
     {
-        List list = conf.getList("test.path");
+        List<?> list = conf.getList("test.path");
         assertEquals(3, list.size());
         assertEquals("C:\\path1\\", list.get(0));
         assertEquals("C:\\path2\\", list.get(1));
@@ -821,9 +821,9 @@
         conf.save(testSavePropertiesFile);
         PropertiesConfiguration checkConf = new PropertiesConfiguration(
                 testSavePropertiesFile);
-        for (Iterator it = copyConf.getKeys(); it.hasNext();)
+        for (Iterator<String> it = copyConf.getKeys(); it.hasNext();)
         {
-            String key = (String) it.next();
+            String key = it.next();
             assertEquals("Wrong value for property " + key, checkConf
                     .getProperty(key), copyConf.getProperty(key));
         }