Skip to content

Commit ab51783

Browse files
peffgitster
authored andcommitted
t5305: simplify packname handling
We generate a series of packfiles test-1-$pack, test-2-$pack, with different properties and then examine them. However we always store the packname generated by pack-objects in the variable packname_1. This probably was meant to be packname_2 in the second test, but it turns out that it doesn't matter: once we are done with the first pack, we can just keep using the same $packname variable. So let's drop the confusing "_1" parameter. At the same time, let's give test-1 and test-2 more descriptive names, which can help keep them straight (note that we _could_ likewise overwrite the packfiles in each test, but by using separate filenames, we are sure that test 2 does not accidentally use the packfile from test 1). Signed-off-by: Jeff King <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 948a7fd commit ab51783

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

t/t5305-include-tag.sh

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,15 +26,15 @@ test_expect_success setup '
2626
'
2727

2828
test_expect_success 'pack without --include-tag' '
29-
packname_1=$(git pack-objects \
29+
packname=$(git pack-objects \
3030
--window=0 \
31-
test-1 <obj-list)
31+
test-no-include <obj-list)
3232
'
3333

3434
test_expect_success 'unpack objects' '
3535
rm -rf clone.git &&
3636
git init clone.git &&
37-
git -C clone.git unpack-objects <test-1-${packname_1}.pack
37+
git -C clone.git unpack-objects <test-no-include-${packname}.pack
3838
'
3939

4040
test_expect_success 'check unpacked result (have commit, no tag)' '
@@ -45,16 +45,16 @@ test_expect_success 'check unpacked result (have commit, no tag)' '
4545
'
4646

4747
test_expect_success 'pack with --include-tag' '
48-
packname_1=$(git pack-objects \
48+
packname=$(git pack-objects \
4949
--window=0 \
5050
--include-tag \
51-
test-2 <obj-list)
51+
test-include <obj-list)
5252
'
5353

5454
test_expect_success 'unpack objects' '
5555
rm -rf clone.git &&
5656
git init clone.git &&
57-
git -C clone.git unpack-objects <test-2-${packname_1}.pack
57+
git -C clone.git unpack-objects <test-include-${packname}.pack
5858
'
5959

6060
test_expect_success 'check unpacked result (have commit, have tag)' '

0 commit comments

Comments
 (0)