[Test]
public void MoveNextAfterUpdate()
{
list.Add(5);
list.Add(8);
list.Add(5);
SCG.IEnumerator<int> e = list.GetEnumerator();
e.MoveNext();
list.Add(99); // <-- Bug: this should be Update()
Assert.Throws<CollectionModifiedException>(() => e.MoveNext());
}
The test checks if calling Add() makes the enumerator fail. It should probably be a call to Update() instead.
I'm not sure if this mistake has been duplicated elsewhere.
The
MoveNextAfterUpdate()
test inArrayListTest
seems to contain a mistake causing it to test the wrong thing:The test checks if calling
Add()
makes the enumerator fail. It should probably be a call toUpdate()
instead.I'm not sure if this mistake has been duplicated elsewhere.