chore: move testotel into internal (envoyproxy#1073)

mathetake · web-flow · commit 256d58f7004e · 2025-08-13T11:16:39.000-07:00
**Description**

There's no reason to make them exposed in public.

---------

Signed-off-by: Takeshi Yoneda &lt;t.y.mathetake@gmail.com&gt;
diff --git a/internal/testing/testotel/collector.go b/internal/testing/testotel/collector.go
diff --git a/internal/testing/testotel/mem.go b/internal/testing/testotel/mem.go
diff --git a/internal/tracing/openinference/openai/chat_completion_config_test.go b/internal/tracing/openinference/openai/chat_completion_config_test.go
@@ -15,8 +15,8 @@ import (
 	oteltrace "go.opentelemetry.io/otel/trace"
 
 	"github.com/envoyproxy/ai-gateway/internal/apischema/openai"
+	"github.com/envoyproxy/ai-gateway/internal/testing/testotel"
 	"github.com/envoyproxy/ai-gateway/internal/tracing/openinference"
-	"github.com/envoyproxy/ai-gateway/tests/testotel"
 )
 
 func TestChatCompletionRecorder_WithConfig_HideInputs(t *testing.T) {
diff --git a/internal/tracing/openinference/openai/chat_completion_test.go b/internal/tracing/openinference/openai/chat_completion_test.go
@@ -17,8 +17,8 @@ import (
 	oteltrace "go.opentelemetry.io/otel/trace"
 
 	"github.com/envoyproxy/ai-gateway/internal/apischema/openai"
+	"github.com/envoyproxy/ai-gateway/internal/testing/testotel"
 	"github.com/envoyproxy/ai-gateway/internal/tracing/openinference"
-	"github.com/envoyproxy/ai-gateway/tests/testotel"
 )
 
 func TestChatCompletionRecorder_StartParams(t *testing.T) {
diff --git a/internal/tracing/openinference/openai/response_error_test.go b/internal/tracing/openinference/openai/response_error_test.go
@@ -15,8 +15,8 @@ import (
 	"go.opentelemetry.io/otel/sdk/trace"
 	oteltrace "go.opentelemetry.io/otel/trace"
 
+	"github.com/envoyproxy/ai-gateway/internal/testing/testotel"
 	"github.com/envoyproxy/ai-gateway/internal/tracing/openinference"
-	"github.com/envoyproxy/ai-gateway/tests/testotel"
 )
 
 func TestRecordResponseError(t *testing.T) {
diff --git a/internal/tracing/span_test.go b/internal/tracing/span_test.go
@@ -14,7 +14,7 @@ import (
 	"go.opentelemetry.io/otel/sdk/trace"
 	oteltrace "go.opentelemetry.io/otel/trace"
 
-	"github.com/envoyproxy/ai-gateway/tests/testotel"
+	"github.com/envoyproxy/ai-gateway/internal/testing/testotel"
 )
 
 func TestChatCompletionSpan_RecordChunk(t *testing.T) {
diff --git a/internal/tracing/tracing_test.go b/internal/tracing/tracing_test.go
@@ -20,10 +20,10 @@ import (
 	"go.opentelemetry.io/otel/trace/noop"
 
 	"github.com/envoyproxy/ai-gateway/internal/apischema/openai"
+	"github.com/envoyproxy/ai-gateway/internal/testing/testotel"
 	tracing "github.com/envoyproxy/ai-gateway/internal/tracing/api"
 	"github.com/envoyproxy/ai-gateway/internal/tracing/openinference"
 	openaitracing "github.com/envoyproxy/ai-gateway/internal/tracing/openinference/openai"
-	"github.com/envoyproxy/ai-gateway/tests/testotel"
 )
 
 // TestNewTracingFromEnv_DefaultServiceName tests that the service name
diff --git a/tests/extproc/vcr/otel_test.go b/tests/extproc/vcr/otel_test.go
@@ -13,8 +13,8 @@ import (
 
 	"github.com/stretchr/testify/require"
 
+	"github.com/envoyproxy/ai-gateway/internal/testing/testotel"
 	"github.com/envoyproxy/ai-gateway/tests/internal/testenvironment"
-	"github.com/envoyproxy/ai-gateway/tests/testotel"
 )
 
 // otelTestEnvironment holds all the services needed for OTEL tests.

Original file line number	Diff line number	Diff line change
`@@ -15,8 +15,8 @@ import (`
`15`	`15`	`oteltrace "go.opentelemetry.io/otel/trace"`
`16`	`16`
`17`	`17`	`"github.com/envoyproxy/ai-gateway/internal/apischema/openai"`
	`18`	`+ "github.com/envoyproxy/ai-gateway/internal/testing/testotel"`
`18`	`19`	`"github.com/envoyproxy/ai-gateway/internal/tracing/openinference"`
`19`		`- "github.com/envoyproxy/ai-gateway/tests/testotel"`
`20`	`20`	`)`
`21`	`21`
`22`	`22`	`func TestChatCompletionRecorder_WithConfig_HideInputs(t *testing.T) {`
Original file line number	Diff line number	Diff line change
`@@ -17,8 +17,8 @@ import (`
`17`	`17`	`oteltrace "go.opentelemetry.io/otel/trace"`
`18`	`18`
`19`	`19`	`"github.com/envoyproxy/ai-gateway/internal/apischema/openai"`
	`20`	`+ "github.com/envoyproxy/ai-gateway/internal/testing/testotel"`
`20`	`21`	`"github.com/envoyproxy/ai-gateway/internal/tracing/openinference"`
`21`		`- "github.com/envoyproxy/ai-gateway/tests/testotel"`
`22`	`22`	`)`
`23`	`23`
`24`	`24`	`func TestChatCompletionRecorder_StartParams(t *testing.T) {`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ import (`
`14`	`14`	`"go.opentelemetry.io/otel/sdk/trace"`
`15`	`15`	`oteltrace "go.opentelemetry.io/otel/trace"`
`16`	`16`
`17`		`- "github.com/envoyproxy/ai-gateway/tests/testotel"`
	`17`	`+ "github.com/envoyproxy/ai-gateway/internal/testing/testotel"`
`18`	`18`	`)`
`19`	`19`
`20`	`20`	`func TestChatCompletionSpan_RecordChunk(t *testing.T) {`
Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,8 @@ import (`
`13`	`13`
`14`	`14`	`"github.com/stretchr/testify/require"`
`15`	`15`
	`16`	`+ "github.com/envoyproxy/ai-gateway/internal/testing/testotel"`
`16`	`17`	`"github.com/envoyproxy/ai-gateway/tests/internal/testenvironment"`
`17`		`- "github.com/envoyproxy/ai-gateway/tests/testotel"`
`18`	`18`	`)`
`19`	`19`
`20`	`20`	`// otelTestEnvironment holds all the services needed for OTEL tests.`