You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by Michael Dürig <mi...@gmail.com> on 2011/12/13 14:52:33 UTC
Re: svn commit: r1213680 - /jackrabbit/sandbox/microkernel/src/test/java/org/apache/jackrabbit/mk/ConflictingMoveTest.java
On 13.12.11 13:30, dpfister@apache.org wrote:
> Author: dpfister
> Date: Tue Dec 13 13:30:13 2011
> New Revision: 1213680
>
> URL: http://svn.apache.org/viewvc?rev=1213680&view=rev
> Log:
> Replace '\r' (Mac OS line terminator) with '\n' (Unix line terminator)
>
[...]
> @Test
> - public void collidingMove() {
> - String head = mk.getHeadRevision();
> -
> - head = mk.commit("/", "+\"x\" : {} \r+\"y\" : {}\n", head, "");
> -
> - try {
> - mk.commit("/", ">\"x\" : \"y\"", head, "");
> - fail("expected to fail since /y already exists");
> - } catch (MicroKernelException e) {
> - // expected
> - }
> - }
Why remove this test?
Michael
Re: svn commit: r1213680 -
/jackrabbit/sandbox/microkernel/src/test/java/org/apache/jackrabbit/mk/ConflictingMoveTest.java
Posted by Dominique Pfister <dp...@adobe.com>.
Hm, no idea how this happened. No intention, though, I reverted the test removal...
On Dec 13, 2011, at 2:52 PM, Michael Dürig wrote:
>
> On 13.12.11 13:30, dpfister@apache.org wrote:
>> Author: dpfister
>> Date: Tue Dec 13 13:30:13 2011
>> New Revision: 1213680
>>
>> URL: http://svn.apache.org/viewvc?rev=1213680&view=rev
>> Log:
>> Replace '\r' (Mac OS line terminator) with '\n' (Unix line terminator)
>>
>
> [...]
>
>> @Test
>> - public void collidingMove() {
>> - String head = mk.getHeadRevision();
>> -
>> - head = mk.commit("/", "+\"x\" : {} \r+\"y\" : {}\n", head, "");
>> -
>> - try {
>> - mk.commit("/", ">\"x\" : \"y\"", head, "");
>> - fail("expected to fail since /y already exists");
>> - } catch (MicroKernelException e) {
>> - // expected
>> - }
>> - }
>
> Why remove this test?
>
> Michael