You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Stefan Seifert (JIRA)" <ji...@apache.org> on 2015/09/18 01:56:04 UTC
[jira] [Resolved] (SLING-5042) ResourceResolver.commit() does not
call session.save() on the backing JCR Session
[ https://issues.apache.org/jira/browse/SLING-5042?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Stefan Seifert resolved SLING-5042.
-----------------------------------
Resolution: Fixed
Fix Version/s: Testing Sling Mock 1.5.2
oh, yes - should be fixed in rev. 1703732
> ResourceResolver.commit() does not call session.save() on the backing JCR Session
> ---------------------------------------------------------------------------------
>
> Key: SLING-5042
> URL: https://issues.apache.org/jira/browse/SLING-5042
> Project: Sling
> Issue Type: Bug
> Components: Testing
> Affects Versions: Testing Sling Mock 1.5.0
> Reporter: Robert Munteanu
> Assignee: Stefan Seifert
> Priority: Minor
> Fix For: Testing Sling Mock 1.5.2
>
>
> The following test snippet passes:
> {code:java}
> resourceResolver.delete(content);
> assertTrue(session.hasPendingChanges());
> resourceResolver.adaptTo(Session.class).save();
> assertFalse(session.hasPendingChanges());
> {code}
> but the following one fails
> {code:java}
> resourceResolver.delete(content);
> assertTrue(session.hasPendingChanges());
> resourceResolver.commit();
> assertFalse(session.hasPendingChanges());
> {code}
> By taking a quick look it seems that {{ResourceResolverContext}} instance of the {{ResourceResolverImpl}} does not have any registered providers, so the commit() operation is effectively a no-op
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)