olimorris / neotest-rspec

🧪 Neotest adapter for RSpec. Works in Docker containers too
MIT License
88 stars 25 forks source link

Filter directories #35

Closed benlubas closed 1 year ago

benlubas commented 1 year ago

Implements the filter_dir method in the neotest adapter interface

This reduces the initial load time of the neotest plugin in large repos by a substantial margin (20s to 9s in my work rails repo)

Currently using some simple rules. If the path has "spec" in it, it's good. If it doesn't, and it's got a "/" in it, it's bad.

These rules allow "engine/spec" to get picked up as well as "spec", but would miss "engine/something/spec"

afaik, that last case is not common or recommended? I'm fairly new to this whole ruby thing. So let me know if that's a problem.

olimorris commented 1 year ago

Nice! Thanks for this and had been on my list for some time. 20s to 9s is an incredible improvement!