diff options
author | Jim Hahn <jrh3@att.com> | 2018-02-20 13:41:59 -0500 |
---|---|---|
committer | Jim Hahn <jrh3@att.com> | 2018-02-20 14:16:41 -0500 |
commit | 7f6b1b0add6878e996e559c15337f46339c98901 (patch) | |
tree | ed381d4fdaaed33a590c06145a9834331e32aa35 /common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/EelfLogger.java | |
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/onap/policy/common/logging/flexlogger/EelfLogger.java')
-rw-r--r-- | common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/EelfLogger.java | 20 |
1 files changed, 6 insertions, 14 deletions
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); } /** |