Review test-cases for dos-unexpected-revert-with-vector

[arturoBeccar]

Task

Determine whether it is necessary to split the test case for dos-unexpected-revert-with-vector, one assessing the authorization and the other assessing the usage of vector.

[arturoBeccar]

• Split the test case into two to showcase different ways of solving the same issue. The vulnerable test case will be the same in both pairs. In the first pair, remediate the issue requiring authorization by the owner to call add_candidates().
  In the second pair, remediate the issue replacing the vector with a mapping like structure using DataKey Enum.

• Update all relevant documentation for the vulnerability and detector.