estellelim / pe

0 stars 0 forks source link

Import data with missing column #10

Open estellelim opened 2 weeks ago

estellelim commented 2 weeks ago

image.png If there is a missing column, the program will not respond at all. There is no error message shown.

soc-se-bot commented 1 week ago

Team's Response

An invalid example and description of the missing column are needed. Not a bug works as exactly intended and the title is wrong this is not a 'missing column' its an empty cell in an optional parameter.

In fact, as a CSV file and spreadsheet, it would be an empty cell. Considering the tag is optional it imported two students correctly and the student with no values in the cell tag was imported with no tags as expected. It is a perfect example of how the program is to work. How can an empty cell for an optional field that works as intended and described produce an error message?

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It was my mistake for not making the report clearer with a better description and a different entry to show it does not work exactly as intended. Sorry for the confusion. The correct entries in the address book is actually something that was imported from before. My screenshot was intended to show that the command I typed in the command box does not get 'entered' (usually if its error it will turn red, if its successful it will be cleared), meaning non-response of the app (the terminal calls exception but app does not gracefully handle it).

image.png

An empty cell in an optional parameter would still have a comma to indicate it is an empty cell while my screenshot does not have one, so I called it a missing column. In fact, if the csv file does not have the correct amount of headers, it will react the same way as well. image.png

Here is what I think the expected behavior should look like: Having a missing column:

image.png

Having a missing header column: image.png