From: Johannes Schindelin Date: Mon, 26 Dec 2005 23:35:21 GMT Subject: [RFC] t5500 considered dangerous Message-ID: 'corrupt a pack and see if verify catches' is bound to fail sometimes for non-obvious reasons: The test case corrupts a pack, and then also the index of that pack, by writing "0" at certain offsets, and then tests if git-verify-pack fails. However, said pack contains a commit, which is variable by virtue of storing two dates. Therefore, the deflated objects are no longer deterministic, and neither the pack. What happens is that in roughly every 256th run, there *is* a 0 at one of these offsets, and the test fails. This patch makes the test deterministic by skipping the commit. Signed-off-by: Johannes Schindelin --- I just hunted down a bug which showed up very rarely, and it did not make the hunting easier. t/t5300-pack-object.sh | 9 ++++----- 1 files changed, 4 insertions(+), 5 deletions(-) diff --git a/t/t5300-pack-object.sh b/t/t5300-pack-object.sh index df6549c..d6b6697 100755 --- a/t/t5300-pack-object.sh +++ b/t/t5300-pack-object.sh @@ -20,12 +20,11 @@ test_expect_success \ done && cat c >d && echo foo >>d && git-update-index --add d && tree=`git-write-tree` && - commit=`git-commit-tree $tree obj-list && { - git-diff-tree --root -p $commit && + git-diff-tree --root -p $tree && while read object do t=`git-cat-file -t $object` && @@ -99,7 +98,7 @@ test_expect_success \ GIT_OBJECT_DIRECTORY=.git2/objects && export GIT_OBJECT_DIRECTORY && cp test-1-${packname_1}.pack test-1-${packname_1}.idx .git2/objects/pack && { - git-diff-tree --root -p $commit && + git-diff-tree --root -p $tree && while read object do t=`git-cat-file -t $object` && @@ -114,7 +113,7 @@ test_expect_success \ export GIT_OBJECT_DIRECTORY && rm -f .git2/objects/pack/test-?.idx && cp test-2-${packname_2}.pack test-2-${packname_2}.idx .git2/objects/pack && { - git-diff-tree --root -p $commit && + git-diff-tree --root -p $tree && while read object do t=`git-cat-file -t $object` &&