David Turner <dturner@xxxxxxxxxxxx> writes: > +test_expect_success 'auto gc with too many loose objects does not attempt to create bitmaps' ' > + test_config gc.auto 3 && > + test_config gc.autodetach false && > + test_config pack.writebitmaps true && > + # We need to create two object whose sha1s start with 17 > + # since this is what git gc counts. As it happens, these > + # two blobs will do so. > + test_commit 263 && > + test_commit 410 && > + # Our first gc will create a pack; our second will create a second pack > + git gc --auto && > + ls .git/objects/pack |grep -v bitmap >existing_packs && Missing SP (compare with the second invocation of the same). > + test_commit 523 && > + test_commit 790 && > + > + git gc --auto 2>err && > + test_i18ngrep ! "^warning:" err && > + ls .git/objects/pack/ | grep -v bitmap >post_packs && > + ! test_cmp existing_packs post_packs This does not look good for two reasons. test_cmp tries to highlight test breakage by being verbose when two files are different while keeping quiet when they are the same. Running it with "!" does not change its expectation. This undesirable effect should be visible when this test is run with "-v" option. Another is that this only tests if the set of packs before and after are different. I think you are expecting that the second invocation will create a new one while leaving the old one intact but this test will not catch a breakage if the second repack instead created just one pack replacing the old one. > +' > + > + > test_done Thanks.