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

Bertrand Delacretaz commented on SLING-10742:
---------------------------------------------

If someone's willing and able to create integration tests directly in the 
{{org.apache.sling.resourceresolver}} module I think that would be nice. The 
ones that Robert mentions look like they could run there as well, with pax exam 
or Sling Mocks, and having the ITs in the same module is a good thing IMO.

> Create integration tests for ResourceResolver
> ---------------------------------------------
>
>                 Key: SLING-10742
>                 URL: https://issues.apache.org/jira/browse/SLING-10742
>             Project: Sling
>          Issue Type: Improvement
>          Components: ResourceResolver
>    Affects Versions: Resource Resolver 1.7.10
>            Reporter: Henry Kuijpers
>            Priority: Minor
>
> Now that the logic in Resource Resolver becomes more and more complex, for 
> example with regards to executing querys, it would be a good idea to create 
> some integration tests for this module.
> Mocking frameworks only go so far, many assumptions are made. In particular, 
> query execution is not done in our mocking frameworks. Instead of executing 
> them, we're basically saying "If we receive query 'x', we return results [y, 
> z]", without validating the query, without parsing it, without executing it. 
> This came up in SLING-10447, where we decided that verifying the correct 
> execution of the querys would be valuable to have. Right now, only the string 
> (that contains the query) is checked, but we're not checking if there are any 
> syntax issues and we're also not checking if the query executes correctly, 
> i.e. returns the expected results.
> We will probably come up with some more complex logic in SLING-10466, which 
> could also benefit from these integration tests.



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

Reply via email to