From: Eli Schwartz <eschwartz93@gmail.com>
To: Taylor Blau <me@ttaylorr.com>, Patrick Steinhardt <ps@pks.im>
Cc: Ramsay Jones <ramsay@ramsayjones.plus.com>,
GIT Mailing-list <git@vger.kernel.org>
Subject: Re: [PATCH] t/meson.build: fix up tests for 'seen' branch
Date: Mon, 21 Oct 2024 21:38:50 -0400 [thread overview]
Message-ID: <9820a7a6-c9c0-44eb-9c04-9445a5061c14@gmail.com> (raw)
In-Reply-To: <ZxbEJbigarbklNJq@nand.local>
[-- Attachment #1.1: Type: text/plain, Size: 1494 bytes --]
On 10/21/24 5:14 PM, Taylor Blau wrote:
> On Mon, Oct 21, 2024 at 08:04:16AM +0200, Patrick Steinhardt wrote:
>> For our integration-style tests I'd be okay with not listing the files
>> individually, such that we instead use e.g. prove(1) to run all tests
>> via a single test target. It would be a regression in functionality as
>> we now cannot easily run e.g. "meson test t0000*", but at least we would
>> not have to maintain the list of test scripts anymore.
>
> I quite like the ability to run tNNNN-*.sh individually when spelled
> like that (i.e. including the test number, but not the full script
> name).
>
> I find that for the areas that I have worked on most over the years that
> I have the relevant test numbers committed by heart, but often don't
> know the full script name. So it is nice to be able to refer to them by
> number only.
>
> Am I reading it write that this suggests that Meson would lose that
> ability?
If "meson test" simply ran the prove(1) utility then it would still be
possible to do:
```
meson test --test-args "--args --to --prove"
```
This would not actually do what you want, since prove(1) doesn't
implement "t0000*" style globbing, does it?
So your reading is correct -- meson's own globbing support relies on
meson knowing the list of all test names, which means you do have to
maintain that list somehow (and have that list update when you do "git
pull" and new tests materialize).
--
Eli Schwartz
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 963 bytes --]
prev parent reply other threads:[~2024-10-22 1:38 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-20 16:00 [PATCH] t/meson.build: fix up tests for 'seen' branch Ramsay Jones
2024-10-21 6:04 ` Patrick Steinhardt
2024-10-21 21:14 ` Taylor Blau
2024-10-22 1:38 ` Eli Schwartz [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9820a7a6-c9c0-44eb-9c04-9445a5061c14@gmail.com \
--to=eschwartz93@gmail.com \
--cc=git@vger.kernel.org \
--cc=me@ttaylorr.com \
--cc=ps@pks.im \
--cc=ramsay@ramsayjones.plus.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).