Missing Detail in Error Message for `addInsurance`

[RobotWizzard]

The error message displayed when the user enters an invalid insurance id for the addInsurance command is:

This Insurance Plan ID is invalid!

This message does not provide the user with knowledge about "what is a valid Insurance Plan ID?" It is recommended to include the valid Insurance Plan IDs within the error message, so users do not have to look up the user guide every time.

E.g. Please enter a valid Insurance Plan ID! (0 - Basic Insurance Plan, 1 - Travel Insurance Plan)

[soc-pe-bot]

Team's Response

Thanks for raising this concern. However, we feel the message is meant to be general and concise so that in future when more plans are added the message does not look congested. Frequent users of the app should know what the insurance plans they use often are and the user guide is meant to guide properly in that respect anyways.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

• I disagree

Reason for disagreement: I think that having a detailed and helpful error message is more important than having a concise one. Users should not be required to refer to the UG whenever something went wrong when using the app. For example, the error messages for an unknown command in the Windows Powershell is quite long, but provides plenty of details of what could've gone wrong.

Unknown command error message in Windows Powershell:

Long error messages could be split into multiple lines to avoid looking "congested". For example, the git error message for unknown command is:

---

❓ Issue severity

Team chose [severity.Low]
Originally [severity.High]

• I disagree

Reason for disagreement: Since severity.Low applies to issues that occur in rare occurrences and this is not a rare occurrence, I think that this issue should be classified as severity.Medium instead of severity.Low.

---