thikhinab / pe Goto Github PK

Command:

Expected:

Error message to show command is invalid as shown in the UG:

Actual:

At this occasion, inclusive can be interpreted to be only inclusive of the latter date.
Better to specify inclusive of both dates.

Better to break it down to two bullet points (one of MacOS/Linux and other for Windows)

Furthermore, it would have been better to name the data.csv different for clarity and confusion that may occur with the folder name

Both could have been formatted to be as user input.
At a glance it could be argued that they refer to two different concepts.

Command:

addt tn/Lecture dt/12-12-2022 1200 t/Weekly Lecture

Repeated add this command:

Expected:
Error message to show duplicate task (no way to identify if the task is there before)

Actual:

Diagram 2

Command:
addc n/test p/543 e/[email protected] u/test2 t/Colleague Lecturer t/colleague lecturer

• Steps to reproduce

Type in addc n/joseph p/+1787877878 e/[email protected] u/ProfJosephNUS t/Colleague Lecturer

User story: adding visiting lecturer from another university

• Expected

Expect the the person to be added to the list.

• Actual

Returns error message with constraints on numbers

• Screenshot

• Steps to reproduce

Type in addc n/joseph p/89993224 e/[email protected] ut/Colleague Lecturer

Reason: Not all people might have github usernames

• Expected

Expect the the person to be added to the list.

• Actual

Returns error message stating the missing prefix

• Screenshot

Test bug

Command;

addc n/josephfdsjfksdjkfksjdkfjskjdfkjksdjfkjskdfkajkdjkjfskdjfksjdkfjksdjkfksdjfksdkjfkjsdkfksdjkfjksdjkfjksdjfksjdkfjksjdkfjksdjkfksdjkfjskdfjksjdkfjksjdkfjksdjfkjskdjfksjdkfjksdjkfjsdkjfkjskdjfksdkfjksdjfkjsdkfjksjdkfjksdjfkjskdjfkljsdkfjkdsjkjfkjkdsfjksjdkf p/5454545 e/[email protected] u/test2 t/Colleague Lecturer

Expected:
The name to be wrapped

Actual:

Have not explained the what the notation "*" means:

Steps:

addc n/joseph p/89993235 e/[email protected] u/profjosephnus t/Colleague Lecturer

Expected:
The person should not be added since the github name (case insensitive) exists.

Actual:

Screenshot:

Before:

After:

Command:
addc n/joseph p/89993234 e/[email protected] u/test t/Colleague Lecturer

Add person with same email with different case

Before:

After:

Expected: The person should not be added.

It would be clearer if it was broken down to two bullet points because the repetition of and/or and OR with numbering makes it confusing