You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by sc...@apache.org on 2003/08/31 15:09:54 UTC

cvs commit: jakarta-commons/collections/src/test/org/apache/commons/collections TestMap.java TestSequencedHashMap.java TestObject.java TestExtendedProperties.java TestCollectionUtils.java BulkTest.java

scolebourne    2003/08/31 06:09:54

  Modified:    collections/src/test/org/apache/commons/collections
                        TestMap.java TestSequencedHashMap.java
                        TestObject.java TestExtendedProperties.java
                        TestCollectionUtils.java BulkTest.java
  Log:
  Javadoc spelling mistakes
  bug 22843, from Janek Bogucki
  
  Revision  Changes    Path
  1.25      +8 -8      jakarta-commons/collections/src/test/org/apache/commons/collections/TestMap.java
  
  Index: TestMap.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons/collections/src/test/org/apache/commons/collections/TestMap.java,v
  retrieving revision 1.24
  retrieving revision 1.25
  diff -u -r1.24 -r1.25
  --- TestMap.java	12 Jul 2003 15:47:53 -0000	1.24
  +++ TestMap.java	31 Aug 2003 13:09:53 -0000	1.25
  @@ -77,7 +77,7 @@
    * operations, simply extend this class, and implement the {@link
    * #makeEmptyMap()} method.
    * <p>
  - * On the other hand, if your map implemenation is wierd, you may have to
  + * On the other hand, if your map implementation is weird, you may have to
    * override one or more of the other protected methods.  They're described
    * below.<P>
    *
  @@ -269,7 +269,7 @@
       /**
        *  Returns the set of values in the mappings used to test the map.  This
        *  method must return an array with the same length as {@link
  -     *  #getSampleKeys()}.  The default implementation contructs a set of
  +     *  #getSampleKeys()}.  The default implementation constructs a set of
        *  String values and includes a single null value if {@link
        *  #useNullValue()} returns <code>true</code>, and includes two values
        *  that are the same if {@link #useDuplicateValues()} returns
  @@ -708,7 +708,7 @@
   	    assertTrue("Map should contain new value after put",
   		       map.containsValue(newValues[i]));
   
  -	    // if duplicates are allowed, we're not guarunteed that the value
  +	    // if duplicates are allowed, we're not guaranteed that the value
   	    // no longer exists, so don't try checking that.
   	    if(!useDuplicateValues()) {
   		assertTrue("Map should not contain old value after second put",
  @@ -1110,7 +1110,7 @@
           //assertEquals("Map should still equal HashMap", confirmed, map);
           // this works though and performs the same verification:
           assertTrue("Map should still equal HashMap", map.equals(confirmed));
  -        // TODO: this should really be rexamined to figure out why LRU map
  +        // TODO: this should really be reexamined to figure out why LRU map
           // behaves like it does (the equals shouldn't modify since all accesses
           // by the confirmed collection should be through an iterator, thus not
           // causing LRUMap to change).
  
  
  
  1.19      +1 -1      jakarta-commons/collections/src/test/org/apache/commons/collections/TestSequencedHashMap.java
  
  Index: TestSequencedHashMap.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons/collections/src/test/org/apache/commons/collections/TestSequencedHashMap.java,v
  retrieving revision 1.18
  retrieving revision 1.19
  diff -u -r1.18 -r1.19
  --- TestSequencedHashMap.java	7 Jan 2003 15:18:15 -0000	1.18
  +++ TestSequencedHashMap.java	31 Aug 2003 13:09:53 -0000	1.19
  @@ -142,7 +142,7 @@
                            origEntry.getKey(), copiedEntry.getKey());
               assertEquals("Cloned value does not match original",
                            origEntry.getValue(), copiedEntry.getValue());
  -            assertEquals("Cloned entry does not match orginal",
  +            assertEquals("Cloned entry does not match original",
                            origEntry, copiedEntry);
           }
           assertTrue("iterator() returned different number of elements than keys()",
  
  
  
  1.21      +3 -3      jakarta-commons/collections/src/test/org/apache/commons/collections/TestObject.java
  
  Index: TestObject.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons/collections/src/test/org/apache/commons/collections/TestObject.java,v
  retrieving revision 1.20
  retrieving revision 1.21
  diff -u -r1.20 -r1.21
  --- TestObject.java	26 Feb 2003 01:33:22 -0000	1.20
  +++ TestObject.java	31 Aug 2003 13:09:53 -0000	1.21
  @@ -305,7 +305,7 @@
   
       /**
        * Reads a Serialized or Externalized Object from disk.
  -     * Useful for creating compatibility tests betweeen
  +     * Useful for creating compatibility tests between
        * different CVS versions of the same class
        * 
        * @param path path to the serialized Object
  
  
  
  1.7       +5 -5      jakarta-commons/collections/src/test/org/apache/commons/collections/TestExtendedProperties.java
  
  Index: TestExtendedProperties.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons/collections/src/test/org/apache/commons/collections/TestExtendedProperties.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- TestExtendedProperties.java	24 Aug 2003 11:29:52 -0000	1.6
  +++ TestExtendedProperties.java	31 Aug 2003 13:09:53 -0000	1.7
  @@ -94,7 +94,7 @@
   
       public void testRetrieve() {
           /*
  -         * should be emptry and return null
  +         * should be empty and return null
            */
           assertEquals("This returns null", eprop.getProperty("foo"), null);
   
  
  
  
  1.16      +5 -5      jakarta-commons/collections/src/test/org/apache/commons/collections/TestCollectionUtils.java
  
  Index: TestCollectionUtils.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons/collections/src/test/org/apache/commons/collections/TestCollectionUtils.java,v
  retrieving revision 1.15
  retrieving revision 1.16
  diff -u -r1.15 -r1.16
  --- TestCollectionUtils.java	4 Apr 2003 22:22:28 -0000	1.15
  +++ TestCollectionUtils.java	31 Aug 2003 13:09:53 -0000	1.16
  @@ -701,7 +701,7 @@
           Collection colb = new ArrayList();
           
           // ...which contain elta and eltb, 
  -        // repsectively.
  +        // respectively.
           cola.add(elta);
           colb.add(eltb);
           
  
  
  
  1.4       +3 -3      jakarta-commons/collections/src/test/org/apache/commons/collections/BulkTest.java
  
  Index: BulkTest.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons/collections/src/test/org/apache/commons/collections/BulkTest.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- BulkTest.java	19 Feb 2003 20:33:11 -0000	1.3
  +++ BulkTest.java	31 Aug 2003 13:09:53 -0000	1.4
  @@ -128,7 +128,7 @@
       // Note:  BulkTest is Cloneable to make it easier to construct 
       // BulkTest instances for simple test methods that are defined in 
       // anonymous inner classes.  Basically we don't have to worry about
  -    // finding wierd constructors.  (And even if we found them, techinically
  +    // finding weird constructors.  (And even if we found them, technically
       // it'd be illegal for anyone but the outer class to invoke them).  
       // Given one BulkTest instance, we can just clone it and reset the 
       // method name for every simple test it defines.  
  @@ -322,7 +322,7 @@
        *  Adds the simple test defined by the given method to the TestSuite.
        *
        *  @param bulk  The instance of the class that defined the method
  -     *   (I know it's wierd.  But the point is, we can clone the instance
  +     *   (I know it's weird.  But the point is, we can clone the instance
        *   and not have to worry about constructors.)
        *  @param m  The simple test method
        */