Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix running tests by directory #27

Merged
merged 1 commit into from
Sep 19, 2024
Merged

Conversation

jhawthorn
Copy link
Contributor

@jhawthorn jhawthorn commented Sep 19, 2024

Fixes #21

Previously the code here didn't work, we returned early but with the wrong format. The calling code in neotest interpreted this as "unsupported" and so would itself run tests for every file it knows of.

This implements running multiple files very similar to how Rake::TestTask does, passing the files via ARGV and adding a -e script to require each file passed.

With this we run all test files in the directory in a single process rather than booting once for each file.

Previously the code here didn't work, we returned early but with the
wrong format. The calling code in neotest interpreted this as
"unsupported" and so would itself run tests for every file it knows of.

This implements running multiple files very similar to how
Rake::TestTask does.
@zidhuss
Copy link
Owner

zidhuss commented Sep 19, 2024

Thanks for the contributions! 🚀

@zidhuss zidhuss merged commit 5623b9a into zidhuss:main Sep 19, 2024
1 check passed
ippachi pushed a commit to ippachi/neotest-minitest that referenced this pull request Nov 8, 2024
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.

Run directory fires off process for each file?
2 participants