The pattern apparently is [`assertEquals(expected, actual)`](http://junit.sourceforge.net/javadoc/org/junit/Assert.html#assertEquals\(java.lang.Object, java.lang.Object\)), i.e. the other way around. I only mention that because the test failure message is confusing if you keep it this way - it would e.g. say "expected 3 but got 1" when you really mean "expected 1 but got 3".