git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v9 5/5] t1006: add tests for git cat-file --allow-unkown-type

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Apr 29, 2015, 21:16 UTC
Message-ID
<CAPig+cSXB1fyRGTtuOGsxiBnAhVwS2DydLmVYmq-+47EAa-7-g@mail.gmail.com>
In-Reply-To
<1430312065-23443-1-git-send-email-karthik.188@gmail.com>
On Wed, Apr 29, 2015 at 8:54 AM, Karthik Nayak <karthik.188@gmail.com> wrote:
Show 13 quoted lines
> Signed-off-by: Karthik Nayak <karthik.188@gmail.com>
> ---
> diff --git a/t/t1006-cat-file.sh b/t/t1006-cat-file.sh
> index ab36b1e..8362019 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 --allow-unkown-type" '
> +       echo $type >expect &&
> +    git cat-file -t --allow-unkown-type $sha1 >actual &&

Somehow the indentation got botched here and in the remaining tests you added. This issue is new since v8.

Show 36 quoted lines
> +       test_cmp expect actual
> +    '
> +
> +    test_expect_success "Size of $type is correct using --allow-unkown-type" '
> +       echo $size >expect &&
> +    git cat-file -s --allow-unkown-type $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,21 @@ test_expect_success '%(deltabase) reports packed delta bases' '
>         }
>  '
>
> +bogus_type="bogus"
> +bogus_content="bogus"
> +bogus_size=$(strlen "$bogus_content")
> +bogus_sha1=$(echo_without_newline "$bogus_content" | git hash-object -t $bogus_type --literally -w --stdin)
> +
> +test_expect_success "Type of broken object is correct" '
> +       echo $bogus_type >expect &&
> +    git cat-file -t --allow-unkown-type $bogus_sha1 >actual &&
> +       test_cmp expect actual
> +'
> +
> +test_expect_success "Size of broken object is correct" '
> +       echo $bogus_size >expect &&
> +    git cat-file -s --allow-unkown-type $bogus_sha1 >actual &&
> +       test_cmp expect actual
> +'
> +
>  test_done
> --
> 2.4.0.rc1.250.g565e85b
Previous: Karthik NayakNext: Karthik Nayak
Message 15 of 17 in “cat-file: teach cat-file a '--allow-unkown-type' option”
  1. 0/5 cat-file: teach cat-file a '--allow-unkown-type' optionkarthik nayak, Apr 29, 2015
  2. 1/5 sha1_file: support reading from a loose object of unknown typeKarthik Nayak, Apr 29, 2015
  3. Junio C HamanoApr 29, 2015
  4. karthik nayakApr 29, 2015
  5. Junio C HamanoApr 29, 2015
  6. karthik nayakApr 30, 2015
  7. 2/5 cat-file: make the options mutually exclusiveKarthik Nayak, Apr 29, 2015
  8. 3/5 cat-file: teach cat-file a '--allow-unknown-type' optionKarthik Nayak, Apr 29, 2015
  9. Junio C HamanoApr 29, 2015
  10. karthik nayakApr 29, 2015
  11. Phil HordApr 29, 2015
  12. karthik nayakApr 29, 2015
  13. Junio C HamanoApr 29, 2015
  14. 5/5 t1006: add tests for git cat-file --allow-unkown-typeKarthik Nayak, Apr 29, 2015
  15. Eric SunshineApr 29, 2015
  16. 4/5 cat-file: add documentation for '--allow-unkown-type' option.Karthik Nayak, Apr 29, 2015
  17. Eric SunshineApr 29, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.