zysimplelife/Addressbook — Issue #6: everything related with unit test
Author: honnix
Created: 2012-03-06 09:13:01+00:00
State: open
Labels: enhancement

* `assertTrue(true)` barely nonsense, you at least have to really verify something

* remove empty methods if you don't need them, i.e. `setUpBeforeClass`, etc.

* all those coding rules toward production code also applies on unit test code
* visibility
* constructor
* ...

* Test class should be named as `XmlAccessTest`

* do not use `if () { assertTrue(false)}`, use `assertNotEquals()`

* add message to assertion