Ted,
Not sure if it mattered, but I used "diff -u -b" for the patches I submitted. I found out that my editor trimmed a lot of lines that only had spaces, and so the diffs I initially generated caught all of them. The ones I eventually submitted didn't have the whitespace changes.
Also, the method I added to the RequestProcessor is being tested in TestActionRedirect. There's no TestRequestProcessor class (yet). If I created one to test one method, it might give someone else the impression that the whole class was being tested.
Or it might encourage someone else to write tests for other methods, since the "ice" would have been "broken"!
Don't let impressions bother you. With tests, any are better than none.
Joe
--
Joe Germuska [EMAIL PROTECTED] http://blog.germuska.com "Imagine if every Thursday your shoes exploded if you tied them the usual way. This happens to us all the time with computers, and nobody thinks of complaining."
-- Jef Raskin
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
