Completely agree about using proper 'asserts' :-).
Just want to remind about one more useful pair of methods -
assertSame()/assertNotSame() which are useful when comparing
instances. I saw many times in my practice that assertTrue(instance1
== instance2) was used intead. Besides, assertEquals() is also used
for that purpose sometimes which is definitely wrong...
Sometimes assertTrue(false) used instead of fail().
Actually, it is not too many 'assert' methods in
junit.framework.Assert class. May be it is a good idea for those who
never did it before to look at. Having all of them in mind will help
to use the most applicable when necessary.
--
Anton Avtamonov,
Intel Middleware Products Division
On 4/17/06, LvJimmy，Jing <[EMAIL PROTECTED]> wrote:
> 2006/4/17, Mark Hindess <[EMAIL PROTECTED]>:
> >
> > Another type of test that could be improved are those using
> > assertTrue to compare two objects for equality. Most of them should
> > be using assertEquals, assertNull or assertNotNull.
> >
> > Using assertEquals automatically gives more meaningful error messages.
> > It saves people writing asserts like:
> >
> > assertTrue("Size should return 100, returned: " + map.size(),
> > map.size() == 100);
> >
> > when this would do just as well:
> >
> > assertEquals("Incorrect size", 100, map.size());
>
>
> Good idea. :)
>
> I'm going to submit a few (big) patches to fix some of these.
>
>
> I think there's a lot of them, even, few of our testcases is written in this
> style, so, good luck :)
> Anyway, we can discuss about this, and see if we can agree on some
> convention.
>
> Regards,
> > Mark.
> >
> > On 4/15/06, Mark Hindess <[EMAIL PROTECTED]> wrote:
> > > Trying to fix/enable the PatternSyntaxExceptionTest (HARMONY-352) was
> > > quite confusing. If you get the arguments to assertEquals the wrong
> > > way around you get errors like:
> > >
> > > error expected <actual> but got <expected>
> > >
> > > rather than:
> > >
> > > error expected <expected> but got <actual>
> > >
> > > The correct way around is to have the expected value before the actual
> > value.
> > > We should all try make sure we get the arguments to assertEquals in the
> > > correct order. And fix any we spot that are the wrong way around. ;-)
> > >
> > > I've fixed a few of the more obviously incorrect method calls in
> > > HARMONY-353.
> > >
> > > Regards,
> > > Mark.
> > >
> > > --
> > > Mark Hindess <[EMAIL PROTECTED]>
> > > IBM Java Technology Centre, UK.
> > >
> >
> >
> > --
> > Mark Hindess <[EMAIL PROTECTED]>
> > IBM Java Technology Centre, UK.
> >
> > ---------------------------------------------------------------------
> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
>
>
> --
>
> Best Regards!
>
> Jimmy, Jing Lv
> 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]
