On Thu, Jun 11, 2009 at 10:42 AM, Egon Willighagen <egon.willighagen@gmail.com> wrote:
On Wed, Jun 10, 2009 at 4:42 AM, Rajarshi Guha<rajarshi.guha@gmail.com> wrote:
>  @Test public void testStateChanged_ButNotAfterRemoval_RingSet() {

<snip>

>         Assert.assertTrue(listener.changed);
>     }
>
> shouldn't that last assert be assertFalse since we expect that
> listener has *not* changed, so if listener.changed is true, then it's
> a test failure. Similarly for the other analogous test cases

Yes, absolutely right! No idea how that sneaked in...

Can you provide a patch to update those asserts?


--
Rajarshi Guha