scolebourne    2003/12/14 05:01:07

  Modified:    collections/src/test/org/apache/commons/collections/iterators
                        AbstractTestIterator.java
               collections/src/test/org/apache/commons/collections/map
                        AbstractTestMap.java
  Log:
  Add additional tests for coverage
  
  Revision  Changes    Path
  1.6       +6 -2      
jakarta-commons/collections/src/test/org/apache/commons/collections/iterators/AbstractTestIterator.java
  
  Index: AbstractTestIterator.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/collections/src/test/org/apache/commons/collections/iterators/AbstractTestIterator.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- AbstractTestIterator.java 18 Nov 2003 22:37:14 -0000      1.5
  +++ AbstractTestIterator.java 14 Dec 2003 13:01:07 -0000      1.6
  @@ -169,6 +169,8 @@
           } catch (NoSuchElementException e) {
           }
           verify();
  +        
  +        assertNotNull(it.toString());
       }
   
       /**
  @@ -203,6 +205,8 @@
               fail("NoSuchElementException must be thrown when Iterator is 
exhausted");
           } catch (NoSuchElementException e) {
           }
  +        
  +        assertNotNull(it.toString());
       }
   
       /**
  
  
  
  1.4       +9 -2      
jakarta-commons/collections/src/test/org/apache/commons/collections/map/AbstractTestMap.java
  
  Index: AbstractTestMap.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/collections/src/test/org/apache/commons/collections/map/AbstractTestMap.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- AbstractTestMap.java      7 Dec 2003 01:21:51 -0000       1.3
  +++ AbstractTestMap.java      14 Dec 2003 13:01:07 -0000      1.4
  @@ -1280,6 +1280,13 @@
               return entry;
           }
   
  +        public void testMapEntrySetRemoveNonMapEntry() {
  +            if (isRemoveSupported() == false) return;
  +            resetFull();
  +            assertEquals(false, getSet().remove(null));
  +            assertEquals(false, getSet().remove(new Object()));
  +        }
  +        
           public void verify() {
               super.verify();
               AbstractTestMap.this.verify();
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to