Re: [PATCH]remove unwanted tweaks for repos_rev for commit verification

Kamesh Jayachandran wrote:
> Lieven Govaerts wrote:
>> Committed this patch in r22191, I did find another location in the
>> revert_replacement_with_props test where repos_rev was tweaked, so I
>> removed that too.
> I have seen that, but it was actually dead code(no usage) along with
> expected_output with no usage down the line thought of removing it on
> a different changeset.
You're right, expected_status and expected_output aren't used at all.
Now I don't care so much about the expected_output, but why not add an
extra status check after the revert is happening?

After all, this is revert_tests.py, so we should at least focus on
checking the behavior of revert in all possible ways, right?

Lieven.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Nov 1 15:33:10 2006