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

[Feature Request] Add testing #66

Open
phoenixpereira opened this issue Dec 14, 2024 · 0 comments
Open

[Feature Request] Add testing #66

phoenixpereira opened this issue Dec 14, 2024 · 0 comments
Labels
blocked Requires another issue to be completed first dependencies Pull requests that update a dependency file enhancement New feature or request testing

Comments

@phoenixpereira
Copy link
Member

Overview

Add pytest as a testing framework and create several tests for each API endpoint.

Details

  • Add pytest as a testing framework
  • Create unit tests to check querying db, if a basic response returned is 200 for a valid API call, and error handling for invalid requests
  • Create about 5 integration tests for each API endpoint with both valid and invalid requests (/subjects, /courses, /courses/{cid})

Why would this feature be useful?

To ensure no regressions.

Additional Notes

Some previous work has been done at #15.
#65 should be fixed first to ensure correct response codes.

@phoenixpereira phoenixpereira added enhancement New feature or request dependencies Pull requests that update a dependency file blocked Requires another issue to be completed first testing labels Dec 14, 2024
@rayokamoto rayokamoto moved this to Todo in Courses API Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Requires another issue to be completed first dependencies Pull requests that update a dependency file enhancement New feature or request testing
Projects
Status: Todo
Development

No branches or pull requests

1 participant