search

gerteck / pe

0 stars 0 forks source link

Wrong error message in `addm` #7

Open gerteck opened 4 months ago

gerteck commented 4 months ago

It is said that appointment cannot be added in the past. However, we are adding the appointment at the exact current time, hence the error message is technically wrong.

image.png

soc-pe-bot commented 4 months ago

Team's Response

The title reads addm but picture shows adda, a bit unclear. So I assumed you are talking about adda and timetable. As shown in your time, it is 16:39 and it is very likely that it has passed 16:39 for a few seconds, to be exact, has passed 16:39:00 for a few seconds. Since the appointment you keyed in has a start time is 16:39:00, you are indeed creating an appointment which is in the past, so the error message is valid.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Apologies on that, I meant adda and timetable. You are right.

However, I still do think that given that the user does not have the the option to specify the seconds, nor is the user shown anything past minutes, the equality should be restricted to minutes. Hence, the error message is technically invalid.

Let us live in the present, and not the past!