geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gdam...@apache.org
Subject cvs commit: incubator-geronimo/sandbox/webdav/src/test/org/apache/geronimo/datastore/impl DirtyMarkerImplTest.java
Date Mon, 01 Mar 2004 13:12:51 GMT
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) {}
       }
       
   }
  
  
  

Mime
View raw message