On Sun, Aug 8, 2010 at 1:39 PM, sebb <seb...@gmail.com> wrote: > StringUtils.length() is now deprecated, and is replaced by > CharSequenceUtils.length(), but surely the deprecated methods should > still be tested? > The deprecation warnings can be suppressed. > > When the deprecated method is deleted, the tests can be dropped entirely. > > If the corresponding CharSequenceUtils.length() tests don't exist, > then they should be added. >
Yes, you're right. I didn't realize the name of the methods and what they're testing. I was in clean-up mode trying to remove all the little yellow lines from the right side of my code editor. I'll try to revert at least these. I don't agree that I'm changing the tests in the varargs cases, though. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org