get rid of interal.app.Logger

but internal/test/ needs its own Logger definition to avoid circular import problems with internal/lg/ tests
nsqio · May 26, 2017 · 4203909 · 4203909
1 parent 91a0f33
commit 4203909
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/internal/app/logger.go b/internal/app/logger.go
diff --git a/internal/test/logger.go b/internal/test/logger.go
@@ -1,8 +1,8 @@
 package test
 
-import (
-	"github.com/nsqio/nsq/internal/app"
-)
+type Logger interface {
+	Output(maxdepth int, s string) error
+}
 
 type tbLog interface {
 	Log(...interface{})
@@ -17,6 +17,6 @@ func (tl *testLogger) Output(maxdepth int, s string) error {
 	return nil
 }
 
-func NewTestLogger(tbl tbLog) app.Logger {
+func NewTestLogger(tbl tbLog) Logger {
 	return &testLogger{tbl}
 }