[#noissue] Fix testRecordNotChainedException test failure

Author: intr3p1d

agent-module/profiler/src/test/java/com/navercorp/pinpoint/profiler/metadata/DefaultExceptionRecorderTest.java

@@ -40,14 +40,13 @@
 public class DefaultExceptionRecorderTest {
 
     private final static Logger logger = LogManager.getLogger(DefaultExceptionRecorderTest.class);
+    private final static long START_TIME = 1;
 
     ExceptionChainSampler exceptionChainSampler = new ExceptionChainSampler(1000);
     ExceptionWrapperFactory exceptionWrapperFactory = new ExceptionWrapperFactory(10, 1048);
 
-    long START_TIME = 1;
 
-
-    class TestExceptionStorage implements ExceptionStorage {
+    static class TestExceptionStorage implements ExceptionStorage {
 
         List<ExceptionWrapper> wrappers;
         public List<ExceptionWrapper> outputStream;
@@ -59,7 +58,7 @@ public TestExceptionStorage() {
 
         @Override
         public void store(List<ExceptionWrapper> wrappers) {
-            logger.error(wrappers);
+            logger.info(wrappers);
             this.wrappers.addAll(wrappers);
         }
 
@@ -85,7 +84,7 @@ public List<ExceptionWrapper> getOutputStream() {
         }
     }
 
-    private Function<Throwable, Throwable> getThrowableFunction(
+    private static Function<Throwable, Throwable> getThrowableFunction(
             DefaultExceptionRecorder recorder,
             List<Throwable> throwable
     ) {
@@ -112,11 +111,11 @@ public void testRecordNothing() {
         DefaultExceptionRecorder exceptionRecorder = new DefaultExceptionRecorder(
                 exceptionChainSampler, exceptionWrapperFactory, context
         );
-        Function<Throwable, Throwable> throwableFunction = getThrowableFunction(
+        Function<Throwable, Throwable> recordThrowable = getThrowableFunction(
                 exceptionRecorder, exceptions
         );
 
-        exceptionRecorder.recordException(null, 0);
+        recordThrowable.apply(null);
         exceptionRecorder.close();
 
         List<ExceptionWrapper> expected = new ArrayList<>();
@@ -133,18 +132,18 @@ public void testRecordException() {
         DefaultExceptionRecorder exceptionRecorder = new DefaultExceptionRecorder(
                 exceptionChainSampler, exceptionWrapperFactory, context
         );
-        Function<Throwable, Throwable> throwableFunction = getThrowableFunction(
+        Function<Throwable, Throwable> recordThrowable = getThrowableFunction(
                 exceptionRecorder, exceptions
         );
 
         List<ExceptionWrapper> expected = null;
         List<ExceptionWrapper> actual = null;
 
         try {
-            level3Error(throwableFunction);
+            level3Error(recordThrowable);
         } catch (Throwable e) {
             expected = newExceptionWrappers(e, START_TIME, 1);
-            exceptionRecorder.recordException(e, START_TIME);
+            recordThrowable.apply(e);
             actual = exceptionStorage.getWrappers();
             Assertions.assertTrue(actual.isEmpty());
         }
@@ -161,7 +160,7 @@ public void testRecordNotChainedException() {
         DefaultExceptionRecorder exceptionRecorder = new DefaultExceptionRecorder(
                 exceptionChainSampler, exceptionWrapperFactory, context
         );
-        Function<Throwable, Throwable> throwableFunction = getThrowableFunction(
+        Function<Throwable, Throwable> recordThrowable = getThrowableFunction(
                 exceptionRecorder, exceptions
         );
 
@@ -173,10 +172,9 @@ public void testRecordNotChainedException() {
         Throwable throwable = null;
 
         try {
-            notChainedException(throwableFunction);
+            notChainedException(recordThrowable);
         } catch (Throwable e) {
             expected1 = newExceptionWrappers(exceptions.get(exceptions.size() - 2), START_TIME, 1);
-            exceptionRecorder.recordException(e, START_TIME);
             logger.warn(exceptionStorage.getWrappers());
             actual1 = new ArrayList<>(exceptionStorage.getWrappers());
             throwable = e;
@@ -204,18 +202,17 @@ public void testRecordRethrowGivenException() {
         DefaultExceptionRecorder exceptionRecorder = new DefaultExceptionRecorder(
                 exceptionChainSampler, exceptionWrapperFactory, context
         );
-        Function<Throwable, Throwable> throwableFunction = getThrowableFunction(
+        Function<Throwable, Throwable> recordThrowable = getThrowableFunction(
                 exceptionRecorder, exceptions
         );
 
         List<ExceptionWrapper> expected = null;
         List<ExceptionWrapper> actual = null;
 
         try {
-            rethrowGivenException(throwableFunction);
+            rethrowGivenException(recordThrowable);
         } catch (Throwable e) {
             expected = newExceptionWrappers(e, START_TIME, 1);
-            exceptionRecorder.recordException(e, START_TIME);
             actual = exceptionStorage.getWrappers();
             Assertions.assertTrue(actual.isEmpty());
         }