search

JagrajAulakh / COMP-4110-microblog

MIT License
0 stars 0 forks source link

#80 api tests #84

Closed JagrajAulakh closed 1 year ago

JagrajAulakh commented 1 year ago

Closes #80

What I did

Made tests for api/posts and api/users.

Didn't make tests for api/tokens and api/auth because

  1. The files are very short
  2. I don't really know how to test this functionality with flask_httpauth 😃

Coverage for posts and users are 97% and 96% respectively:

image

JagrajAulakh commented 1 year ago

After rebase on master, here are the new coverages:

image

ghanem025 commented 1 year ago

Code looks fine, I ran the branch and all 70 pytests passed with 8 warnings. Should be good to merge if there are no conflicts