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
add Budget class, budget and showbudget command #70
add Budget class, budget and showbudget command #70
Conversation
private Model model = new ModelManager(getTypicalAddressBook(), new UserPrefs()); | ||
|
||
@Test | ||
void execute_setNewBudget_success() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
public void --> to remain consistent with other tests and for code quality check
private Model model = new ModelManager(getTypicalAddressBook(), new UserPrefs()); | ||
|
||
@Test | ||
void execute_displayBudget_success() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
public void --> to remain consistent with other tests and for code quality check
*/ | ||
public class Budget { | ||
|
||
public static final String MESSAGE_CONSTRAINTS = "Budget values should be positive number"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we allowing 2dp for budget (like Cost) or budget can use Cost too?
add budget class, with save and load functionalities. added budget and show budget commands budget code quality
5862b67
to
5a84051
Compare
import seedu.address.model.Model; | ||
|
||
/** | ||
* Displays the current SPending |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Budget
|
||
@Test | ||
public void parse_invalidArgs_throwsParseException() { | ||
assertParseFailure(parser, "", String.format(MESSAGE_INVALID_COMMAND_FORMAT, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a negative value argument for test case
add test cases for budget
e5efd3b
to
1d83565
Compare
add budget class, with save and load functionalities. added budget and showbudget commands