On Mon, Oct 14, 2024 at 04:24:11PM +0100, chizobajames21@xxxxxxxxx wrote: > test_expect_success 'test GIT_NO_REPLACE_OBJECTS env variable' ' > - GIT_NO_REPLACE_OBJECTS=1 git cat-file commit $HASH2 | grep "author A U Thor" && > - GIT_NO_REPLACE_OBJECTS=1 git show $HASH2 | grep "A U Thor" > + GIT_NO_REPLACE_OBJECTS=1 git cat-file commit $HASH2 >actual && > + test_grep "author A U Thor" actual && Nit: it looks like there is an extra space between the closing '"' quote character and the filename 'actual'. > @@ -284,8 +313,8 @@ test_expect_success 'bisect and replacements' ' > ' > > test_expect_success 'index-pack and replacements' ' > - git --no-replace-objects rev-list --objects HEAD | > - git --no-replace-objects pack-objects test- && > + git --no-replace-objects rev-list --objects HEAD >actual && > + git --no-replace-objects pack-objects test- <actual && > git index-pack test-*.pack > ' Hmm. In other instances, actual seems like an OK name choice, but here I wonder if 'in' would be more appropriate, since we're feeding it as input to another Git command. Other than those couple of comments, these all look pretty reasonable to me. Thanks, Taylor
