Fix ICollectionContract<T> tests.
authorJonathan Pryor <jonpryor@vt.edu>
Wed, 9 Jun 2010 03:50:12 +0000 (23:50 -0400)
committerJonathan Pryor <jonpryor@vt.edu>
Wed, 9 Jun 2010 03:50:12 +0000 (23:50 -0400)
commitf320f0cb292a28c914d2c68dd7e990063713195c
tree9c8dce915e1c8c1dd36a93014c858e9f9265558a
parentb3c123d1082d77e36a09649627a7c19a1b5bc45b
Fix ICollectionContract<T> tests.

For "sanity", ICollectionContract<T>.RunAllTests should run
CopyTo_Exceptions() before CopyTo(), so that we can assert proper error
behavior before we assert proper regular behavior.

Fix the ICollectionContract<T>.CopyTo_Exceptions() ArgumentOutOfRangeException
check so that we only test one error condition:

Assert.Throws<ArgumentOutOfRangeException>(() => c.CopyTo (new T [1], -1));

has two bugs: the array is two small (`c` contains 3 items, and the created
array of 1 element can't hold 3 elements) and the index is invalid.  The
intention is to check for the invalid index, not the too-small array (which is
checked later in the same method).
src/Cadenza/Test/Cadenza.Collections/ICollectionContract.cs