fix(neotest): nested modules + position updates when switching buffers

[jameshurst]

This PR fixes calculating positions for nested test modules.

I've removed the dir position as well as that was restricting neotest to a single file rather than the project directory, and was also preventing positions being updated when changing buffers.

I also had to add some naive matching logic to is_test_file as without the overriden dir position neotest will show all .rs files in the project directory, even if they contain no tests.

Before

After

[github-actions]

Review Checklist

Does this PR follow the Contribution Guidelines? Following is a partial checklist:

Proper conventional commit scoping:

• For example, fix(lsp): some lsp-related bugfix

• Pull request title has the appropriate conventional commit prefix.

If applicable:

• Tested
  • Tests have been added.
  • Tested manually (Steps to reproduce in PR description).
• Updated documentation.
• Updated CHANGELOG.md

[mrcjkb]

Thanks, this is great! 🙏

Just some small comments...

I've removed the dir position as well as that was restricting neotest to a single file rather than the project directory, and was also preventing positions being updated when changing buffers.

This fixes #208, which is great.
However, it also means that the neotest test_executor can't be used with "all targets" runnables when using :RustLsp testables. I'll have to come up with a solution for that.
I've implemented a fix for that, so it should work if you rebase 😄

[mrcjkb]

Looks like luacheck is failing in CI:

pre-commit-run> Checking lua/rustaceanvim/neotest/init.lua        2 warnings
pre-commit-run>     lua/rustaceanvim/neotest/init.lua:49:38: unused argument name
pre-commit-run>     lua/rustaceanvim/neotest/init.lua:49:54: unused argument root

[mrcjkb]

LGTM. Thanks again for the great work 🙏 😄