search

sandydays / pe

0 stars 0 forks source link

Incorrect handling of auto/N command #4

Open sandydays opened 4 years ago

sandydays commented 4 years ago

Summary: If no vehicles in the district, app claims no vehicle there for auto/N command, when the command is actually incomplete and requires a v/1 tag following it.

Steps to reproduce: new dist/28 auto/N

Expected results: Should prompt the user to type full command with the v/ prefix

Actual results: Actually checks if vehicle is there in the district and outputs there is no vehicle. Checking for the existence of vehicle in the district before checking the command format is correct for auto/N is not right.

Screenshot of issue: image.png

nus-pe-bot commented 4 years ago

Team's Response

Does not make sense to prompt user to choose a vehicle to dispatch when there is no vehicle to dispatch.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]