Merge pull request #5 for v0.3.1 Release

Author: jeevatkm

README.md

@@ -1,9 +1,9 @@
 # log - aah framework
 [![Build Status](https://travis-ci.org/go-aah/log.svg?branch=master)](https://travis-ci.org/go-aah/log) [![codecov](https://codecov.io/gh/go-aah/log/branch/master/graph/badge.svg)](https://codecov.io/gh/go-aah/log/branch/master) [![Go Report Card](https://goreportcard.com/badge/aahframework.org/log.v0)](https://goreportcard.com/report/aahframework.org/log.v0)
-[![Version](https://img.shields.io/badge/version-0.3-blue.svg)](https://github.com/go-aah/log/releases/latest) [![GoDoc](https://godoc.org/aahframework.org/log.v0?status.svg)](https://godoc.org/aahframework.org/log.v0)
+[![Version](https://img.shields.io/badge/version-0.3.1-blue.svg)](https://github.com/go-aah/log/releases/latest) [![GoDoc](https://godoc.org/aahframework.org/log.v0?status.svg)](https://godoc.org/aahframework.org/log.v0)
 [![License](https://img.shields.io/github/license/go-aah/log.svg)](LICENSE)
 
-***v0.3 [released](https://github.com/go-aah/log/releases/latest) and tagged on May 16, 2017***
+***v0.3.1 [released](https://github.com/go-aah/log/releases/latest) and tagged on May 16, 2017***
 
 Simple, flexible & powerful `Go` logger inspired by standard logger & Google glog. aah framework utilizes `log` library across.
 

console_receiver_test.go

@@ -6,7 +6,6 @@ package log
 
 import (
 	"testing"
-	"time"
 
 	"aahframework.org/config.v0"
 	"aahframework.org/test.v0/assert"
@@ -122,5 +121,5 @@ func testConsoleLogger(t *testing.T, cfgStr string) {
 	logger.Error("Yes, yes, yes - finally an error")
 	logger.Errorf("Yes, yes, yes - %v", "finally an error")
 
-	time.Sleep(1 * time.Millisecond)
+	waitToDrain(logger)
 }

default.go

@@ -138,6 +138,12 @@ func SetPattern(pattern string) error {
 	return std.SetPattern(pattern)
 }
 
+// IsBufferEmpty returns true if logger buffer is empty otherwise false.
+// This method can be used to ensure all the log entry is written successfully.
+func IsBufferEmpty() bool {
+	return std.IsBufferEmpty()
+}
+
 func init() {
 	cfg, _ := config.ParseString("log { }")
 	std, _ = New(cfg)

default_test.go

@@ -6,7 +6,6 @@ package log
 
 import (
 	"testing"
-	"time"
 
 	"aahframework.org/config.v0"
 	"aahframework.org/test.v0/assert"
@@ -43,7 +42,7 @@ func TestDefaultLogger(t *testing.T) {
 	testStdPanic("panicf", "this is panicf")
 	testStdPanic("panicln", "this is panicln")
 
-	time.Sleep(1 * time.Millisecond)
+	waitToDrain(std)
 }
 
 func TestDefaultLoggerMisc(t *testing.T) {
@@ -56,7 +55,7 @@ func TestDefaultLoggerMisc(t *testing.T) {
 
 	assert.Nil(t, SetLevel("trace"))
 	assert.Nil(t, SetPattern("%level:-5 %message"))
-	time.Sleep(1 * time.Millisecond)
+	waitToDrain(std)
 }
 
 func testStdPanic(method, msg string) {

file_receiver_test.go

@@ -6,13 +6,12 @@ package log
 
 import (
 	"testing"
-	"time"
 
 	"aahframework.org/config.v0"
 	"aahframework.org/test.v0/assert"
 )
 
-func TestFileLoggerDailyRotation(t *testing.T) {
+func TestFileLoggerRotation(t *testing.T) {
 	cleaupFiles("*.log")
 	fileConfigStr1 := `
   log {
@@ -161,5 +160,5 @@ func testFileLogger(t *testing.T, cfgStr string, loop int) {
 		logger.Errorf("Yes, yes, yes - %v", "finally an error")
 	}
 
-	time.Sleep(2 * time.Millisecond)
+	waitToDrain(logger)
 }

log.go

@@ -65,7 +65,7 @@ const (
 
 var (
 	// Version no. of aahframework.org/log library
-	Version = "0.3"
+	Version = "0.3.1"
 
 	// FmtFlags is the list of log format flags supported by aah/log library
 	// Usage of flag order is up to format composition.
@@ -205,8 +205,14 @@ func (l *Logger) SetReceiver(receiver Receiver) error {
 	return l.receiver.Init(l.cfg)
 }
 
+// IsBufferEmpty returns true if logger buffer is empty otherwise false.
+// This method can be used to ensure all the log entry is written successfully.
+func (l *Logger) IsBufferEmpty() bool {
+	return len(l.ec) == 0
+}
+
 //‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾
-// Logger methods
+// Logger logging methods
 //_______________________________________
 
 // Error logs message as `ERROR`. Arguments handled in the mananer of `fmt.Print`.

log_test.go

@@ -33,7 +33,7 @@ func TestLogDefault(t *testing.T) {
 	testPanic(logger, "panic", "this is panic")
 	testPanic(logger, "panicf", "this is panicf")
 	testPanic(logger, "panicln", "this is panicln")
-	time.Sleep(1 * time.Millisecond)
+	waitToDrain(logger)
 }
 
 func TestMisc(t *testing.T) {
@@ -120,3 +120,11 @@ func cleaupFiles(match string) {
 		}
 	}
 }
+
+func waitToDrain(l *Logger) {
+	for {
+		if l.IsBufferEmpty() {
+			break
+		}
+	}
+}