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/SystemOutLogger.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/SystemOutLogger.java')
| -rw-r--r-- | common-logging/src/main/java/org/onap/policy/common/logging/flexlogger/SystemOutLogger.java | 16 |
1 files changed, 8 insertions, 8 deletions
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; } } |