From 5f3b445e33b50c32122b126f6b69b735775f192b Mon Sep 17 00:00:00 2001
From: Robert Kowalski <robert.kowalski@new-work.se>
Date: Thu, 24 Feb 2022 12:16:44 +0100
Subject: [PATCH] fix: remove unused TestContext

The function `WithTestContext` is never called in the project, and
there is just one usage of `TestContext`, inside one of our loggers
---
 pkg/common/testflag.go | 20 --------------------
 pkg/runner/logger.go   |  6 ------
 2 files changed, 26 deletions(-)
 delete mode 100644 pkg/common/testflag.go

diff --git a/pkg/common/testflag.go b/pkg/common/testflag.go
deleted file mode 100644
index 4988563d3e8..00000000000
--- a/pkg/common/testflag.go
+++ /dev/null
@@ -1,20 +0,0 @@
-package common
-
-import (
-	"context"
-)
-
-type testFlagContextKey string
-
-const testFlagContextKeyVal = testFlagContextKey("test-context")
-
-// TestContext returns whether the context has the test flag set
-func TestContext(ctx context.Context) bool {
-	val := ctx.Value(testFlagContextKeyVal)
-	return val != nil
-}
-
-// WithTextContext sets the test flag in the context
-func WithTestContext(ctx context.Context) context.Context {
-	return context.WithValue(ctx, testFlagContextKeyVal, true)
-}
diff --git a/pkg/runner/logger.go b/pkg/runner/logger.go
index 8da4bf1ba8a..8464801575a 100644
--- a/pkg/runner/logger.go
+++ b/pkg/runner/logger.go
@@ -48,12 +48,6 @@ 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())