Skip to content

feat: add a --list flag to alr test#2078

Merged
AldanTanneo merged 3 commits intoalire-project:masterfrom
AldanTanneo:feat/test-list-flag
Feb 27, 2026
Merged

feat: add a --list flag to alr test#2078
AldanTanneo merged 3 commits intoalire-project:masterfrom
AldanTanneo:feat/test-list-flag

Conversation

@AldanTanneo
Copy link
Member

The --list flag outputs a list (well, a map in structured output mode) of tests matching the input filter. This allows external tools (such as the ada_language_server) to discover ahead of time what tests are available, and potentially to offer users to run them.

@AldanTanneo AldanTanneo requested a review from mosteo February 17, 2026 14:20
@AldanTanneo
Copy link
Member Author

Maybe the issue with the 'corrupted linked list' is that gcc-13 and the system glibc (on ubuntu latest) are mismatched?

@mosteo
Copy link
Member

mosteo commented Feb 17, 2026

Maybe the issue with the 'corrupted linked list' is that gcc-13 and the system glibc (on ubuntu latest) are mismatched?

I hadn't never seen this up to the particular PR where it started. It is also always the same test. So it seems to be something obscure but reproducible. I would expect such a mismatch to be more evident sooner? But I don't really know.

@AldanTanneo
Copy link
Member Author

Hm actually in the current form this will output the list for the first declared alire runner and then bailout... I should add some checks when using the flag 🤔

@AldanTanneo AldanTanneo marked this pull request as draft February 19, 2026 10:19
@AldanTanneo AldanTanneo marked this pull request as ready for review February 23, 2026 13:24
@AldanTanneo AldanTanneo requested review from Fabien-Chouteau and mosteo and removed request for Fabien-Chouteau February 23, 2026 16:49
Copy link
Member

@Fabien-Chouteau Fabien-Chouteau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me as is.

There's one thing I would do to make this more future proof, Get_File_List should return a list/array of records with more info than only a path. Of course I am thinking about my #1935 proposal here, but beyond that it makes sense to compute the display name, source path, bin path, in a single place.

@AldanTanneo
Copy link
Member Author

I will make a followup PR to change the format of both the structured test output and the structured test list output then. (I implemented the structured output as an object instead of just a string for potentially adding fields in the future, so this was on my mind 👍)

@AldanTanneo AldanTanneo merged commit 0b634a3 into alire-project:master Feb 27, 2026
39 checks passed
@AldanTanneo AldanTanneo deleted the feat/test-list-flag branch February 27, 2026 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants