(+cc: Adam Simpkins) Hi, Thanks for doing this tedious work. Elijah Newren wrote: > Also removed several repeated tag and branch deletions Nit: present tense > , since the tags and > branches had already been deleted. I suspect the repeated deletions were to allow skipping individual tests. Maybe something like the following on top of your patch would achieve that. The rest looks good to me, for what it's worth. diff --git a/t/t6016-rev-list-graph-simplify-history.sh b/t/t6016-rev-list-graph-simplify-history.sh index e640c41..5036940 100755 --- a/t/t6016-rev-list-graph-simplify-history.sh +++ b/t/t6016-rev-list-graph-simplify-history.sh @@ -116,12 +116,15 @@ test_expect_success '--graph --simplify-by-decoration' ' test_cmp expected actual ' -# Get rid of all decorations on branch B, and graph with it simplified away -test_expect_success '--graph --simplify-by-decoration prune branch B' ' - rm -f expected && +test_expect_success 'setup: get rid of decorations on branch B' ' git tag -d B2 && git tag -d B1 && - git branch -d B && + git branch -d B +' + +# Graph with branch B simplified away. +test_expect_success '--graph --simplify-by-decoration prune branch B' ' + rm -f expected && echo "* $A7" >> expected && echo "* $A6" >> expected && echo "|\\ " >> expected && -- -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html