spinoandraptos / pe

0 stars 0 forks source link

Incorrect error message for list index #4

Open spinoandraptos opened 10 months ago

spinoandraptos commented 10 months ago

image.png

Input index is clearly an integer but error message reflects it is not an integer, this is problematic when we have very large lists of wildlife data entries, with total number equal to or larger than keyed in index. Such large list sizes may be possible if you are keeping entries of individual insects in the reserve, which can easily lead to a very large number of entries. In this context, the definition of wildlife is unclear, and as a new user I may interpret it to include insects.

soc-pe-bot commented 10 months ago

Team's Response

In your report above, you have not demonstrated that the system contains 9999999999 entries. Hence, the output message is correct as the index does indeed not exist in the system.

To clarify things further, this is the check done for delete command:

image.png

The index is still invalid when

  1. the index does not exist
  2. the index specified is greater than EntryList.getArraySize()

    Items for the Tester to Verify

    :question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]