884 generalize spreadsheet

[andersoncedu]

Issue: Testing functions from spreadsheet.py

Fix: created a new file in test folder called test_spreadsheet.py, and tested almost every single function in spreadsheet.py

Also created an initial UI using a sidebar button, "Create Spreadsheet", and route for that button, however, this was stated to be outside the scope of our issue so it was not completed, only moved to a new issue.

Test:

1. source setup.sh
2. git checkout 884-Generalize-Spreadsheet
3. ./database/reset_database.sh test
4. tests/monitor.sh
   all the tests should be passing.

fix #884

[AndersonStettner]

Brian and I have some problems with the tests, talk to us when you come in so we can go over everything.

[bledsoef]

In general, I would just reevaluate the test cases in general to make sure you are testing all possible scenarios.

Stale

[Karina-Agliullova]

Consider this comment #1102 (comment)

[hoerstl]

Mostly good! Just a bunch of unnecessary .executes() and some incorrect/misleading comments but good job. I would like to see some more testing on the test_totalVolunteerHours() but otherwise great work!

[azabeli]

It looks good to me, I would suggest adding some doc string for some of the function definitions.

[bledsoef]

I looked through everything and it looks good. If you make this one change I will go ahead and approve it. Nice work!

[github-actions]

View Code Coverage

[AndersonStettner]

LGTM! - Finn

Stale