search

cs102-project / LabConnect

LabConnect facilitates communication between students, TA's, tutors, and instructors.
http://labconnect.me
GNU Affero General Public License v3.0
6 stars 0 forks source link

Tester classes and packaging #13

Closed berkan-sahin closed 3 years ago

berkan-sahin commented 3 years ago

Despite aggregating some common classes (namely TestResult and TestState), the UnitTest and StyleChecker (shouldn't this be StyleTest for consistency?) classes are currently packaged separately, while the interface Tester is packaged in models. Since these classes fulfill very similar functions (hence the interface) and aggregate a few common objects, unifying the entire testing functionality under a testing package and modifying TestResult to accommodate style checks seems logical. Since this is a pretty substantial design change, I decided to open an issue before changing the arrangement.

berkan-sahin commented 3 years ago

I am closing this issue, since the design of StyleChecker is being changed to be in line with UnitTest