search

jiaweiteo / pe

0 stars 0 forks source link

Invalid marco allowed #9

Open jiaweiteo opened 3 years ago

jiaweiteo commented 3 years ago

Command: assignmacro a/aasda cs/update asdasd

Expected: Error message since command is invalid.

Actual: Updated macro.

image.png

nus-se-bot commented 3 years ago

Team's Response

It has already been written in the user guide and explained in the developer guide, that a vaild assignmacro command does not guarantee that the saved command is valid, as we give flexibility to the user to save partial commands into a macro, so that the macro used with different sets of additional arguments. Refer to the 2nd example of assignmacro command in the user guide for a concrete example.

Nonetheless, an error message for the respective commands will be shown if the macro is used in such a way that the underlying command is incorrect.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]