On Thu, 01 Sep 2005, Jani Averbach wrote:
> On 2005-09-01 14:16+0200, Andriy Kulchytskyy wrote:
> > Sorry for submitting invalid patch in previous message. This is my first
> > time when I post a patch to an open source project.
>
> Thanks for taking your time and doing it again Subversion's way.
>
> > Find below a new patch, hope it is ok this time. I did check if it
> > compiles. Though I did not do a regression test.
> > We really need this bug fixed.
>
> Here is the deal: I really don't like to commit this without
> regression test and I know I won't have time until weekend.
>
> So, there is a file:
>
> subversion/bindings/java/javahl/src/org/tigris/subversion/javahl/tests/BasicTests.java
>
> which contains function testBasicLocking(). And there is a code
> pattern (ensure existence of exception) how to test this at line 91 in
> the same file.
>
> If you don't have time to do the regression test, that's fine, I will
> do it when I have time to commit.
Being a perhaps a little closer to this code, I did feel comfortable making
this commit. However, I completely agree with Jani that a regression test
would be a good thing here.
- Dan
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Sep 9 00:30:23 2005