[DMAAP-CLIENT] First sonar issues review part2

[dmaap/messagerouter/dmaapclient.git] / src / test / java / org / onap / dmaap / mr / test / support / MRBatchingPublisherMockTest.java

diff --git a/src/test/java/org/onap/dmaap/mr/test/support/MRBatchingPublisherMockTest.java b/src/test/java/org/onap/dmaap/mr/test/support/MRBatchingPublisherMockTest.java
index af76c52..1de5bcb 100644 (file)
--- a/src/test/java/org/onap/dmaap/mr/test/support/MRBatchingPublisherMockTest.java
+++ b/src/test/java/org/onap/dmaap/mr/test/support/MRBatchingPublisherMockTest.java
@@ -4,12 +4,14 @@
  * ================================================================================
  * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved.
  * ================================================================================
+ * Modifications Copyright © 2021 Orange.
+ * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -20,166 +22,165 @@
 
 package org.onap.dmaap.mr.test.support;
 
-import static org.junit.Assert.assertTrue;
-
-import java.util.ArrayList;
-
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
+import org.onap.dmaap.mr.client.MRPublisher.Message;
 
-import org.onap.dmaap.mr.client.MRPublisher.message;
+import java.util.ArrayList;
+
+import static org.junit.Assert.assertTrue;
 
 public class MRBatchingPublisherMockTest {
-       private MRBatchingPublisherMock pub = null;
+    private MRBatchingPublisherMock pub = null;
+
+    private MRBatchingPublisherMock.Entry entry = null;
+
+    @Before
+    public void setUp() throws Exception {
+        pub = new MRBatchingPublisherMock();
+        entry = pub.new Entry("partition", "msg");
+
+    }
+
+    @After
+    public void tearDown() throws Exception {
+
+    }
+
+    @Test
+    public void testToString() {
+
+        entry.toString();
+        assertTrue(true);
+
+    }
+
+    @Test
+    public void testAddListener() {
+
+        pub.addListener(null);
+        assertTrue(true);
+
+    }
+
+    @Test
+    public void testGetCaptures() {
+
+        pub.getCaptures();
+        assertTrue(true);
+
+    }
 
-       private MRBatchingPublisherMock.Entry entry = null;
+    @Test
+    public void testGetCaptures2() {
 
-       @Before
-       public void setUp() throws Exception {
-               pub = new MRBatchingPublisherMock();
-               entry = pub.new Entry("partition", "msg");
+        pub.getCaptures(null);
+        assertTrue(true);
 
-       }
+    }
 
-       @After
-       public void tearDown() throws Exception {
+    @Test
+    public void testReceived() {
 
-       }
+        pub.received();
+        assertTrue(true);
 
-       @Test
-       public void testToString() {
+    }
 
-               entry.toString();
-               assertTrue(true);
+    @Test
+    public void testResend() {
 
-       }
+        pub.reset();
+        assertTrue(true);
 
-       @Test
-       public void testAddListener() {
+    }
 
-               pub.addListener(null);
-               assertTrue(true);
+    @Test
+    public void testSend() {
 
-       }
+        pub.send("partition", "msg");
+        assertTrue(true);
 
-       @Test
-       public void testGetCaptures() {
+    }
 
-               pub.getCaptures();
-               assertTrue(true);
+    @Test
+    public void testSend2() {
 
-       }
+        pub.send("msg");
+        assertTrue(true);
 
-       @Test
-       public void testGetCaptures2() {
+    }
 
-               pub.getCaptures(null);
-               assertTrue(true);
+    @Test
+    public void testSend3() {
+        //sending message m obj
+        pub.send(new ArrayList<Message>());
+        assertTrue(true);
 
-       }
+    }
 
-       @Test
-       public void testReceived() {
+    @Test
+    public void testSend4() {
+        //sending collection of message m objects
+        pub.send(new Message("partition", "msg"));
+        assertTrue(true);
 
-               pub.received();
-               assertTrue(true);
+    }
 
-       }
-       
-       @Test
-       public void testResend() {
+    @Test
+    public void testSendBatchWithResponse() {
 
-               pub.reset();
-               assertTrue(true);
+        pub.sendBatchWithResponse();
+        assertTrue(true);
 
-       }
-       
-       @Test
-       public void testSend() {
+    }
 
-               pub.send("partition", "msg");
-               assertTrue(true);
+    @Test
+    public void testLogTo() {
 
-       }
-       
-       @Test
-       public void testSend2() {
+        pub.logTo(null);
+        assertTrue(true);
 
-               pub.send("msg");
-               assertTrue(true);
+    }
 
-       }
-       
-       @Test
-       public void testSend3() {
-               //sending message m obj
-               pub.send(new ArrayList<message>());
-               assertTrue(true);
+    @Test
+    public void testClearApiCredentials() {
 
-       }
-       
-       @Test
-       public void testSend4() {
-               //sending collection of message m objects
-               pub.send(new message("partition", "msg"));
-               assertTrue(true);
+        pub.clearApiCredentials();
+        assertTrue(true);
 
-       }
-       
-       @Test
-       public void testSendBatchWithResponse() {
+    }
 
-               pub.sendBatchWithResponse();
-               assertTrue(true);
+    @Test
+    public void testSetApiCredentials() {
 
-       }
-       
-       @Test
-       public void testLogTo() {
+        pub.setApiCredentials("apikey", "apisecret");
+        assertTrue(true);
 
-               pub.logTo(null);
-               assertTrue(true);
+    }
 
-       }
-       
-       @Test
-       public void testClearApiCredentials() {
+    @Test
+    public void testClose() {
 
-               pub.clearApiCredentials();
-               assertTrue(true);
+        pub.close();
+        assertTrue(true);
 
-       }
-       
-       @Test
-       public void testSetApiCredentials() {
+    }
 
-               pub.setApiCredentials("apikey", "apisecret");
-               assertTrue(true);
+    @Test
+    public void testClose2() {
 
-       }
-       
-       @Test
-       public void testClose() {
+        pub.close(100, null);
+        assertTrue(true);
 
-               pub.close();
-               assertTrue(true);
+    }
 
-       }
-       
-       @Test
-       public void testClose2() {
+    @Test
+    public void testGetPendingMessageCount() {
 
-               pub.close(100, null);
-               assertTrue(true);
+        pub.getPendingMessageCount();
+        assertTrue(true);
 
-       }
-       
-       @Test
-       public void testGetPendingMessageCount() {
-
-               pub.getPendingMessageCount();
-               assertTrue(true);
-
-       }
+    }
 }