fastify-autoload
fastify-autoload copied to clipboard
fastify
Achieve 100% test coverage
Fixed #369
I failed to cover some branches, so I have removed them. If you can find a way to fully evaluate these branches, please let me know. However, it's possible that they may be obsolete.
I've created an issues folder for issues related tests, which will need to follow this pattern: test/issues/<issue-number>/test.js. Since we're testing the autoloader, we need to create fixtures. This is why I chose to create a folder named after the issue instead of placing <issue-number>.js directly.
If you agree with this approach, I will update this PR to reflect these changes.
Can we achieve this on master instead? I think it would be a moving target on next.
Can we achieve this on master instead? I think it would be a moving target on next.
Should I try to fix the failing TS test here? Or it should be fixed in (or stay commented and fixed later) #371?
The test that's failing here and the one in 371 are different I think
So for this PR, I'd suggest making the tests pass and merging to master, and then focusing on next's failing test at 371
