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
tests: add integration and unit test suites #1
Comments
Hi! I have taken this issue, I imported the smoke.ts file in server.ts and then ran the following commands:-
Doing so I got the above error. Currently, how are you testing things writting in smoke.ts file? And are you getting the same error as well? Also I have figured out that we get almost only 4-5 ServerErrors/Responses 3 out of which are covered in smoke.ts |
Hi! Thanks for taking up the issue :) You don't need to import the test file anywhere, the tests are run using To add the unit tests, simply add another file in the Once you have added tests to the |
Okay thanks for this I'll update on this soon |
Hello! When I run the command Is it issue with version or am I missing something here? `source\server.ts:1:20 test\helpers\fixtures.ts:1:29 source\handlers\index.ts:1:28 672 errors |
These are lint errors, you can fix them by running Also, to run only the tests instead of the linter as well, you can add a command in the "test:unit": "ava --match unit*", |
Thanks a lot for this! from what I understood I have added more testcases in unit.ts file in the test/suites/ folder. If there is any issue with that then please let me know. |
Yes, that's correct :) |
Overview
The integration test suite should simply test each and every API - ensuring all possible successful responses, error responses and server crash cases are covered.
The unit test suite should test every function in the
utilities/
andservices/
folders.Tasks
c8
Each of these tasks can be completed in a separate PR. The last task (regarding coverage) should be completed after the first 3.
If you wish to take up this issue, reply to it below so I can assign you to it. If you are stuck/need any help, you could ask me here itself :)
The text was updated successfully, but these errors were encountered: