[Mono-list] CollectionBaseTest patch
cesar@kentia.matem.unam.mx
cesar@kentia.matem.unam.mx
Sun, 26 May 2002 23:04:00 -0500
Hello
> Thanks for the patch. However, I'd like to point out a few things...
thanks for the comments nick.
> you a nice message saying "Expected <x> but got <a>". If you reverse them,
> then it can be confusing.
still learning c#.
> 2) There's not much point using AssertEquals() over Assert() for Boolean
> values, although it's not exactly "wrong". If you do use AssertEquals(),
> please keep in mind #1 above.
ok, in what cases it's prefered the use of AssertEquals above Assert?.
I'looking through the Test files and I found that in most of them, people have been using
Assert (), and as the HOWTO mention, it's recommended the use of AssertEquals. That's why I decided to change
them. Do you think it's worth to check the test files and change them?
> Those are general comments. I also have a specific one:
> | - Assert(4 == myCollection.Count);
> | + AssertEquals ("A01", 4, mycollection);
> Did you forget the Count property here?
yes, sorry; I forgot the Count property.
I was kinf of nervous, it's my first little patch.
--
césar