On 04/04/2011 11:37 AM, Philip Martin wrote:
> I've been working on a revert implementation that fixes some correctness
> problems. A side effect of this change is that the notifications
> change. In 1.6 revert only notifies for the root of an added tree, my
> new code notifies for all the added nodes. This causes 3 of the
> regression tests to fail. The 1.6 behaviour seems a bit arbitrary to
> me, and reproducing it with my new code would be difficult.
>
> Do we need any sort of backward compatibility in the testsuite? Is it
> acceptable to simply change the regression tests to expect the new
> notifications? Something like this:
The command-line test suite is designed and intended to be run only with the
same branch of the command-line tools as the test themselves, so this is an
acceptable change. We do try (weakly) to allow our tests to be run against
different *server* versions (with to help in validating that we haven't
broken compatibility between client and server, but that's not what you're
talking about here.
--
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on 2011-04-04 18:31:18 CEST