W dniu 29.08.2016 o 10:04, Johannes Schindelin pisze:
> The sequencer was introduced to make the cherry-pick and revert
> functionality available as library function, with the original idea
> being to extend the sequencer to also implement the rebase -i
> The test to ensure that all of the commands in the script are identical
> to the overall operation does not mesh well with that.
Actually the question is what does the test that got removed in this
commit actually check. Is it high-level sanity check that todo list
for git-cherry-pick contains only 'pick', and for git-revert contains
only 'revert'? Or does it check that at the low level sequencer
fails when instruction sheet includes only identical operations?
Only if it is the latter (we are testing too low level details of
how sequencer code works, tying too tightly test with implementation)
the test should be removed. I see that earlier test check that
sequencer handles correctly invalid instructions in todo.
> Therefore let's just get rid of the test that wants to verify that this
> limitation is still in place, in preparation for the upcoming work to
> teach the sequencer to do rebase -i's work.
Is it "upcoming work" as in one of the patches in this series?
If so, which patch?
> Signed-off-by: Johannes Schindelin <johannes.schinde...@gmx.de>
> t/t3510-cherry-pick-sequence.sh | 11 -----------
> 1 file changed, 11 deletions(-)
> diff --git a/t/t3510-cherry-pick-sequence.sh b/t/t3510-cherry-pick-sequence.sh
> index 7b7a89d..6465edf 100755
> --- a/t/t3510-cherry-pick-sequence.sh
> +++ b/t/t3510-cherry-pick-sequence.sh
> @@ -459,17 +459,6 @@ test_expect_success 'malformed instruction sheet 1' '
> test_expect_code 128 git cherry-pick --continue
> -test_expect_success 'malformed instruction sheet 2' '
Hmmm... the description is somewhat lacking (especially compared to
the rest of test), anyway.
BTW. we should probably rename 'malformed instruction sheet 2'
to 'malformed instruction sheet' if there are no further such
tests after this removal, isn't it?
> - pristine_detach initial &&
> - test_expect_code 1 git cherry-pick base..anotherpick &&
> - echo "resolved" >foo &&
> - git add foo &&
> - git commit &&
> - sed "s/pick/revert/" .git/sequencer/todo >new_sheet &&
> - cp new_sheet .git/sequencer/todo &&
> - test_expect_code 128 git cherry-pick --continue
> test_expect_success 'empty commit set' '
> pristine_detach initial &&
> test_expect_code 128 git cherry-pick base..base