search

adrigzr / neotest-mocha

Neotest runner for Mocha test framework
9 stars 9 forks source link

fix: Add cwd to support js-debug-adapter #19

Closed Jack-Barry closed 1 month ago

Jack-Barry commented 1 month ago

Addresses #18

Adds the cwd attribute to the debug config to allow for using nvim-dap to debug tests

adrigzr commented 1 month ago

Hello @Jack-Barry, thanks for the PR. Lgtm.

@MisanthropicBit, you were reviewing the issue #18. What do you think?

MisanthropicBit commented 1 month ago

Thanks for the PR Jack! @adrigzr LGTM too 🙂 Perhaps a test would be in order but I can also add one at a later time.

Jack-Barry commented 1 month ago

@adrigzr do you know why the unit tests action would be hanging here? I'd like to get this merged, but not sure why that check hasn't resolved yet.

@MisanthropicBit I can try and write up tests around this, could take a while for my smooth brain though

adrigzr commented 1 month ago

@Jack-Barry, no idea, but I merged it!

Jack-Barry commented 1 month ago

Thank you!