search

chanyijuan / pe

0 stars 0 forks source link

Same error message for different errors for listProc #9

Open chanyijuan opened 2 years ago

chanyijuan commented 2 years ago

The same error message 'No Procedures to load!' is shown for listProc for client index without procedures, and a client index out of bounds. Think it would be beneficial to the user to know the specific error in this case, so they rectify their command.

When listProc 4 was used (client with no procedures to list): Screenshot 2022-04-16 at 2.52.44 PM.png

When listProc 5 was used (non-existent client): Screenshot 2022-04-16 at 2.52.44 PM.png

nus-se-bot commented 2 years ago

Team's Response

When we attempted to call listProc 5 when the 5th Client did not exist, it printed out the correct error message as shown below. Based on the scrollbar on the Client box, it seems like there is a 5th Client in this picture, so I am assuming that this is a mistake.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.CannotReproduce]

Reason for disagreement: [replace this with your explanation]