diff options
-rw-r--r-- | utils/src/main/java/org/onap/policy/common/utils/logging/LoggerUtils.java | 31 | ||||
-rw-r--r-- | utils/src/test/java/org/onap/policy/common/utils/logging/LoggerUtilsTest.java | 10 |
2 files changed, 5 insertions, 36 deletions
diff --git a/utils/src/main/java/org/onap/policy/common/utils/logging/LoggerUtils.java b/utils/src/main/java/org/onap/policy/common/utils/logging/LoggerUtils.java index 9276f749..89575eb2 100644 --- a/utils/src/main/java/org/onap/policy/common/utils/logging/LoggerUtils.java +++ b/utils/src/main/java/org/onap/policy/common/utils/logging/LoggerUtils.java @@ -23,13 +23,15 @@ package org.onap.policy.common.utils.logging; import ch.qos.logback.classic.LoggerContext; +import lombok.AccessLevel; +import lombok.NoArgsConstructor; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.slf4j.Marker; import org.slf4j.MarkerFactory; - -public class LoggerUtils { +@NoArgsConstructor(access = AccessLevel.PRIVATE) +public final class LoggerUtils { private static final Logger LOGGER = LoggerFactory.getLogger(LoggerUtils.class); @@ -70,17 +72,6 @@ public class LoggerUtils { */ public static final Marker TRANSACTION_LOG_MARKER = MarkerFactory.getMarker(TRANSACTION_LOG_MARKER_NAME); - /** - * Logger delegate. - */ - private final Logger mlogger; - - /** - * Constructor. - */ - public LoggerUtils(final Logger loggerP) { - this.mlogger = checkNotNull(loggerP); - } /** * Set the log level of a logger. @@ -111,18 +102,4 @@ public class LoggerUtils { return logger.getLevel().toString(); } - - /** - * Dependency-free nullcheck. - * - * @param in to be checked - * @param <T> argument (and return) type - * @return input arg - */ - private static <T> T checkNotNull(final T in) { - if (in == null) { - throw new NullPointerException(); - } - return in; - } } diff --git a/utils/src/test/java/org/onap/policy/common/utils/logging/LoggerUtilsTest.java b/utils/src/test/java/org/onap/policy/common/utils/logging/LoggerUtilsTest.java index 61b83b4c..79db2093 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/logging/LoggerUtilsTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/logging/LoggerUtilsTest.java @@ -24,10 +24,9 @@ package org.onap.policy.common.utils.logging; import static org.junit.Assert.assertTrue; -import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; -import org.mockito.runners.MockitoJUnitRunner; +import org.mockito.junit.MockitoJUnitRunner; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -35,13 +34,6 @@ import org.slf4j.LoggerFactory; public class LoggerUtilsTest { protected static final Logger logger = LoggerFactory.getLogger(LoggerUtilsTest.class); - private LoggerUtils util; - - @Before - public void setup() { - this.util = new LoggerUtils(logger); - } - @Test public void testMarker() { assertTrue(logger.isInfoEnabled()); |