nus-cs2103-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Low testability for the mail commands #547

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

The mail commands require us to test using the system’s default mail application and that is low testability as according to the constraints for this project, features that require login are hard to test as shown in the images below. As the system's default mail application is not what I use to email others, it requires me to set up before I can test the mail commands. Hence I feel that the mail commands have low testability which violates the project's constraints.

image.png

image.png


[original: nus-cs2103-AY2122S2/pe-interim#768] [original labels: severity.Low type.FeatureFlaw]

s7manth commented 2 years ago

Team's Response

It is mentioned in the User Guide to change the default mail application if needed, since most users do not use the system default mail application. In this case, it is recommended to change it accordingly.

Duplicate status (if any):

--