AY2425S1-CS2103T-T15-1 / tp

Other
0 stars 5 forks source link

Add Prompt for Confirmation from User #177

Closed RobotWizzard closed 1 week ago

RobotWizzard commented 1 week ago

Adds a new field in CommandResult -- continuationFunction, that is applied when the user confirms the prompt.

MainWindow will check if the returned CommandResult requires a confirmation from user and prompts the user if it is.

Edited commands:

Fixes #103

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 57.89474% with 16 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/main/java/seedu/address/ui/MainWindow.java 0.00% 16 Missing :warning:
Files with missing lines Coverage Δ Complexity Δ
...ava/seedu/address/logic/commands/ClearCommand.java 100.00% <100.00%> (ø) 4.00 <1.00> (+2.00)
...va/seedu/address/logic/commands/CommandResult.java 100.00% <100.00%> (ø) 21.00 <8.00> (+6.00)
...du/address/logic/commands/DeleteClientCommand.java 87.50% <100.00%> (+1.13%) 9.00 <1.00> (+2.00)
...du/address/logic/commands/DeleteRentalCommand.java 100.00% <100.00%> (ø) 13.00 <2.00> (+2.00)
src/main/java/seedu/address/ui/MainWindow.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)

... and 4 files with indirect coverage changes

RobotWizzard commented 1 week ago

codecov is expected to fail for MainWindow since UI components are not tested.