mbaedke commented on PR #730: URL: https://github.com/apache/commons-collections/pull/730#issuecomment-5634899944
> I think we need more tests to verify nothing breaks for the full sequence, exhaustion, remove(), empty leading iterators, and rechaining after at least one successful next() call. Sorry for the late response. After thinking about it: this PR together with https://github.com/apache/commons-collections/pull/628 fundamentally change the way addIterator() treats ChainedIterators. Wouldn't it be very reasonable to parameterize the test so that it runs a second time with makeObject() creating a chain of ChainedIteraors instead of ListIterators? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
