You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jackrabbit.apache.org by Gennady Azarenkov <ga...@gmail.com> on 2006/02/27 16:59:12 UTC

TCK: NodeTest.testRemoveInvalidItemStateException()

hi,

It seems that 
         testSession.save();
should be added after 
         defaultTestNodeSession2.remove();

in the NodeTest.testRemoveInvalidItemStateException()
         try {
                defaultTestNodeSession2.remove();
                fail("Removing a node already deleted by other session should throw an InvalidItemStateException!");
            } catch (InvalidItemStateException e) {
                //ok, works as expected
            }
as InvalidItemStateException  can be thrown by save()

regards,

gena

Re: TCK: NodeTest.testRemoveInvalidItemStateException()

Posted by Marcel Reutegger <ma...@gmx.net>.

Hi Gena,

same here. this has been changed already:
http://svn.apache.org/viewcvs.cgi/incubator/jackrabbit/trunk/core/src/test/org/apache/jackrabbit/test/api/NodeTest.java?rev=209590&r1=208754&r2=209590&diff_format=h

See also:
http://thread.gmane.org/gmane.comp.apache.jackrabbit.devel/2674

regards
  marcel

Gennady Azarenkov wrote:
> hi,
> 
> It seems that 
>          testSession.save();
> should be added after 
>          defaultTestNodeSession2.remove();
> 
> in the NodeTest.testRemoveInvalidItemStateException()
>          try {
>                 defaultTestNodeSession2.remove();
>                 fail("Removing a node already deleted by other session should throw an InvalidItemStateException!");
>             } catch (InvalidItemStateException e) {
>                 //ok, works as expected
>             }
> as InvalidItemStateException  can be thrown by save()
> 
> regards,
> 
> gena
> 
>