Adds some tests with pytest && fixes a json.loads bug - Githubissues

calpoly-csai / api

Official API for the NIMBUS Voice Assistant accessible via HTTP REST protocol.

https://nimbus.api.calpolycsai.com/

GNU General Public License v3.0

9 stars 4 forks source link

Adds some tests with pytest && fixes a json.loads bug #80

Closed mfekadu closed 4 years ago

mfekadu commented 4 years ago

What's New?

tests

Type of change (pick-one)

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

I wrote tests into the /tests/ directory

Checklist (check-all-before-merge)

formatting help: - [x] means "checked' and - [ ] means "unchecked"

[ ] I documented my code according to the Google Python Style Guide
[ ] I ran ./build_docs.sh and the docs look fine
[ ] I ran ./type_check.sh and got no errors
[ ] I ran ./format.sh because it automatically cleans my code for me 😄
[ ] I ran ./lint.sh to check for what "format" missed
[ ] I added my tests to the /tests directory
[ ] I ran ./run_tests.sh and all the tests pass

cameron-toy commented 4 years ago

Nice tests and good catch on the json.loads bug.