You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Konrad Windszus (Jira)" <ji...@apache.org> on 2020/10/28 08:34:00 UTC

[jira] [Resolved] (SLING-9858) Get rid of commons testing dependency

     [ https://issues.apache.org/jira/browse/SLING-9858?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Konrad Windszus resolved SLING-9858.
------------------------------------
    Resolution: Fixed

Fixed in https://github.com/apache/sling-org-apache-sling-models-impl/commit/55f74cb55e7342b73060ca1da8e385796f683e54.

> Get rid of commons testing dependency
> -------------------------------------
>
>                 Key: SLING-9858
>                 URL: https://issues.apache.org/jira/browse/SLING-9858
>             Project: Sling
>          Issue Type: Improvement
>          Components: Sling Models
>    Affects Versions: Models Implementation 1.4.16
>            Reporter: Konrad Windszus
>            Assignee: Konrad Windszus
>            Priority: Major
>             Fix For: Models Implementation 1.4.18
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> Given that there are better alternatives for most packages in Commons Testing (https://issues.apache.org/jira/browse/SLING-7166) we should get rid of the dependency in Models Impl.
> As only a mock for SlingHttpServletRequest is used from commons testing AFAICS we should start using https://github.com/apache/sling-org-apache-sling-servlet-helpers/blob/master/src/main/java/org/apache/sling/servlethelpers/MockSlingHttpServletRequest.java instead.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)