dblock commented on pull request #711: URL: https://github.com/apache/lucene/pull/711#issuecomment-1055853457
> > Should we parameterize the number of iterations parameter and use it as test hook for getting the method to throw IllegalArgumentException? > > I'm not sure how useful that is as a test. I won't argue, but I have hard time accepting that we can fix a real bug, and write no new tests. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
