diff options
author | Jeff King <peff@peff.net> | 2012-05-25 23:42:53 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2012-05-29 13:22:13 -0700 |
commit | f27e7654de8ff609bf5c48e35f2d27ef1ff3676e (patch) | |
tree | ad335c2409e9d678093bbd207cb73fe8fd37735e /builtin/check-attr.c | |
parent | Merge branch 'va/git-p4-test' (diff) | |
download | tgif-f27e7654de8ff609bf5c48e35f2d27ef1ff3676e.tar.xz |
t5701: modernize style
This test is pretty old and did not follow some of our more
modern best practices. In particular:
1. It chdir'd all over the place, leaving later tests to
deal with the fallout. Do our chdirs in subshells
instead.
2. It did not use test_must_fail.
3. It did not use test_line_count.
4. It checked for the non-existence of a ref by looking in the
.git/refs directory (since we pack refs during clone
these days, this will always be succeed, making the
test useless).
Note that one call to "-e .git/refs/..." remains,
because it is checking for the existence of a symbolic
ref, not a ref itself.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/check-attr.c')
0 files changed, 0 insertions, 0 deletions