tabnine-nvim icon indicating copy to clipboard operation
tabnine-nvim copied to clipboard

feat: add tests (wip)

Open aarondill opened this issue 2 years ago • 4 comments

These use plenary's busted-like testing framework

As such, they require that plenary is installed to run.

Note: this is not a dependency for normal runtime.

Should I add a CI check that requires all tests are passing?

aarondill avatar Jan 16 '24 06:01 aarondill

@amirbilu is this something you're willing to pursue, or is it not worth putting my effort into?

aarondill avatar Jan 16 '24 06:01 aarondill

Please note that the current goal of the implemented tests is not to ensure that methods work right (as I've discovered and documented several bugs), but to match the current state of the codebase. This way, when a change is made, it can be quickly tested to ensure that it doesn't break anything else.

aarondill avatar Jan 16 '24 09:01 aarondill

@aarondill this is indeed interesting

amirbilu avatar Jan 22 '24 07:01 amirbilu

I haven't had the opportunity to work on this recently due to real life things. I'll continue to work on this when I can, but I would appreciate contributions if you'd like to accelerate the process.

aarondill avatar Mar 24 '24 15:03 aarondill