summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPamela Dragosh <pdragosh@research.att.com>2020-02-19 16:59:01 -0500
committerPamela Dragosh <pdragosh@research.att.com>2020-02-19 19:59:33 -0500
commit3f4d4125093ca000c735adc1d6430c39e31de38c (patch)
tree5386fae347d6445bdba7599d0f42aefc85d867df
parent56038a31a23e468f51c0b2ab349f748eafe4b6d2 (diff)
Use datatype in matchable translator
With the new changes to models/api, the full policy template will be retrieved along with inherited policy types and datatypes. Changed api caller to return tosca template instead of individual policy type so all policy types and data types are available. Changed stdmatchabletranslator to be able to flexibly go deep with list/map of datatypes of attributes. NOTE: This should be re-factored at a later date as its a bit messy. The naming application was used to test this code, but it was determined by the CCSDK team that they will not do filtering this release. Instead, switched to combined results policy type to return the full policy type. Issue-ID: POLICY-1740 Change-Id: Ieb4e46e7fb2c639a1155be3bc07f5587c4f7b09a Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
-rw-r--r--applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/PolicyApiCaller.java8
-rw-r--r--applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslator.java541
-rw-r--r--applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslatorTest.java23
-rw-r--r--applications/common/src/test/resources/matchable/onap.policies.Test-1.0.0.yaml35
-rw-r--r--applications/common/src/test/resources/matchable/test.policies.input.tosca.yaml9
-rw-r--r--applications/naming/src/main/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplication.java24
-rw-r--r--applications/naming/src/test/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplicationTest.java22
7 files changed, 443 insertions, 219 deletions
diff --git a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/PolicyApiCaller.java b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/PolicyApiCaller.java
index 9d47517a..632bdcb2 100644
--- a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/PolicyApiCaller.java
+++ b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/PolicyApiCaller.java
@@ -1,6 +1,6 @@
/*-
* ============LICENSE_START=======================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -28,8 +28,8 @@ import org.onap.policy.common.endpoints.http.client.HttpClientConfigException;
import org.onap.policy.common.endpoints.http.client.HttpClientFactoryInstance;
import org.onap.policy.common.endpoints.parameters.RestServerParameters;
import org.onap.policy.common.gson.GsonMessageBodyHandler;
-import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicyType;
import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicyTypeIdentifier;
+import org.onap.policy.models.tosca.authorative.concepts.ToscaServiceTemplate;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -75,7 +75,7 @@ public class PolicyApiCaller {
* @return the desired policy type
* @throws PolicyApiException if an error occurs
*/
- public ToscaPolicyType getPolicyType(ToscaPolicyTypeIdentifier type) throws PolicyApiException {
+ public ToscaServiceTemplate getPolicyType(ToscaPolicyTypeIdentifier type) throws PolicyApiException {
try {
Response resp = httpClient
@@ -83,7 +83,7 @@ public class PolicyApiCaller {
switch (resp.getStatus()) {
case HttpURLConnection.HTTP_OK:
- return resp.readEntity(ToscaPolicyType.class);
+ return resp.readEntity(ToscaServiceTemplate.class);
case HttpURLConnection.HTTP_NOT_FOUND:
logger.warn("policy-api not found {}", resp);
throw new NotFoundException(type.toString());
diff --git a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslator.java b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslator.java
index ae09ec10..0d1daaa6 100644
--- a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslator.java
+++ b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslator.java
@@ -2,7 +2,7 @@
* ============LICENSE_START=======================================================
* ONAP
* ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -34,10 +34,8 @@ import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
-import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
-import java.util.Collections;
import java.util.HashMap;
import java.util.LinkedHashMap;
import java.util.LinkedList;
@@ -58,12 +56,12 @@ import org.onap.policy.common.utils.coder.StandardCoder;
import org.onap.policy.common.utils.coder.StandardYamlCoder;
import org.onap.policy.models.decisions.concepts.DecisionRequest;
import org.onap.policy.models.decisions.concepts.DecisionResponse;
+import org.onap.policy.models.tosca.authorative.concepts.ToscaDataType;
import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicy;
import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicyType;
import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicyTypeIdentifier;
import org.onap.policy.models.tosca.authorative.concepts.ToscaProperty;
import org.onap.policy.models.tosca.authorative.concepts.ToscaServiceTemplate;
-import org.onap.policy.models.tosca.simple.concepts.JpaToscaServiceTemplate;
import org.onap.policy.pdp.xacml.application.common.OnapObligation;
import org.onap.policy.pdp.xacml.application.common.PolicyApiCaller;
import org.onap.policy.pdp.xacml.application.common.PolicyApiException;
@@ -86,7 +84,7 @@ public class StdMatchableTranslator extends StdBaseTranslator {
private static final Logger LOGGER = LoggerFactory.getLogger(StdMatchableTranslator.class);
private static final StandardYamlCoder standardYamlCoder = new StandardYamlCoder();
- private final Map<ToscaPolicyTypeIdentifier, ToscaPolicyType> matchablePolicyTypes = new HashMap<>();
+ private final Map<ToscaPolicyTypeIdentifier, ToscaServiceTemplate> matchablePolicyTypes = new HashMap<>();
@Setter
private RestServerParameters apiRestParameters;
@Setter
@@ -236,12 +234,12 @@ public class StdMatchableTranslator extends StdBaseTranslator {
//
// Get the TOSCA Policy Type for this policy
//
- Collection<ToscaPolicyType> toscaPolicyTypes = this.getPolicyTypes(toscaPolicy.getTypeIdentifier());
+ ToscaServiceTemplate toscaPolicyTypeTemplate = this.findPolicyType(toscaPolicy.getTypeIdentifier());
//
// If we don't have any TOSCA policy types, then we cannot know
// which properties are matchable.
//
- if (toscaPolicyTypes.isEmpty()) {
+ if (toscaPolicyTypeTemplate == null) {
throw new ToscaPolicyConversionException(
"Cannot retrieve Policy Type definition for policy " + toscaPolicy.getName());
}
@@ -270,7 +268,7 @@ public class StdMatchableTranslator extends StdBaseTranslator {
// Generate the TargetType - the policy should not be evaluated
// unless all the matchable properties it cares about are matched.
//
- Pair<TargetType, Integer> pairGenerated = generateTargetType(toscaPolicy.getProperties(), toscaPolicyTypes);
+ Pair<TargetType, Integer> pairGenerated = generateTargetType(toscaPolicy, toscaPolicyTypeTemplate);
newPolicyType.setTarget(pairGenerated.getLeft());
//
// Now represent the policy as Json
@@ -326,97 +324,314 @@ public class StdMatchableTranslator extends StdBaseTranslator {
* @param policyTypes Collection of policy Type to find matchable metadata
* @return {@code Pair<TargetType, Integer>} Returns a TargetType and a Total Weight of matchables.
*/
- protected Pair<TargetType, Integer> generateTargetType(Map<String, Object> properties,
- Collection<ToscaPolicyType> policyTypes) {
+ protected Pair<TargetType, Integer> generateTargetType(ToscaPolicy policyType,
+ ToscaServiceTemplate policyTemplate) {
+ //
+ // Our return object
+ //
TargetType targetType = new TargetType();
//
- // Iterate the properties
+ // Top-level list of properties
+ //
+ Map<String, Object> properties = policyType.getProperties();
+ //
+ // To start, we know these properties are for this specific Policy Type ID/Version
+ //
+ ToscaPolicyTypeIdentifier propertiesPolicyId = policyType.getTypeIdentifier();
+ //
+ // Scan the property map for matchables
//
- int totalWeight = findMatchableFromMap(properties, policyTypes, targetType);
+ int totalWeight = findMatchablesInProperties(properties, propertiesPolicyId, policyTemplate, targetType);
LOGGER.info("Total weight is {}", totalWeight);
return Pair.of(targetType, totalWeight);
}
- @SuppressWarnings("unchecked")
- protected int findMatchableFromList(List<Object> listProperties, Collection<ToscaPolicyType> policyTypes,
+ protected int findMatchablesInProperties(Map<String, Object> properties,
+ ToscaPolicyTypeIdentifier propertiesPolicyId,
+ ToscaServiceTemplate policyTemplate,
TargetType targetType) {
- LOGGER.info("findMatchableFromList {}", listProperties);
+ LOGGER.info("findMatchablesInProperties from policy Type {} {}", propertiesPolicyId, properties);
+ //
+ // We better have the policy type definition available from the template
+ //
+ ToscaPolicyType policyType = getToscaPolicyTypeFromTemplate(propertiesPolicyId, policyTemplate);
+ if (policyType == null) {
+ LOGGER.error("Failed to find policy type in template {}", propertiesPolicyId);
+ return 0;
+ }
+ //
+ // Our total weight to return
+ //
int totalWeight = 0;
- for (Object property : listProperties) {
- if (property instanceof List) {
- totalWeight += findMatchableFromList((List<Object>) property, policyTypes, targetType);
- } else if (property instanceof Map) {
- totalWeight += findMatchableFromMap((Map<String, Object>) property, policyTypes, targetType);
+ for (Entry<String, Object> entrySet : properties.entrySet()) {
+ //
+ // Find the property details
+ //
+ Pair<ToscaProperty, ToscaServiceTemplate> property = findProperty(entrySet.getKey(),
+ policyType, propertiesPolicyId, policyTemplate);
+ if (property == null) {
+ continue;
+ }
+ ToscaProperty toscaProperty = property.getLeft();
+ LOGGER.info("Found property {} with type {} schema {}", entrySet.getKey(), toscaProperty.getType(),
+ (toscaProperty.getEntrySchema() == null ? "null" : toscaProperty.getEntrySchema().getType()));
+ //
+ // Is it matchable?
+ //
+ if (checkIsMatchableProperty(toscaProperty)) {
+ //
+ // This will generate the matchables for the property
+ //
+ int weight = generateMatchable(targetType, entrySet.getKey(), entrySet.getValue(),
+ property.getLeft(), property.getRight());
+ LOGGER.info("Weight is {}", weight);
+ totalWeight += weight;
+ } else {
+ //
+ // Not matchable, but we need to check if this contains list or map of datatypes.
+ // Those will need to be searched for matchables.
+ //
+ if ("list".equals(toscaProperty.getType())) {
+ int weight = findMatchablesInList(entrySet.getKey(), entrySet.getValue(), toscaProperty,
+ policyTemplate, targetType);
+ LOGGER.info("Weight list is {}", weight);
+ totalWeight += weight;
+ } else if ("map".equals(toscaProperty.getType())) {
+ int weight = findMatchablesInMap(entrySet.getKey(), entrySet.getValue(), toscaProperty,
+ policyTemplate, targetType);
+ LOGGER.info("Weight map is {}", weight);
+ totalWeight += weight;
+ }
}
}
return totalWeight;
}
- protected int findMatchableFromMap(Map<String, Object> properties, Collection<ToscaPolicyType> policyTypes,
- TargetType targetType) {
- LOGGER.info("findMatchableFromMap {}", properties);
+ @SuppressWarnings("unchecked")
+ protected int findMatchablesInList(String listPropertyName, Object listValue, ToscaProperty listProperty,
+ ToscaServiceTemplate listTemplate, TargetType targetType) {
+ //
+ // Don't bother if there is no schema (which should be a problem) or
+ // its a list of primitives
+ //
+ if (listProperty.getEntrySchema() == null) {
+ LOGGER.error("No entry schema for list property {}", listPropertyName);
+ return 0;
+ }
+ //
+ // If they are primitives, then no need to go through them. ??
+ //
+ if (isYamlType(listProperty.getEntrySchema().getType())) {
+ LOGGER.info("list of primitives");
+ return 0;
+ }
+ //
+ // Find the datatype
+ //
+ ToscaDataType listDataType = listTemplate.getDataTypes().get(listProperty.getEntrySchema().getType());
+ if (listDataType == null) {
+ LOGGER.error("Unable to find datatype {}", listProperty.getEntrySchema().getType());
+ return 0;
+ }
+
int totalWeight = 0;
- for (Entry<String, Object> entrySet : properties.entrySet()) {
+ for (Object datatypeValue : ((Collection<Object>)listValue)) {
//
- // Is this a matchable property?
+ // This should be a map - because this is a list of datatypes.
//
- if (isMatchable(entrySet.getKey(), policyTypes)) {
- LOGGER.info("Found matchable property {}", entrySet.getKey());
- int weight = generateMatchable(targetType, entrySet.getKey(), entrySet.getValue());
- LOGGER.info("Weight is {}", weight);
- totalWeight += weight;
- } else {
+ if (! (datatypeValue instanceof Map)) {
+ LOGGER.error("datatype {} value is not a map {}", listDataType.getName(), datatypeValue.getClass());
+ continue;
+ }
+ for (Entry<String, Object> entrySet : ((Map<String, Object>)datatypeValue).entrySet()) {
+ ToscaProperty toscaProperty = listDataType.getProperties().get(entrySet.getKey());
+ if (toscaProperty == null) {
+ LOGGER.error("Failed to find datatype {} property {}", listDataType.getName(), entrySet.getKey());
+ continue;
+ }
+ LOGGER.info("Found list property {} with type {} schema {}", entrySet.getKey(), toscaProperty.getType(),
+ (toscaProperty.getEntrySchema() == null ? "null" : toscaProperty.getEntrySchema().getType()));
//
- // Check if we need to search deeper
+ // Is it matchable?
//
- totalWeight += checkDeeperForMatchable(entrySet.getValue(), policyTypes, targetType);
+ if (checkIsMatchableProperty(toscaProperty)) {
+ //
+ // This will generate the matchables for the property
+ //
+ int weight = generateMatchable(targetType, entrySet.getKey(), entrySet.getValue(),
+ toscaProperty, listTemplate);
+ LOGGER.info("Weight is {}", weight);
+ totalWeight += weight;
+ } else {
+ //
+ // Not matchable, but we need to check if this contains list or map of datatypes.
+ // Those will need to be searched for matchables.
+ //
+ if ("list".equals(toscaProperty.getType())) {
+ int weight = findMatchablesInList(entrySet.getKey(), entrySet.getValue(), toscaProperty,
+ listTemplate, targetType);
+ LOGGER.info("Weight list is {}", weight);
+ totalWeight += weight;
+ } else if ("map".equals(toscaProperty.getType())) {
+ int weight = findMatchablesInMap(entrySet.getKey(), entrySet.getValue(), toscaProperty,
+ listTemplate, targetType);
+ LOGGER.info("Weight map is {}", weight);
+ totalWeight += weight;
+ }
+ }
}
}
+
return totalWeight;
}
@SuppressWarnings("unchecked")
- protected int checkDeeperForMatchable(Object property, Collection<ToscaPolicyType> policyTypes,
- TargetType targetType) {
- if (property instanceof List) {
- return findMatchableFromList((List<Object>) property, policyTypes, targetType);
- } else if (property instanceof Map) {
- return findMatchableFromMap((Map<String, Object>) property, policyTypes,
- targetType);
- }
- LOGGER.info("checkDeeperForMatchable not necessary for {}", property);
- return 0;
+ protected int findMatchablesInMap(String mapPropertyName, Object mapValue, ToscaProperty mapProperty,
+ ToscaServiceTemplate mapTemplate, TargetType targetType) {
+ //
+ // There needs to be a schema.
+ //
+ if (mapProperty.getEntrySchema() == null) {
+ LOGGER.error("No entry schema for map property {}", mapPropertyName);
+ return 0;
+ }
+ //
+ // If they are primitives, then no need to go through them. ??
+ //
+ if (isYamlType(mapProperty.getEntrySchema().getType())) {
+ LOGGER.debug("map property {} is primitives", mapPropertyName);
+ return 0;
+ }
+ //
+ // Find the datatype
+ //
+ ToscaDataType mapDataType = mapTemplate.getDataTypes().get(mapProperty.getEntrySchema().getType());
+ if (mapDataType == null) {
+ LOGGER.error("Unable to find datatype {}", mapProperty.getEntrySchema().getType());
+ return 0;
+ }
+
+ int totalWeight = 0;
+ for (Entry<String, Object> entrySet : ((Map<String, Object>)mapValue).entrySet()) {
+ ToscaProperty toscaProperty = mapDataType.getProperties().get(entrySet.getKey());
+ if (toscaProperty == null) {
+ LOGGER.error("Failed to find datatype {} property {}", mapDataType.getName(), entrySet.getKey());
+ continue;
+ }
+ LOGGER.info("Found map property {} with type {} schema {}", entrySet.getKey(), toscaProperty.getType(),
+ (toscaProperty.getEntrySchema() == null ? "null" : toscaProperty.getEntrySchema().getType()));
+ //
+ // Is it matchable?
+ //
+ if (checkIsMatchableProperty(toscaProperty)) {
+ //
+ // This will generate the matchables for the property
+ //
+ int weight = generateMatchable(targetType, entrySet.getKey(), entrySet.getValue(),
+ toscaProperty, mapTemplate);
+ LOGGER.info("Weight is {}", weight);
+ totalWeight += weight;
+ } else {
+ //
+ // Not matchable, but we need to check if this contains list or map of datatypes.
+ // Those will need to be searched for matchables.
+ //
+ if ("list".equals(toscaProperty.getType())) {
+ int weight = findMatchablesInList(entrySet.getKey(), entrySet.getValue(), toscaProperty,
+ mapTemplate, targetType);
+ LOGGER.info("Weight list is {}", weight);
+ totalWeight += weight;
+ } else if ("map".equals(toscaProperty.getType())) {
+ int weight = findMatchablesInMap(entrySet.getKey(), entrySet.getValue(), toscaProperty,
+ mapTemplate, targetType);
+ LOGGER.info("Weight map is {}", weight);
+ totalWeight += weight;
+ }
+ }
+ }
+
+ return totalWeight;
}
/**
- * isMatchable - Iterates through available TOSCA Policy Types to determine if a property
- * should be treated as matchable.
+ * findMatchableProperty - Iterates through available TOSCA Policy Types and return the
+ * ToscaProperty and template for the property.
*
* @param propertyName Name of property
* @param policyTypes Collection of TOSCA Policy Types to scan
- * @return true if matchable
+ * @return ToscaProperty and ToscaServiceTemplate if matchable
*/
- protected boolean isMatchable(String propertyName, Collection<ToscaPolicyType> policyTypes) {
- for (ToscaPolicyType policyType : policyTypes) {
- for (Entry<String, ToscaProperty> propertiesEntry : policyType.getProperties().entrySet()) {
- if (checkIsMatchableProperty(propertyName, propertiesEntry)) {
- return true;
+ protected Pair<ToscaProperty, ToscaServiceTemplate> findProperty(String propertyName,
+ ToscaPolicyType policyType, ToscaPolicyTypeIdentifier propertiesPolicyId,
+ ToscaServiceTemplate policyTemplate) {
+ //
+ // See if the property is defined by the policy template
+ //
+ ToscaProperty toscaProperty = policyType.getProperties().get(propertyName);
+ if (toscaProperty != null) {
+ //
+ // Does it contain the matchable property and if so its set to true?
+ //
+ return Pair.of(toscaProperty, policyTemplate);
+ }
+ LOGGER.debug("property {} is not in policy type {}", propertyName, propertiesPolicyId);
+ //
+ // Check its parent policy types
+ //
+ ToscaPolicyTypeIdentifier parentId = getParentDerivedFrom(propertiesPolicyId, policyTemplate);
+ while (parentId != null) {
+ LOGGER.debug("searching parent policy type {}", parentId);
+ //
+ // Search the existing template (should be there during runtime)
+ //
+ ToscaPolicyType parentPolicyType = getParentPolicyType(parentId, policyTemplate);
+ if (parentPolicyType != null) {
+ toscaProperty = parentPolicyType.getProperties().get(propertyName);
+ if (toscaProperty != null) {
+ return Pair.of(toscaProperty, policyTemplate);
}
//
- // Check if its a list or map
+ // Move to the next parent
+ //
+ parentId = getParentDerivedFrom(parentId, policyTemplate);
+ } else {
+ LOGGER.warn("Parent policy type is not found {}", parentId);
+ //
+ // Find the parent policy type. During JUnit this may be in a separate
+ // file. We hope that during runtime the template is complete.
//
- if (isListOrMap(propertiesEntry.getValue().getType())
- && ! isYamlType(propertiesEntry.getValue().getEntrySchema().getType())) {
- LOGGER.info("need to search list or map");
+ ToscaServiceTemplate parentTemplate = findPolicyType(parentId);
+ if (parentTemplate != null) {
+ parentPolicyType = getParentPolicyType(parentId, parentTemplate);
+ if (parentPolicyType != null) {
+ toscaProperty = parentPolicyType.getProperties().get(propertyName);
+ if (toscaProperty != null) {
+ return Pair.of(toscaProperty, parentTemplate);
+ }
+ }
+ //
+ // Move to the next parent
+ //
+ parentId = getParentDerivedFrom(parentId, parentTemplate);
+ } else {
+ LOGGER.error("Unable to find/pull parent policy type {}", parentId);
+ parentId = null;
}
}
}
- LOGGER.info("isMatchable false for {}", propertyName);
- return false;
+ LOGGER.warn("Property {} is NOT found in any template", propertyName);
+ return null;
}
- private boolean isListOrMap(String type) {
- return "list".equalsIgnoreCase(type) || "map".equalsIgnoreCase(type);
+ private ToscaPolicyType getToscaPolicyTypeFromTemplate(ToscaPolicyTypeIdentifier propertiesPolicyId,
+ ToscaServiceTemplate policyTemplate) {
+ for (Entry<String, ToscaPolicyType> entry : policyTemplate.getPolicyTypes().entrySet()) {
+ if (propertiesPolicyId.getName().equals(entry.getKey())
+ && propertiesPolicyId.getVersion().equals(entry.getValue().getVersion())) {
+ return entry.getValue();
+ }
+ }
+ return null;
}
private boolean isYamlType(String type) {
@@ -425,21 +640,18 @@ public class StdMatchableTranslator extends StdBaseTranslator {
}
/**
- * checkIsMatchableProperty - checks the policy contents for matchable field. If the metadata doesn't exist,
- * then definitely not. If the property doesn't exist, then definitely not. Otherwise need to have a metadata
- * section with the matchable property set to true.
+ * checkIsMatchableProperty - checks the property metadata to see if matchable exists.
*
- * @param propertyName String value of property
- * @param propertiesEntry Section of the TOSCA Policy Type where properties and metadata sections are held
+ * @param toscaProperty ToscaProperty
* @return true if matchable
*/
- protected boolean checkIsMatchableProperty(String propertyName, Entry<String, ToscaProperty> propertiesEntry) {
- if (! propertiesEntry.getKey().equals(propertyName)
- || propertiesEntry.getValue().getMetadata() == null) {
+ protected boolean checkIsMatchableProperty(ToscaProperty toscaProperty) {
+ if (toscaProperty.getMetadata() == null) {
return false;
}
- for (Entry<String, String> entrySet : propertiesEntry.getValue().getMetadata().entrySet()) {
+ for (Entry<String, String> entrySet : toscaProperty.getMetadata().entrySet()) {
if ("matchable".equals(entrySet.getKey()) && "true".equals(entrySet.getValue())) {
+ LOGGER.debug("found matchable of type {}", toscaProperty.getType());
return true;
}
}
@@ -454,12 +666,43 @@ public class StdMatchableTranslator extends StdBaseTranslator {
* @param targetType TargetType object to add matches to
* @param key Property key
* @param value Object is the value - which can be a Collection or single Object
+ * @param toscaProperty The property that was found
+ * @param toscaServiceTemplate The template from which the property was found
* @return int Weight of the match.
*/
- @SuppressWarnings("unchecked")
- protected int generateMatchable(TargetType targetType, String key, Object value) {
+ protected int generateMatchable(TargetType targetType, String key, Object value, ToscaProperty toscaProperty,
+ ToscaServiceTemplate toscaServiceTemplate) {
int weight = 0;
if (value instanceof Collection) {
+ //
+ // Further determine how we treat this collection. We will need the schema
+ // if it is not available then we have to bail.
+ //
+ if (toscaProperty.getEntrySchema() == null) {
+ LOGGER.error("No schema for property {} of type {}", key, toscaProperty.getType());
+ }
+ if ("list".equals(toscaProperty.getType())) {
+ return generateMatchableList(targetType, key, value, toscaProperty, toscaServiceTemplate);
+ }
+ if ("map".equals(toscaProperty.getType())) {
+ return generateMatchableMap(targetType, key, value, toscaProperty, toscaServiceTemplate);
+ }
+ } else {
+ AnyOfType anyOf = generateMatches(Arrays.asList(value),
+ new IdentifierImpl(ToscaDictionary.ID_RESOURCE_MATCHABLE + key));
+ if (! anyOf.getAllOf().isEmpty()) {
+ targetType.getAnyOf().add(anyOf);
+ weight = 1;
+ }
+ }
+ return weight;
+ }
+
+ @SuppressWarnings("unchecked")
+ protected int generateMatchableList(TargetType targetType, String key, Object value, ToscaProperty toscaProperty,
+ ToscaServiceTemplate toscaServiceTemplate) {
+ int weight = 0;
+ if (isYamlType(toscaProperty.getEntrySchema().getType())) {
AnyOfType anyOf = generateMatches((Collection<Object>) value,
new IdentifierImpl(ToscaDictionary.ID_RESOURCE_MATCHABLE + key));
if (! anyOf.getAllOf().isEmpty()) {
@@ -467,12 +710,27 @@ public class StdMatchableTranslator extends StdBaseTranslator {
weight = 1;
}
} else {
- AnyOfType anyOf = generateMatches(Arrays.asList(value),
+ LOGGER.debug("PLD use datatype for list?");
+ }
+ return weight;
+ }
+
+ @SuppressWarnings("unchecked")
+ protected int generateMatchableMap(TargetType targetType, String key, Object value, ToscaProperty toscaProperty,
+ ToscaServiceTemplate toscaServiceTemplate) {
+ int weight = 0;
+ if (isYamlType(toscaProperty.getEntrySchema().getType())) {
+ //
+ // PLD TODO - this won't work
+ //
+ AnyOfType anyOf = generateMatches((Collection<Object>) value,
new IdentifierImpl(ToscaDictionary.ID_RESOURCE_MATCHABLE + key));
if (! anyOf.getAllOf().isEmpty()) {
targetType.getAnyOf().add(anyOf);
weight = 1;
}
+ } else {
+ LOGGER.debug("PLD use datatype for map?");
}
return weight;
}
@@ -530,73 +788,29 @@ public class StdMatchableTranslator extends StdBaseTranslator {
return anyOf;
}
- /**
- * Get Policy Type definitions. This could be previously loaded, or could be
- * stored in application path, or may need to be pulled from the API.
- *
- *
- * @param policyTypeId Policy Type Id
- * @return A list of PolicyTypes
- */
- private List<ToscaPolicyType> getPolicyTypes(ToscaPolicyTypeIdentifier policyTypeId) {
- //
- // Create identifier from the policy
- //
- ToscaPolicyTypeIdentifier typeId = new ToscaPolicyTypeIdentifier(policyTypeId);
- //
- // Find the Policy Type
- //
- ToscaPolicyType policyType = findPolicyType(typeId);
- if (policyType == null) {
- return Collections.emptyList();
+ private ToscaPolicyTypeIdentifier getParentDerivedFrom(ToscaPolicyTypeIdentifier policyTypeId,
+ ToscaServiceTemplate template) {
+ for (Entry<String, ToscaPolicyType> entrySet : template.getPolicyTypes().entrySet()) {
+ ToscaPolicyType policyType = entrySet.getValue();
+ if (entrySet.getKey().equals(policyTypeId.getName())
+ && policyType.getVersion().equals(policyTypeId.getVersion())
+ && ! "tosca.policies.Root".equals(policyType.getDerivedFrom())) {
+ return new ToscaPolicyTypeIdentifier(policyType.getDerivedFrom(), "1.0.0");
+ }
}
- //
- // Create our return object
- //
- List<ToscaPolicyType> listTypes = new ArrayList<>();
- listTypes.add(policyType);
- //
- // Look for parent policy types that could also contain matchable properties
- //
- ToscaPolicyType childPolicyType = policyType;
- while (! childPolicyType.getDerivedFrom().startsWith("tosca.policies.Root")) {
- //
- // Create parent policy type id.
- //
- // We will have to assume the same version between child and the
- // parent policy type it derives from.
- //
- // Or do we assume 1.0.0?
- //
- ToscaPolicyTypeIdentifier parentId = new ToscaPolicyTypeIdentifier(childPolicyType.getDerivedFrom(),
- "1.0.0");
- //
- // Find the policy type
- //
- ToscaPolicyType parentPolicyType = findPolicyType(parentId);
- if (parentPolicyType == null) {
- //
- // Probably would be best to throw an exception and
- // return nothing back.
- //
- // But instead we will log a warning
- //
- LOGGER.warn("Missing parent policy type - proceeding anyway {}", parentId);
- //
- // Break the loop
- //
- break;
+
+ return null;
+ }
+
+ private ToscaPolicyType getParentPolicyType(ToscaPolicyTypeIdentifier policyTypeId, ToscaServiceTemplate template) {
+ for (Entry<String, ToscaPolicyType> entrySet : template.getPolicyTypes().entrySet()) {
+ ToscaPolicyType policyType = entrySet.getValue();
+ if (entrySet.getKey().equals(policyTypeId.getName())
+ && policyType.getVersion().equals(policyTypeId.getVersion())) {
+ return policyType;
}
- //
- // Great save it
- //
- listTypes.add(parentPolicyType);
- //
- // Move to the next parent
- //
- childPolicyType = parentPolicyType;
}
- return listTypes;
+ return null;
}
/**
@@ -607,21 +821,27 @@ public class StdMatchableTranslator extends StdBaseTranslator {
* @param policyTypeId ToscaPolicyTypeIdentifier to find
* @return ToscaPolicyType object. Can be null if failure.
*/
- private ToscaPolicyType findPolicyType(ToscaPolicyTypeIdentifier policyTypeId) {
+ private ToscaServiceTemplate findPolicyType(ToscaPolicyTypeIdentifier policyTypeId) {
//
// Is it loaded in memory?
//
- ToscaPolicyType policyType = this.matchablePolicyTypes.get(policyTypeId);
- if (policyType == null) {
+ ToscaServiceTemplate policyTemplate = this.matchablePolicyTypes.get(policyTypeId);
+ if (policyTemplate == null) {
//
// Load the policy
//
- policyType = this.loadPolicyType(policyTypeId);
+ policyTemplate = this.loadPolicyType(policyTypeId);
+ //
+ // Save it
+ //
+ if (policyTemplate != null) {
+ this.matchablePolicyTypes.put(policyTypeId, policyTemplate);
+ }
}
//
// Yep return it
//
- return policyType;
+ return policyTemplate;
}
/**
@@ -632,7 +852,7 @@ public class StdMatchableTranslator extends StdBaseTranslator {
* @param policyTypeId ToscaPolicyTypeIdentifier input
* @return ToscaPolicyType object. Null if failure.
*/
- private ToscaPolicyType loadPolicyType(ToscaPolicyTypeIdentifier policyTypeId) {
+ private ToscaServiceTemplate loadPolicyType(ToscaPolicyTypeIdentifier policyTypeId) {
//
// Construct what the file name should be
//
@@ -662,34 +882,8 @@ public class StdMatchableTranslator extends StdBaseTranslator {
//
LOGGER.info("Read in local policy type {}", policyTypePath.toAbsolutePath());
try {
- ToscaServiceTemplate serviceTemplate = standardYamlCoder.decode(new String(bytes, StandardCharsets.UTF_8),
+ return standardYamlCoder.decode(new String(bytes, StandardCharsets.UTF_8),
ToscaServiceTemplate.class);
- JpaToscaServiceTemplate jtst = new JpaToscaServiceTemplate();
- jtst.fromAuthorative(serviceTemplate);
- ToscaServiceTemplate completedJtst = jtst.toAuthorative();
- //
- // Search for our Policy Type, there really only should be one but
- // this is returned as a map.
- //
- for ( Entry<String, ToscaPolicyType> entrySet : completedJtst.getPolicyTypes().entrySet()) {
- ToscaPolicyType entryPolicyType = entrySet.getValue();
- if (policyTypeId.getName().equals(entryPolicyType.getName())
- && policyTypeId.getVersion().equals(entryPolicyType.getVersion())) {
- LOGGER.info("Found existing local policy type {} {}", entryPolicyType.getName(),
- entryPolicyType.getVersion());
- //
- // Just simply return the policy type right here
- //
- return entryPolicyType;
- } else {
- LOGGER.warn("local policy type contains different name version {} {}", entryPolicyType.getName(),
- entryPolicyType.getVersion());
- }
- }
- //
- // This would be an error, if the file stored does not match what its supposed to be
- //
- LOGGER.error("Existing policy type file does not contain right name and version");
} catch (CoderException e) {
LOGGER.error("Failed to decode tosca template for {}", policyTypePath, e);
}
@@ -708,15 +902,16 @@ public class StdMatchableTranslator extends StdBaseTranslator {
* @param policyTypePath Path object to store locally
* @return ToscaPolicyType object. Null if failure.
*/
- private synchronized ToscaPolicyType pullPolicyType(ToscaPolicyTypeIdentifier policyTypeId, Path policyTypePath) {
+ private synchronized ToscaServiceTemplate pullPolicyType(ToscaPolicyTypeIdentifier policyTypeId,
+ Path policyTypePath) {
//
// This is what we return
//
- ToscaPolicyType policyType = null;
+ ToscaServiceTemplate policyTemplate = null;
try {
PolicyApiCaller api = new PolicyApiCaller(this.apiRestParameters);
- policyType = api.getPolicyType(policyTypeId);
+ policyTemplate = api.getPolicyType(policyTypeId);
} catch (PolicyApiException e) {
LOGGER.error("Failed to make API call", e);
LOGGER.error("parameters: {} ", this.apiRestParameters);
@@ -727,14 +922,14 @@ public class StdMatchableTranslator extends StdBaseTranslator {
// Store it locally
//
try {
- standardYamlCoder.encode(policyTypePath.toFile(), policyType);
+ standardYamlCoder.encode(policyTypePath.toFile(), policyTemplate);
} catch (CoderException e) {
LOGGER.error("Failed to store {} locally to {}", policyTypeId, policyTypePath, e);
}
//
// Done return the policy type
//
- return policyType;
+ return policyTemplate;
}
/**
diff --git a/applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslatorTest.java b/applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslatorTest.java
index e9e8814c..aeb4cf88 100644
--- a/applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslatorTest.java
+++ b/applications/common/src/test/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslatorTest.java
@@ -1,6 +1,6 @@
/*-
* ============LICENSE_START=======================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -70,7 +70,6 @@ import org.onap.policy.common.utils.resources.ResourceUtils;
import org.onap.policy.models.decisions.concepts.DecisionRequest;
import org.onap.policy.models.decisions.concepts.DecisionResponse;
import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicy;
-import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicyType;
import org.onap.policy.models.tosca.authorative.concepts.ToscaServiceTemplate;
import org.onap.policy.models.tosca.simple.concepts.JpaToscaServiceTemplate;
import org.onap.policy.pdp.xacml.application.common.TestUtilsCommon;
@@ -87,7 +86,7 @@ public class StdMatchableTranslatorTest {
private static final StandardYamlCoder yamlCoder = new StandardYamlCoder();
private static int port;
private static RestServerParameters clientParams;
- private static ToscaPolicyType testPolicyType;
+ private static ToscaServiceTemplate testTemplate;
@ClassRule
public static final TemporaryFolder policyFolder = new TemporaryFolder();
@@ -144,14 +143,15 @@ public class StdMatchableTranslatorTest {
//
JpaToscaServiceTemplate jtst = new JpaToscaServiceTemplate();
jtst.fromAuthorative(serviceTemplate);
- ToscaServiceTemplate completedJtst = jtst.toAuthorative();
+ testTemplate = jtst.toAuthorative();
//
- // Find the Policy Type - SHOULD only be one
+ // Make sure the Policy Types are there
//
- assertEquals(1, completedJtst.getPolicyTypes().size());
- testPolicyType = completedJtst.getPolicyTypes().get("onap.policies.Test");
- assertNotNull(testPolicyType);
- logger.info("Test Policy Type {}{}", XacmlPolicyUtils.LINE_SEPARATOR, testPolicyType);
+ assertEquals(3, testTemplate.getPolicyTypes().size());
+ assertNotNull(testTemplate.getPolicyTypes().get("onap.policies.Base"));
+ assertNotNull(testTemplate.getPolicyTypes().get("onap.policies.base.Middle"));
+ assertNotNull(testTemplate.getPolicyTypes().get("onap.policies.base.middle.Test"));
+ logger.info("Test Policy Type {}{}", XacmlPolicyUtils.LINE_SEPARATOR, testTemplate);
}
@AfterClass
@@ -191,6 +191,9 @@ public class StdMatchableTranslatorTest {
//
for (Map<String, ToscaPolicy> policies : completedJtst.getToscaTopologyTemplate().getPolicies()) {
for (ToscaPolicy policy : policies.values()) {
+ //
+ // Test that we can convert the policy
+ //
PolicyType translatedPolicy = translator.convertPolicy(policy);
assertNotNull(translatedPolicy);
assertThat(translatedPolicy.getObligationExpressions().getObligationExpression()).hasSize(1);
@@ -282,7 +285,7 @@ public class StdMatchableTranslatorTest {
public Response getSpecificVersionOfPolicyType(@PathParam("policyTypeId") String policyTypeId,
@PathParam("versionId") String versionId, @HeaderParam("X-ONAP-RequestID") UUID requestId) {
logger.info("request for policy type={} version={}", policyTypeId, versionId);
- return Response.status(Response.Status.OK).entity(testPolicyType).build();
+ return Response.status(Response.Status.OK).entity(testTemplate).build();
}
}
diff --git a/applications/common/src/test/resources/matchable/onap.policies.Test-1.0.0.yaml b/applications/common/src/test/resources/matchable/onap.policies.Test-1.0.0.yaml
index f44a3061..7179c310 100644
--- a/applications/common/src/test/resources/matchable/onap.policies.Test-1.0.0.yaml
+++ b/applications/common/src/test/resources/matchable/onap.policies.Test-1.0.0.yaml
@@ -1,9 +1,30 @@
tosca_definitions_version: tosca_simple_yaml_1_0_0
policy_types:
- onap.policies.Test:
+ onap.policies.Base:
derived_from: tosca.policies.Root
version: 1.0.0
properties:
+ baseNoMatch:
+ type: string
+ baseMatch:
+ type: string
+ metadata:
+ matchable: true
+ onap.policies.base.Middle:
+ derived_from: onap.policies.Base
+ version: 1.0.0
+ properties:
+ middleNoMatch:
+ type: string
+ middleMatch:
+ type: string
+ metadata:
+ matchable: true
+ onap.policies.base.middle.Test:
+ derived_from: onap.policies.base.Middle
+ type_version: 1.0.0
+ version: 1.0.0
+ properties:
nonmatachableString:
type: string
matchableString:
@@ -30,6 +51,10 @@ policy_types:
type: boolean
metadata:
matchable: true
+ nonmatchableListInteger:
+ type: list
+ entry_schema:
+ type: integer
matchableListString:
type: list
metadata:
@@ -39,7 +64,7 @@ policy_types:
propertyOneMap:
type: map
entry_schema:
- type: onap.datatype.level1
+ type: onap.datatype.one
data_types:
onap.datatype.one:
derived_from: tosca.datatypes.Root
@@ -49,7 +74,7 @@ data_types:
oneStringMatchable:
type: string
metadata:
- matachable: true
+ matchable: true
propertyTwoList:
type: list
entry_schema:
@@ -62,7 +87,7 @@ data_types:
twoStringMatchable:
type: string
metadata:
- matachable: true
+ matchable: true
propertyThreeMap:
type: map
entry_schema:
@@ -75,4 +100,4 @@ data_types:
threeStringMatchable:
type: string
metadata:
- matachable: true \ No newline at end of file
+ matchable: true \ No newline at end of file
diff --git a/applications/common/src/test/resources/matchable/test.policies.input.tosca.yaml b/applications/common/src/test/resources/matchable/test.policies.input.tosca.yaml
index 9a68d488..80f72b2f 100644
--- a/applications/common/src/test/resources/matchable/test.policies.input.tosca.yaml
+++ b/applications/common/src/test/resources/matchable/test.policies.input.tosca.yaml
@@ -2,13 +2,17 @@ tosca_definitions_version: tosca_simple_yaml_1_0_0
topology_template:
policies:
- Test.policy:
- type: onap.policies.Test
+ type: onap.policies.base.middle.Test
type_version: 1.0.0
version: 1.0.0
metadata:
policy-id: Test.policy
policy-version: 1
properties:
+ baseNoMatch: Do not match the base
+ baseMatch: base Match
+ middleNoMatch: Do not match the middle
+ middleMatch: middle Match
nonmatachableString: I am NON matchable
matchableString: I should be matched
nonmatachableInteger: 0
@@ -17,6 +21,7 @@ topology_template:
matchableFloat: 1.1
nonmatachableBoolean: false
matachableBoolean: true
+ nonmatchableListInteger: {0, 1, 2}
matchableListString:
- match A
- match B
@@ -25,7 +30,7 @@ topology_template:
oneStringMatchable: One should be matched
propertyTwoList:
- twoString: Two is NOT matchable
- twoStringMatachable: Two should be matched
+ twoStringMatchable: Two should be matched
propertyThreeMap:
threeString: Three is NOT matchable
threeStringMatchable: Three should be matched \ No newline at end of file
diff --git a/applications/naming/src/main/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplication.java b/applications/naming/src/main/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplication.java
index d04fca6a..5d45bfc5 100644
--- a/applications/naming/src/main/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplication.java
+++ b/applications/naming/src/main/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplication.java
@@ -2,7 +2,7 @@
* ============LICENSE_START=======================================================
* ONAP
* ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -22,14 +22,11 @@
package org.onap.policy.xacml.pdp.application.naming;
-import java.nio.file.Path;
import java.util.Arrays;
import java.util.List;
-import org.onap.policy.common.endpoints.parameters.RestServerParameters;
import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicyTypeIdentifier;
import org.onap.policy.pdp.xacml.application.common.ToscaPolicyTranslator;
-import org.onap.policy.pdp.xacml.application.common.XacmlApplicationException;
-import org.onap.policy.pdp.xacml.application.common.std.StdMatchableTranslator;
+import org.onap.policy.pdp.xacml.application.common.std.StdCombinedPolicyResultsTranslator;
import org.onap.policy.pdp.xacml.application.common.std.StdXacmlApplicationServiceProvider;
public class NamingPdpApplication extends StdXacmlApplicationServiceProvider {
@@ -37,7 +34,7 @@ public class NamingPdpApplication extends StdXacmlApplicationServiceProvider {
private static final ToscaPolicyTypeIdentifier supportedPolicy = new ToscaPolicyTypeIdentifier(
"onap.policies.Naming", "1.0.0");
- private StdMatchableTranslator translator = new StdMatchableTranslator();
+ private StdCombinedPolicyResultsTranslator translator = new StdCombinedPolicyResultsTranslator();
@Override
public String applicationName() {
@@ -50,21 +47,6 @@ public class NamingPdpApplication extends StdXacmlApplicationServiceProvider {
}
@Override
- public void initialize(Path pathForData, RestServerParameters policyApiParameters)
- throws XacmlApplicationException {
- //
- // Store our API parameters and path for translator so it
- // can go get Policy Types
- //
- this.translator.setPathForData(pathForData);
- this.translator.setApiRestParameters(policyApiParameters);
- //
- // Let our super class do its thing
- //
- super.initialize(pathForData, policyApiParameters);
- }
-
- @Override
public synchronized List<ToscaPolicyTypeIdentifier> supportedPolicyTypes() {
return Arrays.asList(supportedPolicy);
}
diff --git a/applications/naming/src/test/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplicationTest.java b/applications/naming/src/test/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplicationTest.java
index 02884b63..0077d0b9 100644
--- a/applications/naming/src/test/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplicationTest.java
+++ b/applications/naming/src/test/java/org/onap/policy/xacml/pdp/application/naming/NamingPdpApplicationTest.java
@@ -2,7 +2,7 @@
* ============LICENSE_START=======================================================
* ONAP
* ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -32,6 +32,7 @@ import java.io.FileNotFoundException;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Paths;
+import java.util.Arrays;
import java.util.Collection;
import java.util.Iterator;
import java.util.Map;
@@ -42,8 +43,10 @@ import org.apache.commons.lang3.tuple.Pair;
import org.assertj.core.api.Condition;
import org.junit.BeforeClass;
import org.junit.ClassRule;
+import org.junit.FixMethodOrder;
import org.junit.Test;
import org.junit.rules.TemporaryFolder;
+import org.junit.runners.MethodSorters;
import org.onap.policy.common.endpoints.parameters.RestServerParameters;
import org.onap.policy.common.utils.coder.CoderException;
import org.onap.policy.common.utils.coder.StandardCoder;
@@ -59,6 +62,7 @@ import org.onap.policy.pdp.xacml.xacmltest.TestUtils;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+@FixMethodOrder(MethodSorters.NAME_ASCENDING)
public class NamingPdpApplicationTest {
private static final Logger LOGGER = LoggerFactory.getLogger(NamingPdpApplicationTest.class);
private static Properties properties = new Properties();
@@ -92,7 +96,6 @@ public class NamingPdpApplicationTest {
//
// Setup our temporary folder
//
- // TODO use lambda policyFolder::newFile
XacmlPolicyUtils.FileCreator myCreator = (String filename) -> policyFolder.newFile(filename);
propertiesFile = XacmlPolicyUtils.copyXacmlPropertiesContents("src/test/resources/xacml.properties",
properties, myCreator);
@@ -102,7 +105,6 @@ public class NamingPdpApplicationTest {
String policy = "onap.policies.Naming";
String policyType = ResourceUtils.getResourceAsString("policytypes/" + policy + ".yaml");
LOGGER.info("Copying {}", policyType);
- // TODO investigate UTF-8
Files.write(Paths.get(policyFolder.getRoot().getAbsolutePath(), policy + "-1.0.0.yaml"),
policyType.getBytes());
//
@@ -188,7 +190,19 @@ public class NamingPdpApplicationTest {
// Ask for a decision for available default policies
//
DecisionResponse response = makeDecision();
-
+ //
+ // There is no default policy
+ //
+ assertThat(response).isNotNull();
+ assertThat(response.getPolicies().size()).isEqualTo(0);
+ //
+ // Ask for VNF
+ //
+ baseRequest.getResource().put("policy-type", Arrays.asList("onap.policies.Naming"));
+ //
+ // Ask for a decision for VNF default policies
+ //
+ response = makeDecision();
assertThat(response).isNotNull();
assertThat(response.getPolicies().size()).isEqualTo(1);
//