Remove unused code from internal/logging

Author: bbasata

internal/logging/context.go

@@ -11,22 +11,6 @@ import (
 	testing "github.com/mitchellh/go-testing-interface"
 )
 
-// InitContext creates SDK logger contexts when the provider is running in
-// "production" (not under acceptance testing). The incoming context will
-// already have the root SDK logger and root provider logger setup from
-// terraform-plugin-go tf5server RPC handlers.
-func InitContext(ctx context.Context) context.Context {
-	ctx = tfsdklog.NewSubsystem(ctx, SubsystemHelperSchema,
-		// All calls are through the HelperSchema* helper functions
-		tfsdklog.WithAdditionalLocationOffset(1),
-		tfsdklog.WithLevelFromEnv(EnvTfLogSdkHelperSchema),
-		// Propagate tf_req_id, tf_rpc, etc. fields
-		tfsdklog.WithRootFields(),
-	)
-
-	return ctx
-}
-
 // InitTestContext registers the terraform-plugin-log/tfsdklog test sink,
 // configures the standard library log package, and creates SDK logger
 // contexts. The incoming context is expected to be devoid of logging setup.

internal/logging/context_test.go

@@ -15,43 +15,6 @@ import (
 	"github.com/hashicorp/terraform-plugin-testing/internal/logging"
 )
 
-func TestInitContext(t *testing.T) {
-	t.Parallel()
-
-	var output bytes.Buffer
-
-	ctx := tfsdklogtest.RootLogger(context.Background(), &output)
-
-	// Simulate root logger fields that would have been associated by
-	// terraform-plugin-go prior to the InitContext() call.
-	ctx = tfsdklog.SetField(ctx, "tf_rpc", "GetProviderSchema")
-	ctx = tfsdklog.SetField(ctx, "tf_req_id", "123-testing-123")
-
-	ctx = logging.InitContext(ctx)
-
-	logging.HelperSchemaTrace(ctx, "test message")
-
-	entries, err := tfsdklogtest.MultilineJSONDecode(&output)
-
-	if err != nil {
-		t.Fatalf("unable to read multiple line JSON: %s", err)
-	}
-
-	expectedEntries := []map[string]interface{}{
-		{
-			"@level":    "trace",
-			"@message":  "test message",
-			"@module":   "sdk.helper_schema",
-			"tf_rpc":    "GetProviderSchema",
-			"tf_req_id": "123-testing-123",
-		},
-	}
-
-	if diff := cmp.Diff(entries, expectedEntries); diff != "" {
-		t.Errorf("unexpected difference: %s", diff)
-	}
-}
-
 func TestTestNameContext(t *testing.T) {
 	t.Parallel()
 

internal/logging/environment_variables.go

@@ -20,8 +20,4 @@ const (
 	// unset.
 	EnvTfLogSdkHelperResource = "TF_LOG_SDK_HELPER_RESOURCE"
 
-	// EnvTfLogSdkHelperSchema is an environment variable that sets the logging
-	// level of SDK helper/schema loggers. Infers root SDK logging level, if
-	// unset.
-	EnvTfLogSdkHelperSchema = "TF_LOG_SDK_HELPER_SCHEMA"
 )