joshuan98 / pe

0 stars 0 forks source link

Platform independence #34

Open joshuan98 opened 1 year ago

joshuan98 commented 1 year ago

image.png

This does not satisfy the constraint that

image.png

soc-se-bot commented 1 year ago

Team's Response

Functionality provided is not OS-dependent. Windows users who wish to use timetable fancy mode can either install a terminal which supports it or configure their terminal to support it.

Furthermore, the differences between timetable /fancy and timetable /simple are purely cosmetic. No functionality is lost when the terminal does not support it.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I strongly disagree with the line of reasoning provided by the team. Clearly, the timetable /fancy does not work on Windows as per my screenshot in another issue (#25 in CATcher). Hence, the functionality provided is obviously OS-dependent.

Asking the user to "install a terminal which supports it or configure their terminal to support it" is a slippery slope argument. Firstly, the group provides no support on which Windows terminal supports it or provide an explanation on how to configure command prompt to support it. Secondly, by the team's logic, it means that the onus is on the user to make sure that the program works which simply should not be the case.

Moreover, while it is true that the differences between timetable /fancy and timetable /simple are purely cosmetic, it remains a fact that a Windows user who types /fancy will not be able to glean any discernable information from the output. With no error message provided, it is clear that the program does not fail gracefully.

In fact, the group could have simply disabled this feature on Windows platform entirely. As such, this error could be labelled as a feature flaw instead.

Considering the fact that all Windows users would not be able to use this function but there are other alternatives, I think that the issue severity could be reduced to medium but certainly not lower than that.