This PR is provided by @GatorEducator/team-lancaster and @GatorEducator/team-spencer to increase the code coverage from 64.15% to 99.17% with the following features:
Increasing code coverage for display.py from 82% to 100%.
Provide test_display_welcome_message and restructure display_welcome_message to make it can be easier tested. @Lancasterwu
Increasing code coverage for grouprandom.py from 60% to 100%.
Provide test functions to test group_random_num_group. @wattob
Increasing code coverage for grouprrobin.py from 56% to 100%.
Provide test functions to test group_rrobin_num_group @ALEXANDERB82 & @everitt-andrew
increasing code coverage for readstudentfile.py from 28% to 100%.
Provide test_read_file_populates_csvdata_0 to test read_student_file and add students.csv as an input to test. @thomad74
Increasing code coverage for parseargument.py from 77% to 98%.
Provide a new file test_num_group.py with test functions to test check_valid_num_group. @reibelj
Increasing code coverage for gatorgrouper.py from 40% to 100%.
Provide pragma to not report test coverage for if __name__ == "__main__" @baldeosinghm & @yeej2
NOTE:
There are two quit() functions was not able to be tested in parseargument.py, which caused that 2% of coverage missing.
Random test input generator Hypothesis still need to be implemented. @mmarconi and @libailong are currently working on that.
Description of Pull Request
This PR is provided by @GatorEducator/team-lancaster and @GatorEducator/team-spencer to increase the code coverage from 64.15% to 99.17% with the following features:
display.py
from 82% to 100%.test_display_welcome_message
and restructuredisplay_welcome_message
to make it can be easier tested. @Lancasterwugrouprandom.py
from 60% to 100%.group_random_num_group
. @wattobgrouprrobin.py
from 56% to 100%.group_rrobin_num_group
@ALEXANDERB82 & @everitt-andrewreadstudentfile.py
from 28% to 100%.test_read_file_populates_csvdata_0
to testread_student_file
and addstudents.csv
as an input to test. @thomad74parseargument.py
from 77% to 98%.test_num_group.py
with test functions to testcheck_valid_num_group
. @reibeljgatorgrouper.py
from 40% to 100%.pragma
to not report test coverage forif __name__ == "__main__"
@baldeosinghm & @yeej2NOTE:
quit()
functions was not able to be tested inparseargument.py
, which caused that 2% of coverage missing.Hypothesis
still need to be implemented. @mmarconi and @libailong are currently working on that.This PR Fixes #137
Type of Change
Please describe the pull request as one of the following:
Tags
@Lancasterwu @huangs1 @gkapfham @aubreypc @Michionlion