[FAB-6886] SDK logger new utility function

Current utility 'IsEnabledFor' needs module name
as an argument to find if given level is enabled
for given module, which forces clients to preserve
logger module names for this use.

This new utility, IsEnabledForLogger - takes logger itself
as an argument and it will find module name by accessing
logger argument.


Change-Id: I0242667c12428a7a99e85184f4d7e639e3beb84e
Signed-off-by: Sudesh Shetty <[email protected]>

pkg/logging/logger.go

@@ -74,6 +74,11 @@ func IsEnabledFor(level Level, module string) bool {
 	return moduleLevels.IsEnabledFor(level, module)
 }
 
+// IsEnabledFor will return true if given logging level is enabled for the given logger.
+func IsEnabledForLogger(level Level, logger *Logger) bool {
+	return moduleLevels.IsEnabledFor(level, logger.module)
+}
+
 //Fatal calls Fatal function of underlying logger
 func (l *Logger) Fatal(args ...interface{}) {
 	l.getCurrentLogger().Fatal(args...)

pkg/logging/logger_test.go

@@ -65,9 +65,17 @@ func TestDefaultLogging(t *testing.T) {
 
 	verifyEmpty(t, buf.String(), "debug log isn't supposed to show up for info level")
 
+	//Should be false
+	verifyFalse(t, IsEnabledForLogger(DEBUG, logger), "logging.IsEnabled for is not working as expected, expected false but got true")
+	verifyFalse(t, IsEnabledFor(DEBUG, moduleName), "logging.IsEnabled for is not working as expected, expected false but got true")
+
 	//Now change the log level to DEBUG
 	SetLevel(DEBUG, moduleName)
 
+	//Should be false
+	verifyTrue(t, IsEnabledForLogger(DEBUG, logger), "logging.IsEnabled for is not working as expected, expected true but got false")
+	verifyTrue(t, IsEnabledFor(DEBUG, moduleName), "logging.IsEnabled for is not working as expected, expected true but got false")
+
 	//Test logger.debug outputs
 	verifyBasicLogging(t, DEBUG, logger.Debug, nil, &buf, false)
 	verifyBasicLogging(t, DEBUG, logger.Debugln, nil, &buf, false)