On Jan 23, 2008 2:03 PM, Dan Diephouse <[EMAIL PROTECTED]> wrote:
>
>
>
> Some more general coding suggestions:
> * I noticed a lot of the tests have the assertFoo(...) arguments
> reversed. The thing you want the assertion to match should be the first
> argument.


If you want to update to junit 4.4, I've found that switching to using the
assertThat mechanism added in that release prevents these sorts of mistakes,
and has the other benefits listed in the release notes:

http://junit.sourceforge.net/doc/ReleaseNotes4.4.html

(This never seemed to get as much attention as I would have expected when it
was released)

-- 
Stephen Duncan Jr
www.stephenduncanjr.com

Reply via email to