On Thu, Jul 30, 2015 at 04:17:35PM -0400, Eric Sunshine wrote: > > + test_expect_success "Override hiding of $configsection.hiderefs" ' > > + test_when_finished "test_unconfig $configsection.hiderefs" && > > + git config --add $configsection.hiderefs refs/tags && > > + git config --add $configsection.hiderefs "!refs/tags/magic" && > > + git config --add $configsection.hiderefs refs/tags/magic/one && > > + git ls-remote . >actual && > > + verbose grep refs/tags/magic/two actual > > For completeness, do you also want to add > > !grep refs/tags/magic/one actual > > to confirm that the third hideRefs did indeed override the second one? Yeah, I think that is a good change. We could also test_cmp the whole output, but I didn't want to be dependent on other junk from previous tests. -Peff -- 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