Re: [PATCH 1/2] test-lib: allow test snippets as here-docs
Eric Sunshine <sunshine@sunshineco.com> writes:
> We lose `chainlint` functionality for test bodies specified in this manner.
Show 12 quoted lines
> Restoring such functionality will require some (possibly)
> not-so-subtle changes. There are at least a couple issues which need
> to be addressed:
>
> (1) chainlint.pl:ScriptParser::parse_cmd() only currently recognizes
> `test_expect_* [prereq] 'title' 'body'` but will now also need to
> recognize `test_expect_success [prereq] 'title' - <body-as-here-doc>`.
>
> (2) Until now, chainlint.pl has never had to concern itself with the
> body of a here-doc; it just throws them away. With this new calling
> convention, here-doc bodies become relevant and must be returned by
> the lexer. This may involve some not-so-minor surgery.
Junio C Hamano <gitster@pobox.com>