Fix testing in memory logger race condition (#70)

* Fix testing in memory logger race condition

* add test race condition

testing/logger.go

@@ -31,13 +31,15 @@ func (s *inMemoryEntriesStore) CleanEntries() {
 
 // GetEntries will return the in memory entries list
 func (s *inMemoryEntriesStore) GetEntries() []logger.Entry {
+	defer s.lock()()
 	return s.entries
 }
 
 // GetAndCleanEntries will return and clean the in memory entries list
 func (s *inMemoryEntriesStore) GetAndCleanEntries() []logger.Entry {
-	entries := s.GetEntries()
-	s.CleanEntries()
+	defer s.lock()()
+	entries := s.entries
+	s.entries = []logger.Entry{}
 	return entries
 }
 

testing/logger_test.go

@@ -31,6 +31,18 @@ func TestNewLogger(t *testing.T) {
 	assert.Equal(t, "test Info", entry2.Message)
 }
 
+func TestNewLoggerRace(t *testing.T) {
+	myLogger, store := testing_logger.NewLogger()
+
+	go func() {
+		myLogger.Debug("test Debug")
+	}()
+
+	go func() {
+		store.GetEntries()
+	}()
+}
+
 func TestNewLogger_GetEntries(t *testing.T) {
 	myLogger, store := testing_logger.NewLogger()