Include org.apache.sling.testing.hamcrest in global parents, for easier unit testing

Description

It would be nice to include
```
<dependency>
<groupId>org.apache.sling</groupId>
<artifactId>org.apache.sling.testing.hamcrest</artifactId>
<version></version>
<scope>test</scope>
</dependency>
```
in the global parent, to make unit testing easier with the Sling Hamcrest Matchers.

Activity

Show:
Stefan Seifert
February 24, 2020, 4:38 PM

the aem-global-parent currently does not declare any testing dependencies.
but we could add it as compile dependency to aem-mock so it's avialable in all projects referencing aem-mock as test dependency.

WDYT?

Henry Kuijpers
February 25, 2020, 8:57 AM

@Stefan That sounds good to me as well.

Henry Kuijpers
February 25, 2020, 8:58 AM

I thought one of the global parents defined AEM Mocks as a testing dependency, but maybe I’m wrong there. Wouldn’t it be a good idea to do that?

Stefan Seifert
February 28, 2020, 10:04 AM

https://github.com/wcm-io/wcm-io-testing/commit/f576a1522084af972b53f4dd4d5d9c543ed05c12

i've included it in aem-mocks as compile dependency for convenience.

i'm very hesitant adding and direct dependency to aem-global-parent directly (we only have few for static code analysis and OSGi annotations) - it forces them down on all projects depending from it, you can override them but never can get rid of them downstream.

Fixed

Assignee

Stefan Seifert

Reporter

Henry Kuijpers

Labels

None

Components

Fix versions

Priority

Major
Configure