On Sat, Apr 4, 2015 at 1:44 AM, Karthik Nayak <karthik.188@xxxxxxxxx> wrote: > Signed-off-by: Karthik Nayak <karthik.188@xxxxxxxxx> > --- > diff --git a/t/t1006-cat-file.sh b/t/t1006-cat-file.sh > index ab36b1e..5b74044 100755 > --- a/t/t1006-cat-file.sh > +++ b/t/t1006-cat-file.sh > @@ -47,6 +47,18 @@ $content" > test_cmp expect actual > ' > > + test_expect_success "Type of $type is correct using --literally" ' > + echo $type >expect && > + git cat-file -t --literally $sha1 >actual && > + test_cmp expect actual > + ' > + > + test_expect_success "Size of $type is correct using --literally" ' > + echo $size >expect && > + git cat-file -s --literally $sha1 >actual && > + test_cmp expect actual > + ' > + > test -z "$content" || > test_expect_success "Content of $type is correct" ' > maybe_remove_timestamp "$content" $no_ts >expect && > @@ -296,4 +308,19 @@ test_expect_success '%(deltabase) reports packed delta bases' ' > } > ' > > +bogus_type="bogus" > +bogus_sha1=$(git hash-object -t $bogus_type --literally -w --stdin </dev/null) > + > +test_expect_success "Type of broken object is correct" ' > + echo $bogus_type >expect && > + git cat-file -t --literally $bogus_sha1 >actual && > + test_cmp expect actual > +' > + > +test_expect_success "Size of broken object is correct" ' > + echo "0" >expect && Zero is such a common fallback value when things go wrong that it may not be the best choice for this test. Consequently, it might be better to create the bogus object with non-zero length. Take a look at how 'hello_length' and 'hello_sha1' are computed elsewhere in this script for inspiration. > + git cat-file -s --literally $bogus_sha1 >actual && > + test_cmp expect actual > +' > + > test_done > -- > 2.4.0.rc1.249.g9f2ee54 -- 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