From d924ab57a11af19f92037d150e5e5e8ea52f04a8 Mon Sep 17 00:00:00 2001 From: ToineSiebelink Date: Wed, 2 Feb 2022 12:17:15 +0000 Subject: Test Scenario for replaceListContent -Improved a test scenarios for replaceListContent in CpsDataPersistenceServiceSpec -Removed redundant SQL Data -Used groovy shortcuts where posisble (no need for get...() methods when using Groovy :-) Issue-ID: CPS-868 Change-Id: I3d29bbb0c5d035c6f266cf31bf2886f1bcf5d48a Signed-off-by: mpriyank Signed-off-by: ToineSiebelink --- .../java/org/onap/cps/spi/impl/CpsDataPersistenceServiceImpl.java | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'cps-ri/src/main/java') diff --git a/cps-ri/src/main/java/org/onap/cps/spi/impl/CpsDataPersistenceServiceImpl.java b/cps-ri/src/main/java/org/onap/cps/spi/impl/CpsDataPersistenceServiceImpl.java index c6e28aba9..2d67d7c95 100644 --- a/cps-ri/src/main/java/org/onap/cps/spi/impl/CpsDataPersistenceServiceImpl.java +++ b/cps-ri/src/main/java/org/onap/cps/spi/impl/CpsDataPersistenceServiceImpl.java @@ -48,6 +48,7 @@ import org.onap.cps.spi.entities.DataspaceEntity; import org.onap.cps.spi.entities.FragmentEntity; import org.onap.cps.spi.exceptions.AlreadyDefinedException; import org.onap.cps.spi.exceptions.ConcurrencyException; +import org.onap.cps.spi.exceptions.CpsAdminException; import org.onap.cps.spi.exceptions.CpsPathException; import org.onap.cps.spi.exceptions.DataNodeNotFoundException; import org.onap.cps.spi.model.DataNode; @@ -384,6 +385,10 @@ public class CpsDataPersistenceServiceImpl implements CpsDataPersistenceService } private static String getListElementXpathPrefix(final Collection newListElements) { + if (newListElements.isEmpty()) { + throw new CpsAdminException("Invalid list replacement", + "Cannot replace list elements with empty collection"); + } final String firstChildNodeXpath = newListElements.iterator().next().getXpath(); return firstChildNodeXpath.substring(0, firstChildNodeXpath.lastIndexOf("[") + 1); } -- cgit 1.2.3-korg