Monilnarang opened a new pull request, #958:
URL: https://github.com/apache/incubator-baremaps/pull/958

   **Summary**: 
   - Parameterized tests : `type` and changed name to `testType`
   
   **Elaboration**: 
   - The same method call (`objects.get(x).type()`) is repeated multiple times 
with different inputs, making it redundant and the test harder to maintain and 
extend. 
   - When a test fails, JUnit only shows which assertion failed, but not which 
specific input caused the failure.
   - Adding new test cases requires copying and pasting another line of 
assertion statement instead of simply adding new data.
   
   To accomplish this, I retrofitted this tests into a parameterized unit test. 
This reduces duplication, allows easy extension by simply adding new value 
sets, and makes debugging easier as it clearly indicates which test failed 
instead of requiring a search through individual assertions.
   Also, when run each value set is shown its separate pass/fail status. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to