search

NYU-DevOps-Charlie-CSCI-GA-3033-013 / products

Product Team for CSCI-GA.3033-013
http://nyu-lab-bluemix-charlie-products.mybluemix.net/
1 stars 1 forks source link

Meet code coverage requirements #28

Closed ephraimrosenfeld closed 7 years ago

ephraimrosenfeld commented 7 years ago

Dear Team:

Please review the changes that I've made to meet the coverage requirements. Coverage is now at 97%.

I've added new unit tests and refactored the existing ones to be (what I hope) is cleaner and more modular.

One thing to note is that the final unit test, test_csv_data_import() tests the importing of data via a .csv file and will most likely be replaced if/when we perform persistence.

ephraimrosenfeld commented 7 years ago

The scope of this pull request is too big.

I have decided to break up the effort into independent issues with their respective pull requests.