[SLING-8161] MockSlingHttpServletResponse.sendError(int sc, String msg) does not save msg String

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: Servlet Helpers 1.1.8, Servlet Helpers 1.1.10
• Fix Version/s: Servlet Helpers 1.1.10
• Component/s: Servlets, Testing
• Assignee: Stefan Seifert
• Reporter: Rob McDougall
• Created: 04/Dec/18 16:36
• Updated: 07/Mar/19 16:10
• Resolved: 05/Dec/18 20:13

Description

org.apache.sling.servlethelpers.MockSlingHttpServletResponse.sendError(int sc, String msg) saves the sc parameter, but discards the msg parameter.  This makes it impossible to verify the contents of the message in unit tests that use this mock.

It would be trivial to add a member variable of type String to the class in order to store that message and then add a getter to retrieve it.  This would make it possible to verify the contents of the message in a unit test.