From 95ff3e1578b1ea0307313d4091f6881b3d05cb59 Mon Sep 17 00:00:00 2001 From: "k.kazak" Date: Thu, 2 May 2019 11:58:21 +0200 Subject: fix newest sonar bugs OperationProgressor: restore interrupted state PnfEventReadyDmaapClient: remove useless check for null that never happens Test for this case already exists in PnfEventReadyDmaapClientTest.pnfCorrelationIdIsNotFoundInHttpResponse Change-Id: Ic7e4d0cb701721ee4d9a8bc46d65dc1d847d7704 Issue-ID: SO-1811 Signed-off-by: k.kazak --- .../so/bpmn/infrastructure/pnf/dmaap/PnfEventReadyDmaapClient.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'bpmn/so-bpmn-infrastructure-common/src/main/java/org/onap') diff --git a/bpmn/so-bpmn-infrastructure-common/src/main/java/org/onap/so/bpmn/infrastructure/pnf/dmaap/PnfEventReadyDmaapClient.java b/bpmn/so-bpmn-infrastructure-common/src/main/java/org/onap/so/bpmn/infrastructure/pnf/dmaap/PnfEventReadyDmaapClient.java index 9f8a456002..357b571a5c 100644 --- a/bpmn/so-bpmn-infrastructure-common/src/main/java/org/onap/so/bpmn/infrastructure/pnf/dmaap/PnfEventReadyDmaapClient.java +++ b/bpmn/so-bpmn-infrastructure-common/src/main/java/org/onap/so/bpmn/infrastructure/pnf/dmaap/PnfEventReadyDmaapClient.java @@ -135,7 +135,8 @@ public class PnfEventReadyDmaapClient implements DmaapClient { HttpResponse response = httpClient.execute(getRequest); List idList = getPnfCorrelationIdListFromResponse(response); - if (idList != null && idList.size() > 0) { + // idList is never null + if (!idList.isEmpty()) { // send only body of response registerClientResponse(idList.get(0), EntityUtils.toString(response.getEntity(), "UTF-8")); } -- cgit 1.2.3-korg