You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by Maxim <ma...@anahoret.com> on 2005/07/07 11:53:55 UTC

Re[2]: Test question: NodeTest.testRemoveInvalidItemStateException()

Hello Marcel.

Thursday, July 7, 2005, 12:01:42 PM, you wrote:

> I agree with you that the test case is too restrictive. The test case
> should try a save and then fail if no InvalidItemStateException is thrown:
>
> Index: NodeTest.java
> ===================================================================
> --- NodeTest.java       (revision 209033)
> +++ NodeTest.java       (working copy)
> @@ -555,6 +555,7 @@
>               // try to remove already deleted node with session 2
>               try {
>                   defaultTestNodeSession2.remove();
> +                testSession.save();
>                   fail("Removing a node already deleted by other session 
> should throw an InvalidItemStateException!");
>               } catch (InvalidItemStateException e) {
>                   //ok, works as expected
>
> WDYT?

Sure this would resolve the issue. But then how this change can be
reflected in the "official" TCK which is available at day.com for
download?

Thank you.

-- 
Best regards, Maxim.
Anahoret Team.


Re: Test question: NodeTest.testRemoveInvalidItemStateException()

Posted by Marcel Reutegger <ma...@gmx.net>.
Hi Maxim,

Maxim wrote:
> Sure this would resolve the issue. But then how this change can be
> reflected in the "official" TCK which is available at day.com for
> download?

This case is covered by the TCK Appeals Process. The test case will then 
be put on an exclude list and the change to the test case will probably 
go into a later maintenance release of the TCK. I'm not sure what the 
exact schedule is for that release...

regards
  marcel