diff options
author | 2021-05-05 12:06:00 +0300 | |
---|---|---|
committer | 2021-05-11 09:39:59 +0300 | |
commit | 25e3306737b7284b051dfeaedb39ef83323504d9 (patch) | |
tree | 8f5b64bfb21bf11ebbbca558a23666128dcdc5e8 /cps-service/src/main/java/org/onap | |
parent | 757b328b542d91a96d2c095744303d40edcb67f9 (diff) |
Create list-node elements (part1): CPS service and persistence layers
+ fix integrity violation exception exposed out of persistence layer
+ refactor CpsDataServiceImplSpec to eliminate repeated code
Issue-ID: CPS-360
Change-Id: Id70341fe54bf3c31af661f6aae04a7a80f4a1e9d
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
Diffstat (limited to 'cps-service/src/main/java/org/onap')
4 files changed, 62 insertions, 0 deletions
diff --git a/cps-service/src/main/java/org/onap/cps/api/CpsDataService.java b/cps-service/src/main/java/org/onap/cps/api/CpsDataService.java index 8552c6c0d..8e59ebcbb 100644 --- a/cps-service/src/main/java/org/onap/cps/api/CpsDataService.java +++ b/cps-service/src/main/java/org/onap/cps/api/CpsDataService.java @@ -57,6 +57,21 @@ public interface CpsDataService { @NonNull String jsonData); /** + * Persists child data fragment representing list-node (with one or more elements) under existing data node + * for the given anchor and dataspace. + * + * @param dataspaceName dataspace name + * @param anchorName anchor name + * @param parentNodeXpath parent node xpath + * @param jsonData json data representing list element + * @throws DataValidationException when json data is invalid (incl. list-node being empty) + * @throws DataNodeNotFoundException when parent node cannot be found by parent node xpath + * @throws AlreadyDefinedException when any of child data nodes is having xpath of already existing node + */ + void saveListNodeData(@NonNull String dataspaceName, @NonNull String anchorName, @NonNull String parentNodeXpath, + @NonNull String jsonData); + + /** * Retrieves datanode by XPath for given dataspace and anchor. * * @param dataspaceName dataspace name diff --git a/cps-service/src/main/java/org/onap/cps/api/impl/CpsDataServiceImpl.java b/cps-service/src/main/java/org/onap/cps/api/impl/CpsDataServiceImpl.java index fd0f76b53..523657a7f 100755 --- a/cps-service/src/main/java/org/onap/cps/api/impl/CpsDataServiceImpl.java +++ b/cps-service/src/main/java/org/onap/cps/api/impl/CpsDataServiceImpl.java @@ -21,11 +21,13 @@ package org.onap.cps.api.impl; +import java.util.Collection; import org.onap.cps.api.CpsAdminService; import org.onap.cps.api.CpsDataService; import org.onap.cps.api.CpsModuleService; import org.onap.cps.spi.CpsDataPersistenceService; import org.onap.cps.spi.FetchDescendantsOption; +import org.onap.cps.spi.exceptions.DataValidationException; import org.onap.cps.spi.model.DataNode; import org.onap.cps.spi.model.DataNodeBuilder; import org.onap.cps.utils.YangUtils; @@ -65,6 +67,17 @@ public class CpsDataServiceImpl implements CpsDataService { } @Override + public void saveListNodeData(final String dataspaceName, final String anchorName, + final String parentNodeXpath, final String jsonData) { + final Collection<DataNode> dataNodesCollection = + buildDataNodeCollectionFromJson(dataspaceName, anchorName, parentNodeXpath, jsonData); + if (dataNodesCollection.isEmpty()) { + throw new DataValidationException("Invalid list data.", "List node is empty."); + } + cpsDataPersistenceService.addListDataNodes(dataspaceName, anchorName, parentNodeXpath, dataNodesCollection); + } + + @Override public DataNode getDataNode(final String dataspaceName, final String anchorName, final String xpath, final FetchDescendantsOption fetchDescendantsOption) { return cpsDataPersistenceService.getDataNode(dataspaceName, anchorName, xpath, fetchDescendantsOption); @@ -103,6 +116,19 @@ public class CpsDataServiceImpl implements CpsDataService { .build(); } + private Collection<DataNode> buildDataNodeCollectionFromJson(final String dataspaceName, final String anchorName, + final String parentNodeXpath, final String jsonData) { + + final var anchor = cpsAdminService.getAnchor(dataspaceName, anchorName); + final var schemaContext = getSchemaContext(dataspaceName, anchor.getSchemaSetName()); + + final NormalizedNode<?, ?> normalizedNode = YangUtils.parseJsonData(jsonData, schemaContext, parentNodeXpath); + return new DataNodeBuilder() + .withParentNodeXpath(parentNodeXpath) + .withNormalizedNodeTree(normalizedNode) + .buildCollection(); + } + private SchemaContext getSchemaContext(final String dataspaceName, final String schemaSetName) { return yangTextSchemaSourceSetCache.get(dataspaceName, schemaSetName).getSchemaContext(); } diff --git a/cps-service/src/main/java/org/onap/cps/spi/CpsDataPersistenceService.java b/cps-service/src/main/java/org/onap/cps/spi/CpsDataPersistenceService.java index 48f9763ee..0ed3bf005 100644 --- a/cps-service/src/main/java/org/onap/cps/spi/CpsDataPersistenceService.java +++ b/cps-service/src/main/java/org/onap/cps/spi/CpsDataPersistenceService.java @@ -55,6 +55,18 @@ public interface CpsDataPersistenceService { @NonNull DataNode dataNode); /** + * Adds list node child elements to a Fragment. + * + * @param dataspaceName dataspace name + * @param anchorName anchor name + * @param parentNodeXpath parent node xpath + * @param dataNodes collection of data nodes representing list node elements + */ + + void addListDataNodes(@NonNull String dataspaceName, @NonNull String anchorName, @NonNull String parentNodeXpath, + @NonNull Collection<DataNode> dataNodes); + + /** * Retrieves datanode by XPath for given dataspace and anchor. * * @param dataspaceName dataspace name @@ -100,4 +112,5 @@ public interface CpsDataPersistenceService { */ Collection<DataNode> queryDataNodes(@NonNull String dataspaceName, @NonNull String anchorName, @NonNull String cpsPath, @NonNull FetchDescendantsOption fetchDescendantsOption); + } diff --git a/cps-service/src/main/java/org/onap/cps/spi/exceptions/AlreadyDefinedException.java b/cps-service/src/main/java/org/onap/cps/spi/exceptions/AlreadyDefinedException.java index 9e54f3493..1352a9b6c 100644 --- a/cps-service/src/main/java/org/onap/cps/spi/exceptions/AlreadyDefinedException.java +++ b/cps-service/src/main/java/org/onap/cps/spi/exceptions/AlreadyDefinedException.java @@ -19,6 +19,8 @@ package org.onap.cps.spi.exceptions; +import java.util.Collection; + /** * Already defined exception. Indicates the cps object with same name already exists. */ @@ -70,4 +72,10 @@ public class AlreadyDefinedException extends CpsAdminException { final Throwable cause) { return new AlreadyDefinedException("Data node", xpath, contextName, cause); } + + public static AlreadyDefinedException forDataNodes(final Collection<String> xpaths, final String contextName, + final Throwable cause) { + final var name = String.format("(one or more) of %s", xpaths); + return new AlreadyDefinedException("Data node", name, contextName, cause); + } } |