SDNHandler callback correlation issue.

SDNHandler callback correlation issue. Should not go back to waiting
for callback when final_indicator=Y is receibed in callback. Raise an
error when responseCode in callback mesage is not 200.

Change-Id: Ia55ab5a9bafdfe877a1221b67d5f3258039a3900
Issue-ID: SO-1761
Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>

2 files changed, 45 insertions, 1 deletions

diff --git a/bpmn/so-bpmn-tasks/src/main/java/org/onap/so/bpmn/infrastructure/sdnc/exceptions/SDNCErrorResponseException.java b/bpmn/so-bpmn-tasks/src/main/java/org/onap/so/bpmn/infrastructure/sdnc/exceptions/SDNCErrorResponseException.java
new file mode 100644
index 0000000000..e0f88a20a5
--- /dev/null
+++ b/bpmn/so-bpmn-tasks/src/main/java/org/onap/so/bpmn/infrastructure/sdnc/exceptions/SDNCErrorResponseException.java
@@ -0,0 +1,13 @@
+package org.onap.so.bpmn.infrastructure.sdnc.exceptions;
+
+public class SDNCErrorResponseException extends Exception {
+
+	/**
+	 * 
+	 */
+	private static final long serialVersionUID = 7807799223298140702L;
+	
+	public SDNCErrorResponseException(String message) {
+		super(message);
+	}
+}
diff --git a/bpmn/so-bpmn-tasks/src/main/java/org/onap/so/bpmn/infrastructure/sdnc/tasks/SDNCRequestTasks.java b/bpmn/so-bpmn-tasks/src/main/java/org/onap/so/bpmn/infrastructure/sdnc/tasks/SDNCRequestTasks.java
index a4ef28496e..24f642ae72 100644
--- a/bpmn/so-bpmn-tasks/src/main/java/org/onap/so/bpmn/infrastructure/sdnc/tasks/SDNCRequestTasks.java
+++ b/bpmn/so-bpmn-tasks/src/main/java/org/onap/so/bpmn/infrastructure/sdnc/tasks/SDNCRequestTasks.java
@@ -20,7 +20,15 @@
 
 package org.onap.so.bpmn.infrastructure.sdnc.tasks;
 
+import java.io.StringReader;
+
+import javax.xml.parsers.DocumentBuilder;
+import javax.xml.parsers.DocumentBuilderFactory;
+import javax.xml.xpath.XPath;
+import javax.xml.xpath.XPathFactory;
+
 import org.camunda.bpm.engine.delegate.DelegateExecution;
+import org.onap.so.bpmn.infrastructure.sdnc.exceptions.SDNCErrorResponseException;
 import org.onap.so.client.exception.BadResponseException;
 import org.onap.so.client.exception.ExceptionBuilder;
 import org.onap.so.client.exception.MapperException;
@@ -31,6 +39,8 @@ import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
 import org.springframework.web.client.HttpClientErrorException;
+import org.w3c.dom.Document;
+import org.xml.sax.InputSource;
 
 import com.jayway.jsonpath.JsonPath;
 import com.jayway.jsonpath.PathNotFoundException;
@@ -44,6 +54,7 @@ public class SDNCRequestTasks {
 	private static final String MESSAGE = "_MESSAGE";
 	private static final String CORRELATOR = "_CORRELATOR";
 	protected static final String IS_CALLBACK_COMPLETED = "isCallbackCompleted";
+	protected static final String SDNC_SUCCESS = "200";
 	
 	@Autowired
 	private ExceptionBuilder exceptionBuilder;
@@ -82,9 +93,24 @@ public class SDNCRequestTasks {
 		try {
 			SDNCRequest request = (SDNCRequest)execution.getVariable(SDNC_REQUEST);
 			String asyncRequest = (String) execution.getVariable(request.getCorrelationName()+MESSAGE);
-			String finalMessageIndicator = JsonPath.read(asyncRequest, "$.input.ack-final-indicator");		
+			
+	        DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance ();
+	        DocumentBuilder db = dbf.newDocumentBuilder();
+	        Document doc = db.parse(new InputSource(new StringReader(asyncRequest)));
+
+	        String finalMessageIndicator = getXmlElement(doc, "/input/ack-final-indicator");
 			boolean isCallbackCompleted = convertIndicatorToBoolean(finalMessageIndicator);
 			execution.setVariable(IS_CALLBACK_COMPLETED, isCallbackCompleted);
+			if(isCallbackCompleted) {
+				String responseCode = getXmlElement(doc, "/input/response-code");
+				String responseMessage = getXmlElement(doc, "/input/response-message"); 
+				if(!SDNC_SUCCESS.equalsIgnoreCase(responseCode)) {
+					throw new SDNCErrorResponseException(responseMessage);
+				}
+			}
+		} catch (SDNCErrorResponseException e) {
+			logger.error("SDNC error response - " + e.getMessage());
+			exceptionBuilder.buildAndThrowWorkflowException(execution, 7000, e.getMessage());
 		} catch (Exception e) {
 			logger.error("Error procesing SDNC callback", e);
 			exceptionBuilder.buildAndThrowWorkflowException(execution, 7000, "Error procesing SDNC callback");
@@ -99,4 +125,9 @@ public class SDNCRequestTasks {
 		return "Y".equals(finalMessageIndicator);
 	}
 	
+	protected String getXmlElement(Document doc, String exp) throws Exception {
+		XPath xPath = XPathFactory.newInstance().newXPath();
+		return xPath.evaluate(exp, doc);
+	}
+	
 }