Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Min test score #167

Merged
merged 15 commits into from
Apr 13, 2019
Merged

Min test score #167

merged 15 commits into from
Apr 13, 2019

Conversation

dlqs
Copy link

@dlqs dlqs commented Apr 9, 2019

feature: add minimum num questions for test score to be recorded

some more tests added for sort and report
system tests added for test and report session
rfr

public void test() {
Model expectedModel = getModel();

/* ----------------- Performing report operations ---------------------- */

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha this comment needs to change to 'test'

Copy link

@KerrynEer KerrynEer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything else looks good! Thanks for writing the system tests 👍

@dlqs dlqs merged commit a3db6dc into cs2103-ay1819s2-w10-4:master Apr 13, 2019
@dlqs dlqs added this to the v1.4 milestone Apr 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants