search

bobbylaw / pe

0 stars 0 forks source link

Test command #9

Open bobbylaw opened 4 years ago

bobbylaw commented 4 years ago

Not very intuitive with test followed by index of the question. Should only allow test when the flashcard is opened

nus-se-bot commented 4 years ago

Team's Response

Hi, thank you for giving us your feedback on the syntax for testing mcq questions.

To test MCQ questions, I’m asked to open the question based on index using the open INDEX command. After this, I’m asked to test myself by using the test INDEX o/ANS syntax. Why do I need to put an index in a second time? Aren’t I already in the question that I want to try out?

Our intent for include INDEX in test is to make it analagous with testing an open ended question, where you can test the open ended question without using open on it. Having INDEX in the both types of test commands, the user just needs to remember the to include the INDEX for both testing an open ended question and testing an MCQ, instead of having to remember two seperate path flows as suggested, which is:

Instead, by having the INDEX always there, you know for a fact that whether you open or not, you will still need the INDEX to test the question.

The issue is worsened by the fact that there is no indication of which question I am currently viewing in the user interface

We have included the up keystroke to go to the previous command you have typed, so that in case you forgot the INDEX that you typed moments ago, you can use the up keystroke to see the previous open INDEX. However, your suggestion is to include the indication of the question sounds good as well and we can include that in future version.

As this is a suggestion, under the PE rules as shown below, we will reject this issue:

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I feel that this is part of the features that don't fit well with the product due to the syntax

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]