Hello Mikhail,

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

I will remove these useless assertEquals in NIO module tests.

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

Reply via email to