[SLING-7744] MockRequestPathInfo: Implement getSuffixResource()

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: Servlet Helpers 1.1.6
• Fix Version/s: Servlet Helpers 1.1.8
• Component/s: Servlets, Testing
• Flags: Patch
• Assignee: Stefan Seifert
• Reporter: Jens Lauterbach
• Created: 18/Jun/18 13:46
• Updated: 06/Aug/18 08:17
• Resolved: 31/Jul/18 11:11

Description

The org.apache.sling.servlethelpers.MockRequestPathInfo allows consumers to set almost every field except the suffixResource. Currently, the getSuffixResource() just throws a UnsupportedOperationException.

I propose implementing a setSuffixResource method equivalent to all the other setters in this mock.

As far as I can tell this can be easily implemented (see attached patch) and would improve usability of this mock a little bit.

Issue Links

• relates to: SLING-7804 sling-mock: Instantiate MockRequestPathInfo for resource resolver context
• links to: GitHub Pull Request #1
• links to: GitHub Pull Request #2