search

GatorIncubator / gatorgrouper

:busts_in_silhouette: Automated Group Formation Tool Enabling Effective Team Work
GNU General Public License v3.0
20 stars 20 forks source link

Refactored parse_arguments.py, removed bugs and added test cases #232

Closed baldeosinghm closed 5 years ago

baldeosinghm commented 5 years ago

Refactored parse_arguments.py, removed bugs and added test cases

This pull request contains refactored code in parse_arguments.py. The quit() commands were removed to make testing of the program easier and also make the code more efficient. In its place, the string "Invalid entry" was coded. Furthermore, I made test cases to increase code coverage with test_arguments.py (@enpuyou also helped out). Also, there was a bug that was fixed in parse_arguments) where student groups of 1 and 0 were being considered, when they shouldn't.

Fixes #202, fixes #194

Type of Change

Please describe the pull request as one of the following:

Tags

@enpuyou - helped in creating a test case for parse_arguments.py

Lancasterwu commented 5 years ago

This branch should be reverted to the version before being merged from mvp-terminal-only. Hence I will close it for now.