sling-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nicolas Peltier (JIRA)" <>
Subject [jira] [Commented] (SLING-7692) MockSession.revert(false) should be supported
Date Fri, 01 Jun 2018 07:28:00 GMT


Nicolas Peltier commented on SLING-7692:

{quote}would be better to do such things no a branch first{quote} yeah sorry about that. Do
you think --force push on github would be reflected on gitbox? 

{quote}jcr-mock has support for multiple sessions{quote} isn't this the issue? I mean the
problem then is not only refresh(false) should throw unsupported, but refresh(*). I guess
then it drills down either to support multiple session, and not support refreshes, either
not support multiple session, and support refresh(false) (refresh(true) being cool then as
a NOP).
I'd lean towards the later as i see less cases of multiple session than reverts in a unit
test context, but i'll take your word.

> MockSession.revert(false) should be supported
> ---------------------------------------------
>                 Key: SLING-7692
>                 URL:
>             Project: Sling
>          Issue Type: Improvement
>          Components: Testing
>    Affects Versions: Testing JCR Mock 1.3.2
>            Reporter: Nicolas Peltier
>            Assignee: Nicolas Peltier
>            Priority: Major
>             Fix For: Testing JCR Mock 1.3.4
> MockSession does not cancel the changes when refresh(false) is called, hence resolver.revert()
not working in case JCR_MOCK is used

This message was sent by Atlassian JIRA

View raw message