Skip to content

Conversation

@icweaver
Copy link
Contributor

@icweaver icweaver commented Oct 25, 2025

Hey folks, thanks for these awesome updates in v2! I noticed that the new find_tests function seemed to be returning repeated path names for tests in subdirectories, so I made this small change if that's alright.

Before:

subdir/subdir_test" => :(include("./test/subdir/subdir/subdir_test.jl"))
"basic"              => :(include("./test/basic.jl"))

After:

subdir/subdir_test" => :(include("./test/subdir/subdir_test.jl"))
"basic"              => :(include("./test/basic.jl"))

@icweaver
Copy link
Contributor Author

Ah, windows. Two shakes

Copy link
Collaborator

@giordano giordano left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Can you please bump the patch number of the version so we can quickly do a new release after this?

@giordano giordano merged commit 3aac7bb into JuliaTesting:main Oct 25, 2025
20 checks passed
@icweaver icweaver deleted the icweaver/test-discovery branch October 25, 2025 17:44
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.

2 participants