[darcs-users] darcs patch: test that tentative leftover is cleared (and 2 more)

Jason Dagit dagit at codersbase.com
Wed Sep 2 22:38:37 UTC 2009

On Wed, Sep 2, 2009 at 4:02 PM, Kamil Dworakowski<kamil at dworakowski.name> wrote:
> This time there are three patches to comply with the request to submit
> tests separately.
> I have created a new test that ensures tentative state left after an
> uncommitted transaction does not affect subsequent ones. I have also
> extended the test for the issue1406, more on that in a moment.
> When I was submitting the original patch I didn't know what
> checkpoints were. It turns out that my original patch left the
> remove_from_tentative_inventory function still manipulating some real
> state of the repository, namely the checkpoints inventory. I have now
> changed it so that, analogous to the normal inventory, checkpoints
> inventory is being manipulated tentatively. I have extended the test
> for issue1406 to cover that. Now it checks that real checkpoints
> inventory is not modified by a failed amend. You can't create checkpoints
> with the current version of darcs, so I have created a repo with an
> older version and put it in tests/repos dir.
> Wed Sep  2 09:52:12 BST 2009  Kamil Dworakowski <kamil at dworakowski.name>
>  * test that tentative leftover is cleared
> Wed Sep  2 23:03:21 BST 2009  Kamil Dworakowski <kamil at dworakowski.name>
>  * activate and extend issue1406 test
>  extend with checkpoint modification checking
> Wed Sep  2 23:10:53 BST 2009  Kamil Dworakowski <kamil at dworakowski.name>
>  * resolve issue1406: amend-record unrecords on a test failure
>  A failing test on amend no longer unrecords the original patch. The
>  failure manifested itself only on old-fashioned repositories. Change
>  DarcsRepo.remove_from_tentative_inventory not to modify the pristine
>  nor the inventory.

I'm still reviewing it, but I noticed what I think is a typo.
"hasChackpoints"  This should be "hasCheckpoints", right?


More information about the darcs-users mailing list