Thread (1 message) 1 message, 1 author, 2024-07-01

Re: [PATCH 1/2] test-lib: allow test snippets as here-docs

From: Junio C Hamano <hidden>
Date: 2024-07-01 23:43:40

Eric Sunshine [off-list ref] writes:
We lose `chainlint` functionality for test bodies specified in this manner.
Ouch.
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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help