This is an archive of the discontinued Mercurial Phabricator instance.

amend: add tests for amending only some files from commit to be amended
ClosedPublic

Authored by singhsrb on Aug 31 2017, 10:01 PM.

Details

Summary

We do not have robust enough tests for scenarios where only some files in a
changeset are amended. This presents an interesting scenario because the
working copy could have modified versions of the remaining files in the
pre-amend changeset. Therefore, I have added some tests to ensure that amend
behaves as expected in these scenarios.

Test Plan

Ensured that the test "test-commit-amend.t" passes.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

singhsrb created this revision.Aug 31 2017, 10:01 PM
singhsrb added inline comments.Aug 31 2017, 10:04 PM
tests/test-commit-amend.t
1227

I will add a newline before this line!

durham added a subscriber: durham.Sep 1 2017, 3:27 PM
durham added inline comments.
tests/test-commit-amend.t
1185

I think you're inside a repo at this point. You may need to cd ../ to get out of it before you init a new one.

singhsrb updated this revision to Diff 1565.Sep 1 2017, 6:18 PM
This revision was automatically updated to reflect the committed changes.