Fixes for Backward compatibility issues 57/7357/1
authorTej, Tarun <tt3868@att.com>
Fri, 11 Aug 2017 16:20:16 +0000 (12:20 -0400)
committerTarun Tej Velaga <tt3868@att.com>
Fri, 11 Aug 2017 16:37:25 +0000 (16:37 +0000)
Changes to the API interface and Java SDK to support backward compatibility.

Issue-Id: POLICY-165
Change-Id: I2efe8196614cfbd8b733abceb505bd6b91aa97f7
Signed-off-by: Tarun Tej Velaga <tt3868@att.com>
12 files changed:
ONAP-PDP-REST/src/main/java/org/onap/policy/pdp/rest/api/models/ConfigPolicyAPIRequest.java
ONAP-PDP-REST/src/main/java/org/onap/policy/pdp/rest/api/services/GetConfigService.java
ONAP-PDP-REST/src/main/java/org/onap/policy/pdp/rest/api/services/GetDecisionService.java
ONAP-PDP-REST/src/test/java/org/onap/policy/pdp/rest/api/services/PDPServicesTest.java
ONAP-PDP-REST/src/test/java/org/onap/policy/pdp/rest/api/test/PolicyEngineServicesTest.java
PolicyEngineAPI/src/main/java/org/onap/policy/api/ConfigRequestParameters.java
PolicyEngineAPI/src/main/java/org/onap/policy/api/DecisionRequestParameters.java
PolicyEngineAPI/src/main/java/org/onap/policy/api/PolicyConfigType.java
PolicyEngineAPI/src/main/java/org/onap/policy/api/PolicyEngine.java
PolicyEngineAPI/src/main/java/org/onap/policy/api/PolicyParameters.java
PolicyEngineAPI/src/main/java/org/onap/policy/std/StdPolicyEngine.java
PolicyEngineAPI/src/test/java/org/onap/policy/test/DecisionRequestParametersTest.java

index dd40bfe..e7cb407 100644 (file)
  */
 package org.onap.policy.pdp.rest.api.models;
 
-import io.swagger.annotations.ApiModel;
-
 import java.io.Serializable;
 import java.util.Map;
 
+import io.swagger.annotations.ApiModel;
+
 @ApiModel
 public class ConfigPolicyAPIRequest implements Serializable{
     private static final long serialVersionUID = -4103391389984557025L;
@@ -65,6 +65,13 @@ public class ConfigPolicyAPIRequest implements Serializable{
     public String getOnapName() {
         return onapName;
     }
+    /**
+     * @return the onapName
+     */
+    @Deprecated
+    public String getEcompName() {
+        return onapName;
+    }
     /**
      * @return the configName
      */
@@ -113,6 +120,13 @@ public class ConfigPolicyAPIRequest implements Serializable{
     public void setOnapName(String onapName) {
         this.onapName = onapName;
     }
+    /**
+     * @param ecompName the onapName to set
+     */
+    @Deprecated
+    public void setEcompName(String ecompName) {
+        this.onapName = ecompName;
+    }
     /**
      * @param configName the configName to set
      */
index 1817d88..58d5c8f 100644 (file)
@@ -302,7 +302,7 @@ public class GetConfigService {
             return false;
         }
         if(configRequestParameters.getOnapName()==null && configRequestParameters.getPolicyName()==null){
-            message = XACMLErrorConstants.ERROR_DATA_ISSUE + "Cannot proceed without onapComponentName or PolicyName";
+            message = XACMLErrorConstants.ERROR_DATA_ISSUE + "Cannot proceed without onapName or PolicyName";
             return false;
         }
         return true;
index 66beb06..a74203c 100644 (file)
@@ -164,7 +164,7 @@ public class GetDecisionService {
             message = XACMLErrorConstants.ERROR_DATA_ISSUE + "No Decision Request Paramaters";
             return false; 
         }
-        onapComponentName = decisionRequestParameters.getONAPComponentName();
+        onapComponentName = decisionRequestParameters.getOnapName();
         decisionAttributes = decisionRequestParameters.getDecisionAttributes();
         if (onapComponentName == null || onapComponentName.isEmpty()) {
             message = XACMLErrorConstants.ERROR_DATA_ISSUE + "No onapComponentName given : " + onapComponentName;
index e77a851..4e8a0fa 100644 (file)
@@ -73,7 +73,7 @@ public class PDPServicesTest {
                DecisionRequestParameters pep = new DecisionRequestParameters();
                Map<String,String> eventAttributes = new HashMap<>();
                eventAttributes.put("TEST", "test");
-               pep.setONAPComponentName("te123");
+               pep.setOnapName("te123");
                pep.setDecisionAttributes(eventAttributes);
                PDPServices fixture = new PDPServices();
 
@@ -119,7 +119,7 @@ public class PDPServicesTest {
                 .add("Request", Json.createObjectBuilder()
                                 .add("AccessSubject", Json.createObjectBuilder()
                                                 .add("Attribute", Json.createObjectBuilder()
-                                                                .add("Value", pep.getONAPComponentName())
+                                                                .add("Value", pep.getOnapName())
                                                                 .add("AttributeId", "ONAPName")))
                                 .add("Resource", Json.createObjectBuilder()
                                                 .add("Attribute", resourceArray))
index 2a76f58..aa55a2a 100644 (file)
@@ -236,7 +236,7 @@ public class PolicyEngineServicesTest {
                DecisionRequestParameters pep = new DecisionRequestParameters();
                Map<String,String> eventAttributes = new HashMap<>();
                eventAttributes.put("TEST", "test");
-               pep.setONAPComponentName("te123");
+               pep.setOnapName("te123");
                pep.setDecisionAttributes(eventAttributes);
                //Failure Tests. 
                mockMvc.perform(post("/getDecision")).andExpect(status().isBadRequest());
@@ -245,10 +245,10 @@ public class PolicyEngineServicesTest {
                mockMvc.perform(post("/getDecision").content(PolicyUtils.objectToJsonString(pep)).contentType(MediaType.APPLICATION_JSON)
                                .headers(headers).header(UUIDHEADER, "123")).andExpect(status().isOk());
                pep.setDecisionAttributes(null);
-               pep.setONAPComponentName(null);
+               pep.setOnapName(null);
                mockMvc.perform(post("/getDecision").content(PolicyUtils.objectToJsonString(pep)).contentType(MediaType.APPLICATION_JSON)
                                .headers(headers).header(UUIDHEADER, UUID.randomUUID())).andExpect(status().isBadRequest());
-               pep.setONAPComponentName("testing");
+               pep.setOnapName("testing");
                mockMvc.perform(post("/getDecision").content(PolicyUtils.objectToJsonString(pep)).contentType(MediaType.APPLICATION_JSON)
                                .headers(headers)).andExpect(status().isBadRequest());
                pep.setDecisionAttributes(eventAttributes);
index cdee967..dcdc178 100644 (file)
@@ -31,7 +31,7 @@ import java.util.UUID;
  */
 public class ConfigRequestParameters {
        private String policyName;
-       private String onapComponentName;
+       private String onapName;
        private String configName;
        private Map<String,String> configAttributes;
        private UUID requestID;
@@ -49,12 +49,23 @@ public class ConfigRequestParameters {
        /**
         * Sets the ONAP Component Name of the Config policy which needs to be retrieved. 
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose configuration is required.
+        * @param onapName the <code>String</code> format of the onapName whose configuration is required.
         */
-       public void setOnapName(String onapComponentName){
-               this.onapComponentName = onapComponentName;
+       public void setOnapName(String onapName){
+               this.onapName = onapName;
        }
        
+       /**
+     * Sets the ONAP Component Name of the Config policy which needs to be retrieved. 
+     * 
+     * @param ecompName the <code>String</code> format of the onapName whose configuration is required.
+     * @deprecated use {@link #setOnapName(String)} instead.  
+     */
+       @Deprecated
+    public void setEcompName(String ecompName){
+        this.onapName = ecompName;
+    }
+       
        /**
         * Sets the Config Name of the Config policy which needs to be retrieved.  
         * 
@@ -97,9 +108,20 @@ public class ConfigRequestParameters {
         * @return <code>String</code> format of the ONAP Component Name. 
         */
        public String getOnapName(){
-               return onapComponentName;
+               return onapName;
        }
        
+       /**
+     * Gets the ONAP Component Name of the Request Parameters. 
+     * 
+     * @return <code>String</code> format of the ONAP Component Name. 
+     * @deprecated use {@link #getOnapName()} instead. 
+     */
+       @Deprecated
+    public String getEcompName(){
+        return onapName;
+    }
+       
        /**
         * Gets the Config name of the Request Parameters. 
         * 
index 27e419f..5cf9d64 100644 (file)
@@ -30,7 +30,7 @@ import java.util.UUID;
  * @version 0.1
  */
 public class DecisionRequestParameters {
-       private String onapComponentName;
+       private String onapName;
        private Map<String,String> decisionAttributes;
        private UUID requestID;
        
@@ -43,32 +43,56 @@ public class DecisionRequestParameters {
        /**
         * Constructor with Parameters
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose Decision is required.
+        * @param onapName the <code>String</code> format of the onapName whose Decision is required.
         * @param decisionAttributes the <code>Map</code> of <code>String,String</code> format of the decisionAttributes that contain the ID and values.
         * @param requestID unique request ID which will be passed throughout the ONAP components to correlate logging messages.
         */
-       public DecisionRequestParameters(String onapComponentName, Map<String,String> decisionAttributes, UUID requestID){
-               this.onapComponentName = onapComponentName;
+       public DecisionRequestParameters(String onapName, Map<String,String> decisionAttributes, UUID requestID){
+               this.onapName = onapName;
                this.decisionAttributes = decisionAttributes;
                this.requestID = requestID;
        }
        
        /**
-        * Gets the ONAPComponentName of the Decision Request Parameters. 
+        * Gets the onapName of the Decision Request Parameters. 
         * 
-        * @return ONAPComponentName the <code>String</code> format of the onapComponentName of the Decision Request Parameters.
+        * @return onapName the <code>String</code> format of the onapName of the Decision Request Parameters.
         */
-       public String getONAPComponentName() {
-               return onapComponentName;
+       public String getOnapName() {
+               return onapName;
        }
+       
+       /**
+     * Gets the onapName of the Decision Request Parameters. 
+     * 
+     * @return onapName the <code>String</code> format of the onapName of the Decision Request Parameters.
+     * @deprecated use {@link #getOnapName()} instead. 
+     */
+       @Deprecated
+    public String getECOMPComponentName() {
+        return onapName;
+    }
+       
        /**
-        * Sets the ONAPComponentName of the Decision Request parameters.  
+        * Sets the onapName of the Decision Request parameters.  
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose Decision is required.
+        * @param onapName the <code>String</code> format of the onapName whose Decision is required.
         */
-       public void setONAPComponentName(String onapComponentName) {
-               this.onapComponentName = onapComponentName;
+       public void setOnapName(String onapName) {
+               this.onapName = onapName;
        }
+       
+       /**
+     * Sets the ecompComponentName of the Decision Request parameters.  
+     * 
+     * @param ecompName the <code>String</code> format of the onapName whose Decision is required.
+     * @deprecated use {@link #setOnapName(String)} instead. 
+     */
+       @Deprecated
+    public void setECOMPComponentName(String ecompName) {
+        this.onapName = ecompName;
+    }
+       
        /**
         * Gets the Decision Attributes from Decision Request Parameters. 
         * 
index 77c978d..59fd427 100644 (file)
@@ -57,7 +57,9 @@ public enum PolicyConfigType{
         * Indicates DCAE Micro Service based Policy. 
         */
        MicroService("MS"),
-       
+       /**
+        * Indicates Custom Extended Policy type. 
+        */
        Extended("EXTENDED")
        ;
        
index bfb4f1e..7bd4598 100644 (file)
@@ -121,22 +121,22 @@ public class PolicyEngine{
        }
        
        /**
-        * Gets the configuration from the PolicyDecisionPoint(PDP) for the <code>String</code> which represents the onapComponentName 
+        * Gets the configuration from the PolicyDecisionPoint(PDP) for the <code>String</code> which represents the onapName 
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose configuration is required. 
+        * @param onapName the <code>String</code> format of the onapName whose configuration is required. 
         * @return <code>Collection</code> of {@link org.onap.policy.api.PolicyConfig} which has the configuration.
         * @throws PolicyConfigException PolicyConfig Exception
         * @deprecated use {@link #getConfig(ConfigRequestParameters)} Instead.
         */
        @Deprecated
-       public Collection<PolicyConfig> getConfig(String onapComponentName) throws PolicyConfigException {
-               return getConfig(setConfigRequestParameters(null, onapComponentName, null, null, null));
+       public Collection<PolicyConfig> getConfig(String onapName) throws PolicyConfigException {
+               return getConfig(setConfigRequestParameters(null, onapName, null, null, null));
        }
        
        /**
-        * Gets the configuration from the PolicyDecisionPoint(PDP) for the <code>String</code> which represents the onapComponentName 
+        * Gets the configuration from the PolicyDecisionPoint(PDP) for the <code>String</code> which represents the onapName 
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose configuration is required. 
+        * @param onapName the <code>String</code> format of the onapName whose configuration is required. 
         * @return <code>Collection</code> of {@link org.onap.policy.api.PolicyConfig} which has the configuration.
         * @param requestID unique request ID which will be passed throughout the ONAP components to correlate logging messages.
         * A different request ID should be passed for each request.
@@ -144,32 +144,32 @@ public class PolicyEngine{
         * @deprecated use {@link #getConfig(ConfigRequestParameters)} Instead.
         */
        @Deprecated
-       public Collection<PolicyConfig> getConfig(String onapComponentName, UUID requestID) throws PolicyConfigException {
-               return getConfig(setConfigRequestParameters(null, onapComponentName, null, null, requestID));
+       public Collection<PolicyConfig> getConfig(String onapName, UUID requestID) throws PolicyConfigException {
+               return getConfig(setConfigRequestParameters(null, onapName, null, null, requestID));
        }
        
        /**
-        * Requests the configuration of the <code>String</code> which represents the onapComponentName and <code>String</code>
+        * Requests the configuration of the <code>String</code> which represents the onapName and <code>String</code>
         * which represents the configName and returns the configuration if different Configurations exist for the
-        * particular onapComponentName.  
+        * particular onapName.  
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose configuration is required.
+        * @param onapName the <code>String</code> format of the onapName whose configuration is required.
         * @param configName the <code>String</code> format of the configurationName whose configuration is required.
         * @return <code>Collection</code> of {@link org.onap.policy.api.PolicyConfig} which has the configuration. 
         * @throws PolicyConfigException PolicyConfig Exception
         * @deprecated use {@link #getConfig(ConfigRequestParameters)} Instead.
         */
        @Deprecated
-       public Collection<PolicyConfig> getConfig(String onapComponentName, String configName) throws PolicyConfigException {
-               return getConfig(setConfigRequestParameters(null, onapComponentName, configName, null, null));
+       public Collection<PolicyConfig> getConfig(String onapName, String configName) throws PolicyConfigException {
+               return getConfig(setConfigRequestParameters(null, onapName, configName, null, null));
        }
        
        /**
-        * Requests the configuration of the <code>String</code> which represents the onapComponentName and <code>String</code>
+        * Requests the configuration of the <code>String</code> which represents the onapName and <code>String</code>
         * which represents the configName and returns the configuration if different Configurations exist for the
-        * particular onapComponentName.  
+        * particular onapName.  
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose configuration is required.
+        * @param onapName the <code>String</code> format of the onapName whose configuration is required.
         * @param configName the <code>String</code> format of the configurationName whose configuration is required.
         * @param requestID unique request ID which will be passed throughout the ONAP components to correlate logging messages.
         * A different request ID should be passed for each request.
@@ -178,16 +178,16 @@ public class PolicyEngine{
         * @deprecated use {@link #getConfig(ConfigRequestParameters)} Instead.
         */
        @Deprecated
-       public Collection<PolicyConfig> getConfig(String onapComponentName, String configName, UUID requestID) throws PolicyConfigException {
-               return getConfig(setConfigRequestParameters(null, onapComponentName, configName, null, requestID));
+       public Collection<PolicyConfig> getConfig(String onapName, String configName, UUID requestID) throws PolicyConfigException {
+               return getConfig(setConfigRequestParameters(null, onapName, configName, null, requestID));
        }
        
        /**
-        * Requests the configuration of the <code>String</code> which represents the onapComponentName, <code>String</code>
+        * Requests the configuration of the <code>String</code> which represents the onapName, <code>String</code>
         * which represents the configName and <code>Map</code> of <code>String,String</code> which has the configAttribute and returns the specific 
         * configuration related to the configAttributes mentioned.
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose configuration is required.
+        * @param onapName the <code>String</code> format of the onapName whose configuration is required.
         * @param configName the <code>String</code> format of the configurationName whose configuration is required.
         * @param configAttributes the <code>Map</code> of <code>String,String</code> format of the configuration attributes which are required.
         * @return <code>Collection</code> of {@link org.onap.policy.api.PolicyConfig} which has the configuration.
@@ -195,16 +195,16 @@ public class PolicyEngine{
         * @deprecated use {@link #getConfig(ConfigRequestParameters)} Instead.
         */
        @Deprecated
-       public Collection<PolicyConfig> getConfig(String onapComponentName, String configName, Map<String, String> configAttributes) throws PolicyConfigException{
-               return getConfig(setConfigRequestParameters(null, onapComponentName, configName, configAttributes, null));
+       public Collection<PolicyConfig> getConfig(String onapName, String configName, Map<String, String> configAttributes) throws PolicyConfigException{
+               return getConfig(setConfigRequestParameters(null, onapName, configName, configAttributes, null));
        }
        
        /**
-        * Requests the configuration of the <code>String</code> which represents the onapComponentName, <code>String</code>
+        * Requests the configuration of the <code>String</code> which represents the onapName, <code>String</code>
         * which represents the configName and <code>Map</code> of <code>String,String</code> which has the configAttribute and returns the specific 
         * configuration related to the configAttributes mentioned.
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose configuration is required.
+        * @param onapName the <code>String</code> format of the onapName whose configuration is required.
         * @param configName the <code>String</code> format of the configurationName whose configuration is required.
         * @param configAttributes the <code>Map</code> of <code>String,String</code> format of the configuration attributes which are required.
         * @param requestID unique request ID which will be passed throughout the ONAP components to correlate logging messages.
@@ -214,8 +214,8 @@ public class PolicyEngine{
         * @deprecated use {@link #getConfig(ConfigRequestParameters)} Instead.
         */
        @Deprecated
-       public Collection<PolicyConfig> getConfig(String onapComponentName, String configName, Map<String, String> configAttributes, UUID requestID) throws PolicyConfigException{
-               return getConfig(setConfigRequestParameters(null, onapComponentName, configName, configAttributes, requestID));
+       public Collection<PolicyConfig> getConfig(String onapName, String configName, Map<String, String> configAttributes, UUID requestID) throws PolicyConfigException{
+               return getConfig(setConfigRequestParameters(null, onapName, configName, configAttributes, requestID));
        }
        
        /**
@@ -285,21 +285,21 @@ public class PolicyEngine{
        /**
         * Sends the decision Attributes specified to the PEP and returns back the PolicyDecision. 
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose Decision is required.
+        * @param onapName the <code>String</code> format of the onapName whose Decision is required.
         * @param decisionAttributes the <code>Map</code> of <code>String,String</code> format of the decisionAttributes that must contain the ID and values.
         * @return {@link org.onap.policy.api.DecisionResponse} which has the Decision. 
         * @throws PolicyDecisionException PolicyDecision Exception
         * @deprecated use {@link #getDecision(DecisionRequestParameters)} Instead.
         */
        @Deprecated
-       public DecisionResponse getDecision(String onapComponentName, Map<String,String> decisionAttributes) throws PolicyDecisionException {
-               return stdPolicyEngine.getDecision(onapComponentName, decisionAttributes, null);
+       public DecisionResponse getDecision(String onapName, Map<String,String> decisionAttributes) throws PolicyDecisionException {
+               return stdPolicyEngine.getDecision(onapName, decisionAttributes, null);
        }
        
        /**
         * Sends the decision Attributes specified to the PEP and returns back the PolicyDecision. 
         * 
-        * @param onapComponentName the <code>String</code> format of the onapComponentName whose Decision is required.
+        * @param onapName the <code>String</code> format of the onapName whose Decision is required.
         * @param decisionAttributes the <code>Map</code> of <code>String,String</code> format of the decisionAttributes that must contain the ID and values.
         * @param requestID unique request ID which will be passed throughout the ONAP components to correlate logging messages.
         * A different request ID should be passed for each request.
@@ -308,8 +308,8 @@ public class PolicyEngine{
         * @deprecated use {@link #getDecision(DecisionRequestParameters)} Instead.
         */
        @Deprecated
-       public DecisionResponse getDecision(String onapComponentName, Map<String,String> decisionAttributes, UUID requestID) throws PolicyDecisionException {
-               return stdPolicyEngine.getDecision(onapComponentName, decisionAttributes, requestID);
+       public DecisionResponse getDecision(String onapName, Map<String,String> decisionAttributes, UUID requestID) throws PolicyDecisionException {
+               return stdPolicyEngine.getDecision(onapName, decisionAttributes, requestID);
        }
        
        /**
@@ -590,10 +590,10 @@ public class PolicyEngine{
        }
        
        // Internal Setter Method to help build configRequestParameters.
-       private ConfigRequestParameters setConfigRequestParameters(String policyName, String onapComponentName, String configName, Map<String, String> configAttributes, UUID requestID){
+       private ConfigRequestParameters setConfigRequestParameters(String policyName, String onapName, String configName, Map<String, String> configAttributes, UUID requestID){
                ConfigRequestParameters configRequestParameters = new ConfigRequestParameters();
                configRequestParameters.setPolicyName(policyName);
-               configRequestParameters.setOnapName(onapComponentName);
+               configRequestParameters.setOnapName(onapName);
                configRequestParameters.setConfigName(configName);
                configRequestParameters.setConfigAttributes(configAttributes);
                configRequestParameters.setRequestID(requestID);
index 0d76d8f..1078916 100644 (file)
@@ -53,7 +53,7 @@ public class PolicyParameters {
        private RuleProvider ruleProvider;
        private String controllerName;
        private ArrayList<String> dependencyNames;
-       private Date TTLDate;
+       private Date ttlDate;
        private boolean guard = false;
        private String riskLevel = "5";
        private String riskType = "default";
@@ -146,6 +146,17 @@ public class PolicyParameters {
        public String getOnapName() {
                return onapName;
        }
+       
+       /**
+     * Gets the ONAP Name value of the Policy Paramters. 
+     * 
+     * @return <code>String</code> format of the ONAP Name
+     * @deprecated Use {@link #getOnapName()} instead. 
+     */
+       @Deprecated
+    public String getEcompName() {
+        return onapName;
+    }
 
        /**
         * Sets the ONAP Name field of the Policy Parameters. 
@@ -155,6 +166,18 @@ public class PolicyParameters {
        public void setOnapName(String onapName) {
                this.onapName = onapName;
        }
+       
+       /**
+     * Sets the ONAP Name field of the Policy Parameters. 
+     * 
+     * @param ecompName the <code>String</code> format of the ONAP Name
+     * @deprecated use {@link #setOnapName(String)} instead. 
+     */
+       @Deprecated
+    public void setEcompName(String ecompName) {
+        this.onapName = ecompName;
+    }
+    
 
        /**
         * Gets the Config Name value of the Policy Parameters. 
@@ -486,10 +509,10 @@ public class PolicyParameters {
        /**
         * Sets the TTLDate field of the Policy Parameters. 
         * 
-        * @param TTLDate the <code>Date</code> format of the TTLDate value
+        * @param ttlDate the <code>Date</code> format of the TTLDate value
         */
-       public void setTtlDate(Date TTLDate){
-               this.TTLDate = TTLDate;
+       public void setTtlDate(Date ttlDate){
+               this.ttlDate = ttlDate;
        }
        
        /**
@@ -498,7 +521,7 @@ public class PolicyParameters {
         * @return the <code>Date</code> value of the TTLDate for Config Policies
         */
        public Date getTtlDate(){
-               return TTLDate;
+               return ttlDate;
        }
        
        /**
index 608bb8d..8a6c6c6 100644 (file)
@@ -205,8 +205,8 @@ public class StdPolicyEngine {
        /*
         * getDecision using the decision Attributes.
         */
-       public DecisionResponse getDecision(String onapComponentName, Map<String, String> decisionAttributes, UUID requestID) throws PolicyDecisionException {
-               return getDecisionImpl(onapComponentName, decisionAttributes, requestID);
+       public DecisionResponse getDecision(String onapName, Map<String, String> decisionAttributes, UUID requestID) throws PolicyDecisionException {
+               return getDecisionImpl(onapName, decisionAttributes, requestID);
        }
        
        /*
@@ -218,7 +218,7 @@ public class StdPolicyEngine {
                        LOGGER.error(message);
                        throw new PolicyDecisionException(message);
                }
-               return getDecisionImpl(decisionRequestParameters.getONAPComponentName(), decisionRequestParameters.getDecisionAttributes(), decisionRequestParameters.getRequestID());
+               return getDecisionImpl(decisionRequestParameters.getOnapName(), decisionRequestParameters.getDecisionAttributes(), decisionRequestParameters.getRequestID());
        }
        
        /*
@@ -523,7 +523,7 @@ public class StdPolicyEngine {
         }
     }
 
-    public DecisionResponse getDecisionImpl(String onapComponentName,
+    public DecisionResponse getDecisionImpl(String onapName,
                                                                                        Map<String, String> decisionAttributes,
                                                                                        UUID requestID) throws PolicyDecisionException {
                String resource= "getDecision";
@@ -533,7 +533,7 @@ public class StdPolicyEngine {
         try {
             DecisionRequestParameters decisionRequestParameters = new DecisionRequestParameters();
             decisionRequestParameters.setDecisionAttributes(decisionAttributes);
-            decisionRequestParameters.setONAPComponentName(onapComponentName);
+            decisionRequestParameters.setOnapName(onapName);
             decisionRequestParameters.setRequestID(requestID);
             body = PolicyUtils.objectToJsonString(decisionRequestParameters);
         } catch (JsonProcessingException e) {
index 746822f..a29bb13 100644 (file)
@@ -51,7 +51,7 @@ public class DecisionRequestParametersTest {
 
                // add additional test code here
                assertNotNull(result);
-               assertEquals(null, result.getONAPComponentName());
+               assertEquals(null, result.getOnapName());
                assertEquals(null, result.getDecisionAttributes());
                assertEquals(null, result.getRequestID());
        }
@@ -74,7 +74,7 @@ public class DecisionRequestParametersTest {
 
                // add additional test code here
                assertNotNull(result);
-               assertEquals("", result.getONAPComponentName());
+               assertEquals("", result.getOnapName());
        }
 
        /**
@@ -108,7 +108,7 @@ public class DecisionRequestParametersTest {
                throws Exception {
                DecisionRequestParameters fixture = new DecisionRequestParameters("", new Hashtable<String, String>(), UUID.randomUUID());
 
-               String result = fixture.getONAPComponentName();
+               String result = fixture.getOnapName();
 
                // add additional test code here
                assertEquals("", result);
@@ -168,7 +168,7 @@ public class DecisionRequestParametersTest {
                DecisionRequestParameters fixture = new DecisionRequestParameters("", new Hashtable<String, String>(), UUID.randomUUID());
                String onapComponentName = "";
 
-               fixture.setONAPComponentName(onapComponentName);
+               fixture.setOnapName(onapComponentName);
 
                // add additional test code here
        }