inteligr8/alfresco-community-repo
1
0
Fork 0
mirror of https://github.com/Alfresco/alfresco-community-repo.git synced 2025-07-31 17:39:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

REPO-5209: Prevent creation of node updated event when property added and removed in same transaction (#113)

Browse Source
This commit is contained in:
Chris Shields
2020-11-09 11:34:53 +00:00
committed by GitHub
parent 6d5583c048
commit 27d5e144a9
3 changed files with 38 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
repository/src/test/java/org/alfresco/repo/event2/UpdateRepoEventIT.java
View File

@@ -30,6 +30,7 @@ import java.io.Serializable;
import java.util.Collection;
import java.util.HashMap;
import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Set;
@@ -978,4 +979,32 @@ public class UpdateRepoEventIT extends AbstractContextAwareRepoEvent
assertEquals(1, aspectsBefore.size());
assertTrue(aspectsBefore.contains("cm:auditable"));
}
@Test
public void testAddAndRemovePropertyInTheSameTransaction()
{
final NodeRef nodeRef = createNode(ContentModel.TYPE_CONTENT);
checkNumOfEvents(1);
NodeResource resource = getNodeResource(1);
// Check properties
assertTrue(resource.getProperties().isEmpty());
// Add and remove cm:userName property
retryingTransactionHelper.doInTransaction(() -> {
Map<QName, Serializable> properties = Map.of(ContentModel.PROP_USERNAME, "user1");
nodeService.addProperties(nodeRef, properties);
nodeService.removeProperty(nodeRef, ContentModel.PROP_USERNAME);
return null;
});
// There should only be a create event
resource = getNodeResource(1);
assertTrue(resource.getProperties().isEmpty());
// Check there isn't a node update event
List<RepoEvent<EventData<NodeResource>>> nodeUpdatedEvents = getFilteredEvents(EventType.NODE_UPDATED);
assertEquals(0, nodeUpdatedEvents.size());
}
}