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

Add unit tests for APIs #4

Open
NikhilM98 opened this issue May 28, 2020 · 4 comments
Open

Add unit tests for APIs #4

NikhilM98 opened this issue May 28, 2020 · 4 comments
Labels
good first issue Good for newcomers tests

Comments

@NikhilM98
Copy link
Owner

Add unit tests for the School Portal API.

Requirements:

  • The structure of the tests should be similar to that of Sugarizer-Server.
  • The tests should run on executing npm run test command.
@NikhilM98 NikhilM98 added good first issue Good for newcomers tests labels May 28, 2020
@ricknjacky
Copy link

There are already test files users.js and deployments.js for unit testing APIs, are there some tests that remain? or am I missing something here? Please do update me, if I am. I'd like to work on this if the issue is still open.

@NikhilM98
Copy link
Owner Author

Yes. You can use those tests for reference to add more tests.
The tests are there but the coverage of the tests is not high enough.

@ricknjacky
Copy link

Yes. You can use those tests for reference to add more tests.
The tests are there but the coverage of the tests is not high enough.

Thanks for the clarification.

PS:- I ran npm run test command after cloning the repository, and this the output I get:-

Screenshot (653)

Npm error log.txt

Any heads up on how to obviate this error?

@NikhilM98
Copy link
Owner Author

@ricknjacky you're getting this error because you have not set-up SSP Server Properly. You need to set-up Sugarizer Chart Environment as described here.

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 tests
Projects
None yet
Development

No branches or pull requests

2 participants