2006/6/30, Andrew Zhang <[EMAIL PROTECTED]>:
Hello Mikhail,

Totally agree with you. assertEquals doesn't make any sense here.

I will remove these useless assertEquals in NIO module tests.

Great!




Thanks!


On 6/30/06, Mikhail Loenko <[EMAIL PROTECTED]> wrote:
>
> It is not clear why we have assertEquals in this type of constructions
>
>        try {
>            assertEquals(CAPACITY_NORMAL, this.channel1.read(buf));
>            fail("Should throw ClosedChannelException");
>        } catch (ClosedChannelException e) {
>            // correct
>        }
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>


--
Andrew Zhang
China Software Development Lab, IBM



---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to