search

stleary / JSON-Java-unit-test

Junit test harness to validate the JSON-Java GitHub project code.
Apache License 2.0
28 stars 45 forks source link

improve the confused assert message #96

Closed Alanscut closed 4 years ago

Alanscut commented 4 years ago

The test cases for bean parse are good, but some assert messages make people coufused, some fields are found but described as not working, while another fields are not found but described as found.

stleary commented 4 years ago

@Alanscut thanks for helping improve the unit tests. I think some of your assert messages may still be off, have commented same. Let me know if you disagree. These asserts are very confusing, adding comments and clear assert messages helps everyone to understand them better.

Alanscut commented 4 years ago

@stleary Thanks your suggestion! It make the assert messages more clear :+1: