cvs commit: incubator-geronimo/sandbox/webdav/src/test/org/apache/geronimo/datastore/impl DirtyMarkerImplTest.java

[gdamour]

gdamour     2004/03/01 05:12:51

  Modified:    sandbox/webdav/src/test/org/apache/geronimo/datastore/impl
                        DirtyMarkerImplTest.java
  Log:
  Reporter: Jacek Laskowski.
  
  JUnit test clean-up.
  
  Revision  Changes    Path
  1.2       +13 -28    
incubator-geronimo/sandbox/webdav/src/test/org/apache/geronimo/datastore/impl/DirtyMarkerImplTest.java
  
  Index: DirtyMarkerImplTest.java
  ===================================================================
  RCS file: 
/home/cvs/incubator-geronimo/sandbox/webdav/src/test/org/apache/geronimo/datastore/impl/DirtyMarkerImplTest.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- DirtyMarkerImplTest.java  29 Feb 2004 13:14:11 -0000      1.1
  +++ DirtyMarkerImplTest.java  1 Mar 2004 13:12:51 -0000       1.2
  @@ -29,60 +29,45 @@
           DirtyMarkerImpl dirtyMarker = new DirtyMarkerImpl();
           dirtyMarker.setIsNew(true);
   
  -        Exception ex = null;
           try {
               dirtyMarker.setIsDelete(true);
  -        } catch (IllegalArgumentException e) {
  -            ex = e;
  -        }
  -        assertNotNull("Transition should ne impossible", ex);
  +            fail("Transition should not be possible");
  +        } catch (IllegalArgumentException e) {}
           
           try {
               dirtyMarker.setIsDirty(true);
  -        } catch (IllegalArgumentException e) {
  -            ex = e;
  -        }
  -        assertNotNull("Transition should ne impossible", ex);
  +            fail("Transition should not be possible");
  +        } catch (IllegalArgumentException e) {}
       }
   
       public void testDirty() {
           DirtyMarkerImpl dirtyMarker = new DirtyMarkerImpl();
           dirtyMarker.setIsDirty(true);
   
  -        Exception ex = null;
           try {
               dirtyMarker.setIsNew(true);
  -        } catch (IllegalArgumentException e) {
  -            ex = e;
  -        }
  -        assertNotNull("Transition should ne impossible", ex);
  +            fail("Transition should not be possible");
  +        } catch (IllegalArgumentException e) {}
           
           try {
               dirtyMarker.setIsDelete(true);
  -        } catch (IllegalArgumentException e) {
  -            ex = e;
  -        }
  -        assertNotNull("Transition should ne impossible", ex);
  +            fail("Transition should not be possible");
  +        } catch (IllegalArgumentException e) {}
       }
       
       public void testDelete() {
           DirtyMarkerImpl dirtyMarker = new DirtyMarkerImpl();
           dirtyMarker.setIsDelete(true);
   
  -        Exception ex = null;
           try {
               dirtyMarker.setIsNew(true);
  -        } catch (IllegalArgumentException e) {
  -            ex = e;
  -        }
  -        assertNotNull("Transition should ne impossible", ex);
  +            fail("Transition should not be possible");
  +        } catch (IllegalArgumentException e) {}
           
           try {
               dirtyMarker.setIsDirty(true);
  -        } catch (IllegalArgumentException e) {
  -            ex = e;
  -        }
  -        assertNotNull("Transition should ne impossible", ex);
  +            fail("Transition should not be possible");
  +        } catch (IllegalArgumentException e) {}
       }
       
   }