[ 
https://issues.apache.org/jira/browse/SLING-9903?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17230665#comment-17230665
 ] 

Bertrand Delacretaz edited comment on SLING-9903 at 11/12/20, 2:32 PM:
-----------------------------------------------------------------------

[~Henry Kuijpers], as you wrote the existing in-Sling tests, what do you think 
of 
[https://github.com/apache/sling-org-apache-sling-adapter-annotations/pull/2] ?

It's just testing the {{OSGI-INF/*.xml}} files for data that's generated due to 
our annotations, instead of running a Sling instance. Do you think we need to 
test more than that?


was (Author: bdelacretaz):
[~Henry Kuijpers] what do you think of 
[https://github.com/apache/sling-org-apache-sling-adapter-annotations/pull/2] ?

It's just testing the {{OSGI-INF/*.xml}} files for data that's generated due to 
our annotations, instead of running a Sling instance. Do you think we need to 
test more than that?

> Validate the generated OSGi metatada instead of running a Sling instance
> ------------------------------------------------------------------------
>
>                 Key: SLING-9903
>                 URL: https://issues.apache.org/jira/browse/SLING-9903
>             Project: Sling
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: Adapter Annotations 2.0.0
>            Reporter: Bertrand Delacretaz
>            Assignee: Bertrand Delacretaz
>            Priority: Minor
>
> I took another look at the 
> https://github.com/apache/sling-org-apache-sling-adapter-annotations and 
> considering that the annotations act at build time only (together with the 
> bnd plugin) I don't think we need to start a Sling instance to validate them.
> I'll prepare a PR that checks the XML files generated in the OSGI-INF folder, 
> that should be sufficient.



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

Reply via email to