diff options
author | 2018-02-20 13:41:59 -0500 | |
---|---|---|
committer | 2018-02-20 14:16:41 -0500 | |
commit | 7f6b1b0add6878e996e559c15337f46339c98901 (patch) | |
tree | ed381d4fdaaed33a590c06145a9834331e32aa35 /common-logging/src/main/java/org | |
parent | 19d3bfa5a1e6f69cde7a5e814b1975330249dcd0 (diff) |
Increase junit coverage in ONAP-logging
Added some junit tests to increase coverage of ONAP-logger.
Many classes still need test cases.
Also fixed some sonar issues:
Returned interface instead of ConcurrentHashMap.
Used EnumMap instead of HashMap.
Used StringBuilder instead of StringBuffer.
Replaced literals with constants.
Replaced enum1.toString().equals(enum2.toString()) with enum1 == enum2
Updated license dates.
Updated another license date.
Change-Id: I777b6a913d7d03187a29dfeb817800b386ee23aa
Issue-ID: POLICY-582
Signed-off-by: Jim Hahn <jrh3@att.com>
Diffstat (limited to 'common-logging/src/main/java/org')
9 files changed, 88 insertions, 92 deletions
diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/DroolsPDPMDCInfo.java b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/DroolsPDPMDCInfo.java index c1e99822..ae4cc929 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/DroolsPDPMDCInfo.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/DroolsPDPMDCInfo.java @@ -1,8 +1,8 @@ -/*- +/* * ============LICENSE_START======================================================= * ONAP-Logging * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -21,6 +21,7 @@ package org.onap.policy.common.logging.eelf; import java.util.concurrent.ConcurrentHashMap; +import java.util.concurrent.ConcurrentMap; public class DroolsPDPMDCInfo implements MDCInfo { @@ -43,7 +44,7 @@ public class DroolsPDPMDCInfo implements MDCInfo { * @return the instance of ConcurrentHashMap */ @Override - public ConcurrentHashMap<String, String> getMDCInfo() { + public ConcurrentMap<String, String> getMDCInfo() { return mdcMap; } } diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/ErrorCodeMap.java b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/ErrorCodeMap.java index 6d347c9c..4df86a4d 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/ErrorCodeMap.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/ErrorCodeMap.java @@ -1,8 +1,8 @@ -/*- +/* * ============LICENSE_START======================================================= * ONAP-Logging * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -20,7 +20,7 @@ package org.onap.policy.common.logging.eelf; -import java.util.HashMap; +import java.util.EnumMap; /** * @@ -30,7 +30,7 @@ import java.util.HashMap; public class ErrorCodeMap { - protected static final HashMap<MessageCodes, ErrorCodeInfo> hm = new HashMap<>(); + protected static final EnumMap<MessageCodes, ErrorCodeInfo> hm = new EnumMap<>(MessageCodes.class); private static final String ERROR_PERMISSIONS = "POLICY-100E"; private static final String ERROR_PERMISSIONS_DESCRIPTION = "This is a Permissions Error. Please check the error message for detail information"; diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/EventTrackInfo.java b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/EventTrackInfo.java index 6c31a990..077a5a70 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/EventTrackInfo.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/EventTrackInfo.java @@ -1,8 +1,8 @@ -/*- +/* * ============LICENSE_START======================================================= * ONAP-Logging * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -21,13 +21,14 @@ package org.onap.policy.common.logging.eelf; import java.util.concurrent.ConcurrentHashMap; +import java.util.concurrent.ConcurrentMap; /** * EventTrackInfo contains a ConcurrentHashMap of EventData. */ public class EventTrackInfo { - private ConcurrentHashMap<String, EventData> eventInfo = null; + private final ConcurrentMap<String, EventData> eventInfo; public EventTrackInfo() { /* @@ -78,7 +79,7 @@ public class EventTrackInfo { /** * Returns a ConcurrentHashMap of EventData */ - public ConcurrentHashMap<String, EventData> getEventInfo() { + public ConcurrentMap<String, EventData> getEventInfo() { return eventInfo; } } diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/EventTrackInfoHandler.java b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/EventTrackInfoHandler.java index 12f1af61..a79ab521 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/EventTrackInfoHandler.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/EventTrackInfoHandler.java @@ -1,8 +1,8 @@ -/*- +/* * ============LICENSE_START======================================================= * ONAP-Logging * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -25,6 +25,7 @@ import java.time.Instant; import java.util.ArrayList; import java.util.TimerTask; import java.util.concurrent.ConcurrentHashMap; +import java.util.concurrent.ConcurrentMap; /** * @@ -54,7 +55,7 @@ public class EventTrackInfoHandler extends TimerTask { if (eventTrackInfo == null) { return; } - ConcurrentHashMap<String, EventData> eventInfo = eventTrackInfo.getEventInfo(); + ConcurrentMap<String, EventData> eventInfo = eventTrackInfo.getEventInfo(); if (eventInfo == null || eventInfo.isEmpty()) { return; } diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/MDCInfo.java b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/MDCInfo.java index 0d0b5dff..f49b34a7 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/MDCInfo.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/MDCInfo.java @@ -1,8 +1,8 @@ -/*- +/* * ============LICENSE_START======================================================= * ONAP-Logging * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -20,7 +20,7 @@ package org.onap.policy.common.logging.eelf; -import java.util.concurrent.ConcurrentHashMap; +import java.util.concurrent.ConcurrentMap; /** * @@ -34,5 +34,5 @@ public interface MDCInfo { * Returns MDC info. * @return MDC info */ - ConcurrentHashMap<String, String> getMDCInfo(); + ConcurrentMap<String, String> getMDCInfo(); } diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/PolicyLogger.java b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/PolicyLogger.java index b43dca45..58db1da4 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/PolicyLogger.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/PolicyLogger.java @@ -1,8 +1,8 @@ -/*- +/* * ============LICENSE_START======================================================= * ONAP-Logging * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -20,12 +20,9 @@ package org.onap.policy.common.logging.eelf; -import org.slf4j.MDC; - import static org.onap.policy.common.logging.eelf.Configuration.*; import org.onap.policy.common.logging.flexlogger.LoggerType; -import org.onap.policy.common.logging.eelf.PolicyLogger; import com.att.eelf.configuration.EELFLogger; import com.att.eelf.configuration.EELFLogger.Level; import com.att.eelf.configuration.EELFManager; @@ -43,7 +40,9 @@ import java.util.Properties; import java.util.Timer; import java.util.TimerTask; import java.util.UUID; -import java.util.concurrent.ConcurrentHashMap; +import java.util.concurrent.ConcurrentMap; + +import org.slf4j.MDC; /** * @@ -63,6 +62,8 @@ public class PolicyLogger { private static EELFLogger debugLogger = EELFManager.getInstance() .getDebugLogger(); + + private static final String POLICY_LOGGER = "PolicyLogger"; private static EventTrackInfo eventTracker = new EventTrackInfo(); @@ -95,7 +96,7 @@ public class PolicyLogger { private static Level auditLevel = Level.INFO; private static Level metricsLevel = Level.INFO; private static Level errorLevel = Level.ERROR; - private static String CLASS_NAME = "ClassName"; + private static String classNameProp = "ClassName"; private static final String DATE_FORMAT = "yyyy-MM-dd'T'HH:mm:ss.SSS+00:00"; private static final String COMPLETE_STATUS = "COMPLETE"; @@ -107,7 +108,7 @@ public class PolicyLogger { hostName = InetAddress.getLocalHost().getHostName(); hostAddress = InetAddress.getLocalHost().getHostAddress(); } catch (UnknownHostException e) { - PolicyLogger.error(MessageCodes.EXCEPTION_ERROR, e, "PolicyLogger", "UnknownHostException"); + PolicyLogger.error(MessageCodes.EXCEPTION_ERROR, e, POLICY_LOGGER, "UnknownHostException"); } } } @@ -145,11 +146,11 @@ public class PolicyLogger { } public static String getClassname() { - return CLASS_NAME; + return classNameProp; } public static synchronized void setClassname(String name) { - CLASS_NAME = name; + classNameProp = name; } /** @@ -184,7 +185,7 @@ public class PolicyLogger { MDC.put(MDC_SERVER_FQDN, hostName); MDC.put(MDC_SERVER_IP_ADDRESS, hostAddress); } catch (Exception e) { - errorLogger.error(MessageCodes.EXCEPTION_ERROR, e, "PolicyLogger"); + errorLogger.error(MessageCodes.EXCEPTION_ERROR, e, POLICY_LOGGER); } Instant startTime = Instant.now(); Instant endTime = Instant.now(); @@ -225,7 +226,7 @@ public class PolicyLogger { MDC.put(MDC_KEY_REQUEST_ID, transId); if(mdcInfo != null && mdcInfo.getMDCInfo() != null && !mdcInfo.getMDCInfo().isEmpty()){ - ConcurrentHashMap<String, String> mdcMap = mdcInfo.getMDCInfo(); + ConcurrentMap<String, String> mdcMap = mdcInfo.getMDCInfo(); Iterator<String> keyIterator = mdcMap.keySet().iterator(); String key; @@ -239,7 +240,7 @@ public class PolicyLogger { MDC.put(MDC_SERVER_FQDN, hostName); MDC.put(MDC_SERVER_IP_ADDRESS, hostAddress); } catch (Exception e) { - errorLogger.error(MessageCodes.EXCEPTION_ERROR, e, "PolicyLogger"); + errorLogger.error(MessageCodes.EXCEPTION_ERROR, e, POLICY_LOGGER); } Instant startTime = Instant.now(); Instant endTime = Instant.now(); @@ -336,7 +337,7 @@ public class PolicyLogger { MDC.put(MDC_SERVER_FQDN, hostName); MDC.put(MDC_SERVER_IP_ADDRESS, hostAddress); } catch (Exception e) { - errorLogger.error(MessageCodes.EXCEPTION_ERROR, e, "PolicyLogger"); + errorLogger.error(MessageCodes.EXCEPTION_ERROR, e, POLICY_LOGGER); } MDC.put(MDC_INSTANCE_UUID, ""); MDC.put(MDC_ALERT_SEVERITY, ""); @@ -363,7 +364,7 @@ public class PolicyLogger { * @param arguments */ public static void info(MessageCodes msg, String... arguments) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); debugLogger.info(msg, arguments); } @@ -374,7 +375,7 @@ public class PolicyLogger { * @param arguments */ public static void info(MessageCodes msg, String className, String... arguments) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); debugLogger.info(msg, arguments); } @@ -384,7 +385,7 @@ public class PolicyLogger { * @param arg0 */ public static void info( String className, String arg0) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); debugLogger.info(MessageCodes.GENERAL_INFO, arg0); } @@ -394,7 +395,7 @@ public class PolicyLogger { * @param arg0 */ public static void info(Object arg0) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); debugLogger.info(MessageCodes.GENERAL_INFO, String.valueOf(arg0)); } @@ -406,7 +407,7 @@ public class PolicyLogger { */ public static void info(MessageCodes msg, Throwable arg0, String... arguments) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); String arguments2 = getNormalizedStackTrace(arg0, arguments); debugLogger.info(msg, arguments2); } @@ -420,7 +421,7 @@ public class PolicyLogger { */ public static void info(MessageCodes msg, String className, Throwable arg0, String... arguments) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); String arguments2 = getNormalizedStackTrace(arg0, arguments); debugLogger.info(msg, arguments2); } @@ -431,7 +432,7 @@ public class PolicyLogger { * @param className class name */ public static void warn( String className, String arg0) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); debugLogger.warn(MessageCodes.GENERAL_INFO, arg0); } @@ -440,7 +441,7 @@ public class PolicyLogger { * @param arg0 */ public static void warn(Object arg0) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); debugLogger.warn(MessageCodes.GENERAL_WARNING, "" + arg0); } @@ -449,7 +450,7 @@ public class PolicyLogger { * @param arg0 */ public static void warn(String arg0) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); debugLogger.warn(MessageCodes.GENERAL_WARNING, arg0); } @@ -459,7 +460,7 @@ public class PolicyLogger { * @param arguments */ public static void warn(MessageCodes msg, String... arguments) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); debugLogger.warn(msg, arguments); } @@ -470,7 +471,7 @@ public class PolicyLogger { * @param arguments */ public static void warn(MessageCodes msg, String className, String... arguments) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); debugLogger.warn(msg, arguments); } @@ -482,7 +483,7 @@ public class PolicyLogger { */ public static void warn(MessageCodes msg, Throwable arg0, String... arguments) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); String arguments2 = getNormalizedStackTrace(arg0, arguments); debugLogger.warn(msg, arguments2); } @@ -496,7 +497,7 @@ public class PolicyLogger { */ public static void warn(MessageCodes msg, String className, Throwable arg0, String... arguments) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); String arguments2 = getNormalizedStackTrace(arg0, arguments); debugLogger.warn(msg, arguments2); } @@ -507,7 +508,7 @@ public class PolicyLogger { * @param arg0 log message */ public static void error( String className, String arg0) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); if(ErrorCodeMap.hm.get(MessageCodes.GENERAL_ERROR) != null){ MDC.put(ERROR_CODE, ErrorCodeMap.hm.get(MessageCodes.GENERAL_ERROR).getErrorCode()); MDC.put(ERROR_DESCRIPTION, ErrorCodeMap.hm.get(MessageCodes.GENERAL_ERROR).getErrorDesc()); @@ -521,7 +522,7 @@ public class PolicyLogger { * @param arg0 */ public static void error(String arg0) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); MDC.put(ERROR_CATEGORY, ERROR_CATEGORY_VALUE); if(ErrorCodeMap.hm.get(MessageCodes.GENERAL_ERROR) != null){ @@ -537,7 +538,7 @@ public class PolicyLogger { * @param arg0 */ public static void error(Object arg0) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); MDC.put(ERROR_CATEGORY, ERROR_CATEGORY_VALUE); if(ErrorCodeMap.hm.get(MessageCodes.GENERAL_ERROR) != null){ @@ -556,7 +557,7 @@ public class PolicyLogger { */ public static void error(MessageCodes msg, Throwable arg0, String... arguments) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); MDC.put(ERROR_CATEGORY, ERROR_CATEGORY_VALUE); if(ErrorCodeMap.hm.get(msg) != null){ @@ -577,7 +578,7 @@ public class PolicyLogger { */ public static void error(MessageCodes msg, String className, Throwable arg0, String... arguments) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); MDC.put(ERROR_CATEGORY, ERROR_CATEGORY_VALUE); if(ErrorCodeMap.hm.get(msg) != null){ @@ -595,7 +596,7 @@ public class PolicyLogger { * @param arguments */ public static void error(MessageCodes msg, String... arguments) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); MDC.put(ERROR_CATEGORY, ERROR_CATEGORY_VALUE); if(ErrorCodeMap.hm.get(msg) != null){ @@ -612,7 +613,7 @@ public class PolicyLogger { * @param arguments */ public static void debug(MessageCodes msg, String... arguments) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); debugLogger.debug(msg, arguments); } @@ -622,7 +623,7 @@ public class PolicyLogger { * @param arg0 */ public static void debug( String className, String arg0) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); debugLogger.debug(MessageCodes.GENERAL_INFO, arg0); } @@ -631,7 +632,7 @@ public class PolicyLogger { * @param arg0 */ public static void debug(String arg0) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); debugLogger.debug(arg0); } @@ -641,7 +642,7 @@ public class PolicyLogger { */ public static void debug(Object arg0) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); debugLogger.debug("" + arg0); } @@ -652,7 +653,7 @@ public class PolicyLogger { */ public static void audit(String className, Object arg0) { MDC.put(STATUS_CODE, COMPLETE_STATUS); - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); auditLogger.info("" + arg0); } @@ -662,7 +663,7 @@ public class PolicyLogger { */ public static void audit(Object arg0) { MDC.put(STATUS_CODE, COMPLETE_STATUS); - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); auditLogger.info("" + arg0); } @@ -674,7 +675,7 @@ public class PolicyLogger { */ public static void debug(MessageCodes msg, Throwable arg0, String... arguments) { - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); String arguments2 = getNormalizedStackTrace(arg0, arguments); errorLogger.error(msg, arguments2); } @@ -688,7 +689,7 @@ public class PolicyLogger { */ public static void debug(MessageCodes msg, String className, Throwable arg0, String... arguments) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); String arguments2 = getNormalizedStackTrace(arg0, arguments); errorLogger.error(msg, arguments2); } @@ -746,7 +747,7 @@ public class PolicyLogger { * @param arg0 */ public static void trace( String className, String arg0) { - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); errorLogger.info(MessageCodes.GENERAL_INFO, arg0); } @@ -756,7 +757,7 @@ public class PolicyLogger { */ public static void trace(Object arg0){ - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); debugLogger.trace(""+arg0); } /** @@ -1012,7 +1013,7 @@ public class PolicyLogger { seTimeStamps(); - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); String serviceName = MDC.get(MDC_SERVICE_NAME); MDC.put(MDC_KEY_REQUEST_ID, eventId); metricsLogger.info(MessageCodes.RULE_AUDIT_END_INFO, @@ -1063,7 +1064,7 @@ public class PolicyLogger { */ public static void metrics(String className, Object arg0) { seTimeStamps(); - MDC.put(CLASS_NAME, className); + MDC.put(classNameProp, className); String serviceName = MDC.get(MDC_SERVICE_NAME); metricsLogger.info(MessageCodes.RULE_METRICS_INFO, serviceName, ""+arg0); @@ -1075,7 +1076,7 @@ public class PolicyLogger { */ public static void metrics(Object arg0) { seTimeStamps(); - MDC.put(CLASS_NAME, ""); + MDC.put(classNameProp, ""); String serviceName = MDC.get(MDC_SERVICE_NAME); metricsLogger.info(MessageCodes.RULE_METRICS_INFO, serviceName, ""+arg0); @@ -1101,7 +1102,7 @@ public class PolicyLogger { t.printStackTrace(pw); String newStValue = sw.toString().replace ('|', '!').replace ("\n", " - "); int curSize = arguments == null ? 0 : arguments.length; - StringBuffer newArgument = new StringBuffer(); + StringBuilder newArgument = new StringBuilder(); for(int i=0; i<curSize; i++) { newArgument.append(arguments[i]); newArgument.append(":"); diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/EelfLogger.java b/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/EelfLogger.java index cc3dba30..ff097edf 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/EelfLogger.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/EelfLogger.java @@ -1,8 +1,8 @@ -/*- +/* * ============LICENSE_START======================================================= * ONAP-Logging * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -226,12 +226,8 @@ public class EelfLogger implements Logger, Serializable { * @return boolean */ @Override - public boolean isAuditEnabled(){ - if(PolicyLogger.getAuditLevel() != null && PolicyLogger.getAuditLevel().toString().equals(Level.OFF.toString())){ - return false; - }else { - return true; - } + public boolean isAuditEnabled(){ + return(PolicyLogger.getAuditLevel() != Level.OFF); } /** @@ -239,12 +235,8 @@ public class EelfLogger implements Logger, Serializable { * @return boolean */ @Override - public boolean isMetricsEnabled(){ - if(PolicyLogger.getMetricsLevel() != null && PolicyLogger.getMetricsLevel().toString().equals(Level.OFF.toString())){ - return false; - }else { - return true; - } + public boolean isMetricsEnabled(){ + return(PolicyLogger.getMetricsLevel() != Level.OFF); } /** diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/Logger4J.java b/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/Logger4J.java index 746012d1..30349d5c 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/Logger4J.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/Logger4J.java @@ -1,8 +1,8 @@ -/*- +/* * ============LICENSE_START======================================================= * ONAP-Logging * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -183,7 +183,7 @@ public class Logger4J implements org.onap.policy.common.logging.flexlogger.Logge */ @Override public boolean isAuditEnabled(){ - return !(PolicyLogger.getAuditLevel() != null && PolicyLogger.getAuditLevel().toString().equals(Level.OFF.toString())); + return(PolicyLogger.getAuditLevel() != Level.OFF); } /** @@ -191,8 +191,8 @@ public class Logger4J implements org.onap.policy.common.logging.flexlogger.Logge * @return boolean */ @Override - public boolean isMetricsEnabled(){ - return !(PolicyLogger.getMetricsLevel() != null && PolicyLogger.getMetricsLevel().toString().equals(Level.OFF.toString())); + public boolean isMetricsEnabled(){ + return(PolicyLogger.getMetricsLevel() != Level.OFF); } /** diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/SystemOutLogger.java b/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/SystemOutLogger.java index 295e259a..cb005c13 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/SystemOutLogger.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/SystemOutLogger.java @@ -1,8 +1,8 @@ -/*- +/* * ============LICENSE_START======================================================= * ONAP-Logging * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -76,7 +76,7 @@ public class SystemOutLogger implements Logger, Serializable { */ private void initLevel(){ - if(PolicyLogger.getDebugLevel() != null && PolicyLogger.getDebugLevel().toString().equals(Level.DEBUG.toString())){ + if(PolicyLogger.getDebugLevel() == Level.DEBUG){ isDebugEnabled = true; isInfoEnabled = true; isWarnEnabled = true; @@ -84,27 +84,27 @@ public class SystemOutLogger implements Logger, Serializable { isDebugEnabled = false; } - if(PolicyLogger.getDebugLevel() != null && PolicyLogger.getDebugLevel().toString().equals(Level.INFO.toString())){ + if(PolicyLogger.getDebugLevel() == Level.INFO){ isInfoEnabled = true; isWarnEnabled = true; isDebugEnabled = false; } - if(PolicyLogger.getDebugLevel() != null && PolicyLogger.getDebugLevel().toString().equals(Level.OFF.toString())){ + if(PolicyLogger.getDebugLevel() == Level.OFF){ isInfoEnabled = false; isWarnEnabled = false; isDebugEnabled = false; } - if(PolicyLogger.getErrorLevel() != null && PolicyLogger.getErrorLevel().toString().equals(Level.OFF.toString())){ + if(PolicyLogger.getErrorLevel() == Level.OFF){ isErrorEnabled = false; } - if(PolicyLogger.getAuditLevel() != null && PolicyLogger.getAuditLevel().toString().equals(Level.OFF.toString())){ + if(PolicyLogger.getAuditLevel() == Level.OFF){ isAuditEnabled = false; } - if(PolicyLogger.getMetricsLevel() != null && PolicyLogger.getMetricsLevel().toString().equals(Level.OFF.toString())){ + if(PolicyLogger.getMetricsLevel() == Level.OFF){ isMetricsEnabled = false; } } |