-
Notifications
You must be signed in to change notification settings - Fork 5
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
Implement Bidder Seller JUNIT Command and Parser #151
Conversation
Codecov Report
@@ Coverage Diff @@
## master #151 +/- ##
============================================
+ Coverage 54.85% 59.09% +4.24%
- Complexity 783 857 +74
============================================
Files 158 161 +3
Lines 3019 3068 +49
Branches 332 338 +6
============================================
+ Hits 1656 1813 +157
+ Misses 1271 1162 -109
- Partials 92 93 +1
Continue to review full report at Codecov.
|
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.
LGTM
this.bidder = bidder; | ||
} | ||
|
||
@Override | ||
public CommandResult execute(Model model) throws CommandException { | ||
requireNonNull(model); | ||
|
||
if (model.hasPerson(bidder)) { | ||
if (model.hasBidder(bidder)) { | ||
throw new CommandException(MESSAGE_DUPLICATE_PERSON); |
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.
Very small issue but MESSAGE_DUPLICATE_BIDDER
?
@FXML | ||
private void setAutoTab(EntityType entityType) { | ||
TabBar personAndJobTabPane = new TabBar(this.logic); | ||
personAndJobTabPane.setTab(entityType); |
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.
Naming could be better perhaps
No description provided.