initial flask_api tests - 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

initial flask_api tests #138

Closed Jason-Ku closed 4 years ago

Jason-Ku commented 4 years ago

What's New?

Add some starter tests for the flask app
Rebased
Added is_view variable to Entity classes
Check if an entity is a view before update/insert
Added test for view check

How Has This Been Tested?

sh run_tests.sh

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
[x] I added my tests to the /tests directory
[x] I ran ./run_tests.sh and all the tests pass

Jason-Ku commented 4 years ago

@cameron-toy does the init db/nimbus part differently, which I have no problem with, I may update this PR to follow his model

chidiewenike commented 4 years ago

@cameron-toy Does this conflict with your most recent PR? I do believe that this could be the solution to the memory leak.