Sonar:Access management wfenginemgr

common/RestClient.java
file name:RestClient.java

Change-Id: Idabb6ed057ab678178e11092eaf3d307c4f51dc7
Issue-ID: VFC-857
Signed-off-by: ajay priyadarshi <ajay.priyadarshi@ril.com>

2 files changed, 19 insertions, 19 deletions

diff --git a/wfenginemgrservice/src/test/java/org/onap/workflow/externalservice/service/activitiservice/ActivitiServiceConsumerTest.java b/wfenginemgrservice/src/test/java/org/onap/workflow/externalservice/service/activitiservice/ActivitiServiceConsumerTest.java
index af90c41..263cd30 100644
--- a/wfenginemgrservice/src/test/java/org/onap/workflow/externalservice/service/activitiservice/ActivitiServiceConsumerTest.java
+++ b/wfenginemgrservice/src/test/java/org/onap/workflow/externalservice/service/activitiservice/ActivitiServiceConsumerTest.java
@@ -37,32 +37,32 @@ import org.onap.workflow.externalservice.entity.activitientitiy.ActivitiStartPro
 public class ActivitiServiceConsumerTest {

   @Test

   public void undeploybpmnfile() throws ClientProtocolException, IOException {

-    RestClient.isTest = true;

+    RestClient.setIsTest(true);

     RestResponse response = ActivitiServiceConsumer.undeploybpmnfile("22");

     Assert.assertTrue(response != null);

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

   }

 

   @Test

   public void startBpmnProcess() throws Exception {

-    RestClient.isTest = true;

+    RestClient.setIsTest(true);

     RestResponse response =

         ActivitiServiceConsumer.startBpmnProcess(mock(ActivitiStartProcessRequest.class));

     Assert.assertTrue(response != null);

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

   }

 

   @Test

   public void testDeleteDeployProcess() throws Exception {

-    RestClient.isTest = true;

+    RestClient.setIsTest(true);

     RestResponse response = ActivitiServiceConsumer.deleteDeployProcess("22");

     Assert.assertTrue(response != null);

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

   }

 

   @Test

   public void testDeleteDeployProcessReturnNull() throws Exception {

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

     RestResponse response = ActivitiServiceConsumer.deleteDeployProcess("22");

     if (response != null) {

       Assert.assertTrue(response.getResult() == null);

@@ -73,16 +73,16 @@ public class ActivitiServiceConsumerTest {
 

   @Test

   public void testStartProcessShouldReturnResponse() throws Exception {

-    RestClient.isTest = true;

+    RestClient.setIsTest(true);

     RestResponse response =

         ActivitiServiceConsumer.startProcess(mock(ActivitiStartProcessRequest.class));

     Assert.assertTrue(response != null);

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

   }

 

   @Test

   public void testStartProcessShouldReturnNull() {

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

     RestResponse response =

         ActivitiServiceConsumer.startProcess(mock(ActivitiStartProcessRequest.class));

     Assert.assertTrue(response.getResult() == null);

@@ -90,7 +90,7 @@ public class ActivitiServiceConsumerTest {
 

   @Test

   public void deploybpmnfile() throws Exception {

-    RestClient.isTest = true;

+    RestClient.setIsTest(true);

     WorkflowAppConfig workflowAppConfig = new WorkflowAppConfig();

     MsbClientConfig msbClientConfig = new MsbClientConfig();

     msbClientConfig.setMsbSvrPort(2);

@@ -102,7 +102,7 @@ public class ActivitiServiceConsumerTest {
     when(ins.read(Mockito.anyObject())).thenReturn(-21);

     ActivitiDeployResponse result = ActivitiServiceConsumer.deploybpmnfile(ins, "result");

     Assert.assertTrue(result == null);

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

   }

 

-}
\ No newline at end of file
+}

diff --git a/wfenginemgrservice/src/test/java/org/onap/workflow/resources/WorkflowInstanceWrapperTest.java b/wfenginemgrservice/src/test/java/org/onap/workflow/resources/WorkflowInstanceWrapperTest.java
index 487051f..b2253d6 100644
--- a/wfenginemgrservice/src/test/java/org/onap/workflow/resources/WorkflowInstanceWrapperTest.java
+++ b/wfenginemgrservice/src/test/java/org/onap/workflow/resources/WorkflowInstanceWrapperTest.java
@@ -44,36 +44,36 @@ public class WorkflowInstanceWrapperTest {
 

   @Test

   public final void testDeployBpmnFile() throws Exception {

-    RestClient.isTest = true;

+    RestClient.setIsTest(true);

     InputStream fileInputStream = mock(InputStream.class);

     FormDataContentDisposition f = mock(FormDataContentDisposition.class);

     String filename = "test";

     when(fileInputStream.read(Mockito.anyObject())).thenReturn(-21);

     Response reponse = workflowInstanceWrapper.deployBpmnFile(filename, fileInputStream, f);

     Assert.assertEquals(Response.Status.INTERNAL_SERVER_ERROR.getStatusCode(), reponse.getStatus());

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

   }

 

   @Test

   public final void testUndeployBpmnFile() throws Exception {

-    RestClient.isTest = true;

+    RestClient.setIsTest(true);

     RestResponse response = WorkflowInstanceWrapper.getInstance().undeployBpmnFile(null);

     if (response != null) {

       Assert.assertTrue(response.getResult() == null);

     }

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

   }

 

   @Test

   public final void testStartProcess() throws Exception {

-    RestClient.isTest = true;

+    RestClient.setIsTest(true);

     ActivitiStartProcessRequest startProcessRequest = new ActivitiStartProcessRequest();

     startProcessRequest.setProcessDefinitionKey("first");

     StartProcessRequest tt = new StartProcessRequest();

     tt.setProcessDefinitionKey("first");

     Response reponse = workflowInstanceWrapper.startProcess(tt);

     Assert.assertEquals(Response.Status.OK.getStatusCode(), reponse.getStatus());

-    RestClient.isTest = false;

+    RestClient.setIsTest(false);

   }

 

 }