HuangRongzhen/AddressBook — Issue #5: everything related with unit test
Author: honnix
Created: 2012-03-08 08:43:39+00:00
State: open
Labels: bug, enhancement

* do not comment out unit test, use `@Ignored` annotation

* why there are many `public final` method?

* do not mock logger, use slf4j-nop is you don't want to log at all during unit test

* `assertXxx` should have message passed as the first argument

* do not declare very long string, at least you should use `+` to concat

* don't need to catch exception and `fail()`, just let it throw and junit will treat it as failed case

* remove empty method if not used

* *NEVER* compare XML by using pure string