feat: allow existing logger from context (#898)
We should reuse an existing context logger if in test context. This will allow test to setup act with a null logger to assert log messages. Co-authored-by: Markus Wolf <markus.wolf@new-work.se> Co-authored-by: Markus Wolf <markus.wolf@new-work.se>
This commit is contained in:
@@ -48,6 +48,12 @@ func WithJobLogger(ctx context.Context, jobName string, secrets map[string]strin
|
||||
nextColor++
|
||||
|
||||
logger := logrus.New()
|
||||
if common.TestContext(ctx) {
|
||||
fieldLogger := common.Logger(ctx)
|
||||
if fieldLogger != nil {
|
||||
logger = fieldLogger.(*logrus.Logger)
|
||||
}
|
||||
}
|
||||
logger.SetFormatter(formatter)
|
||||
logger.SetOutput(os.Stdout)
|
||||
logger.SetLevel(logrus.GetLevel())
|
||||
|
Reference in New Issue
Block a user