Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increase unit test coverage > 90% #122

Open
blomqma opened this issue Feb 1, 2024 · 0 comments
Open

Increase unit test coverage > 90% #122

blomqma opened this issue Feb 1, 2024 · 0 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@blomqma
Copy link
Owner

blomqma commented Feb 1, 2024

With some recent changes, most of the logic is now performed by the CLI commands generate and validate that lowered the test coverage significantly. These commands are unfortunately not unit tested very broadly at the moment. To address this, it would be good to have unit tests for most of the functions in: https://github.com/blomqma/next-rest-framework/tree/main/packages/next-rest-framework/src/cli

There is already a solid unit testing framework and conventions set in the tests directory that could be extended: https://github.com/blomqma/next-rest-framework/tree/main/packages/next-rest-framework/tests

Note that the desired coverage doesn't need to be reached in one go, any incremental improvement to this is appreciated!

How to run tests?

  • Install dependencies:pnpm i
  • Run the tests from the root directory or the ./packages/next-rest-framework directory: pnpm test
@blomqma blomqma added help wanted Extra attention is needed good first issue Good for newcomers labels Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant