https://issues.apache.org/bugzilla/show_bug.cgi?id=54395

Glen Taylor <inetdevboy+apa...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #29835|0                           |1
        is obsolete|                            |

--- Comment #4 from Glen Taylor <inetdevboy+apa...@gmail.com> ---
Created attachment 29836
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=29836&action=edit
Updated unit test per comments

Well, the JUnit 3.8.1 pulled in by Maven doesn't have an "assertFail(String)"
method, but I think the "fail(String)" alternative is there in spirit. 
Point(s) taken, unit test attachment updated.

I had been doing more with the Exception in case 1, and didn't refactor far
enough when that changed, and case 2 was copy/paste laziness (blah,blah). 
Thanks for the good suggestions.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to