search

mechastriker3 / pe

0 stars 0 forks source link

Corrupting the data by adding fake fields does not result in data being cleared #3

Open mechastriker3 opened 3 years ago

mechastriker3 commented 3 years ago

I added a fake field into the first person in bookingsystem.json (see below)

image.pngfakeField.png

But upon startup, the data on the screen was not cleared( as expected in the test case in the DG).

image.pngScreen Shot 2021-04-16 at 2.27.14 PM.png

nus-pe-bot commented 3 years ago

Team's Response

We are not sure what the problem is because that is not what the developer guide suggested in order to corrupt the data. So we are not sure why you say that it is "as expected in the test case in the DG".

Screenshot 2021-04-17 at 12.50.19.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Putting many ways to cause an invalid data implies that the users are free to corrupt the data as they please and the bookingsystem.json should be replaced with an empty one when the app starts up, instead the invalid extra field is just saved in the json.

:question: Issue severity

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

Reason for disagreement: The command did not work as intended as bookingsystem.json should return with a empty list when the data is invalid.