ChrisCScott / forecaster

A personal finances forecasting tool for Canadian retirement planning
Other
1 stars 2 forks source link

Refactor tests to break up multi-unit methods #42

Closed ChrisCScott closed 5 years ago

ChrisCScott commented 6 years ago

Currently, most tests methods comprise several unit tests. In most cases, there isn't a good reason for this; much of the shared setup logic could be moved to setUp and each unit test could be its own method.

Some test cases have already been updated in this way (mostly in the canada submodule), but many are still relatively monolithic. This should be cleaned up so that each unit of functionality receives its own method.