search

avrilgk / pe

0 stars 0 forks source link

Unable to go back to main menu after inputting gym session #2

Open avrilgk opened 2 months ago

avrilgk commented 2 months ago

image.png

Please see above for the error produced.

As per the UG, the user is supposed to be able to exit to the main menu after inputting a new gym session

image.png

However, an exception is thrown when I enter the back command after the enter the 2nd gym session.

Reproducing the error: workout /e:run /d:5.15 /t:25:03 /date:25-03-2023

Successfully added a new run session Type Time Distance Pace Date run 25:03 5.15 4:52/km 2023-03-25

workout /e:gym /n:2 /date:25-03-2023

Please enter the details of station 1. (Format: e.g. Bench Press /s:2 /r:4 /w:10,20) Enter 'back' to go back to the main menu!

bench press /s:2 /r:4 /w:10,20

Please enter the details of station 2. (Format: e.g. Bench Press /s:2 /r:4 /w:10,20) Enter 'back' to go back to the main menu!

squat /r:2 /s:2 /w:10.5,20.5

Successfully added a new gym session Station 1 bench press: 2 sets

nus-se-script commented 2 months ago

Team's Response

Works as intended. User has specified the gym session has 2 stations and the application has received 2 gym station inputs. Thus, it automatically exits from the gym station input portion and returns to the main menu. Thus, the back command does not work, which is intended.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: From the UG, under the gym 'Adding Gym Stations' section, it states that 'If you want to exit the gym station input prompt and go back to the main menu to use other commands, use back to do so'.

image.png

This can be understood that the user has to input the back command to return to the main menu. As the UG did not state that the user is automatically redirected back to the main menu after inputting the maximum number of gym stations that was indicated, the user will not know that he does not need to enter the back command if he wishes to go back to the main menu.

image.png

Moreover, there is no indication on the command line that states that the user is back in the main menu.

Hence, the issue cannot be rejected, but rather, of low severity. Because though it is unlikely to affect normal operations, it will still cause minor inconvenience. Especially since it should be expected that the product works exactly as stated in the UG.

image.png