cleanup: remove _test suffix

Renames harness and utility files (e.g., harness_test.go -> harness.go)
within the EPP integration test suite to align with project conventions.

Additionally moves shared global variables (k8sClient, testEnv) to
harness.go to ensure visibility for the renamed files, resolving
compiler scope issues that arise when separating test logic from
infrastructure code.

Author: LukeAVanDrie

test/integration/bbr/harness_test.go test/integration/bbr/harness.gotest/integration/bbr/harness_test.go renamed to test/integration/bbr/harness.go

@@ -27,15 +27,21 @@ import (
 	"github.com/google/uuid"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
+	"go.uber.org/zap/zapcore"
 	"google.golang.org/grpc"
 	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
+	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
 	"k8s.io/apimachinery/pkg/types"
 	metricsutils "k8s.io/component-base/metrics/testutil"
 	ctrl "sigs.k8s.io/controller-runtime"
 	"sigs.k8s.io/controller-runtime/pkg/cache"
+	"sigs.k8s.io/controller-runtime/pkg/client"
 	crconfig "sigs.k8s.io/controller-runtime/pkg/config"
+	"sigs.k8s.io/controller-runtime/pkg/envtest"
+	"sigs.k8s.io/controller-runtime/pkg/log/zap"
 	crmetrics "sigs.k8s.io/controller-runtime/pkg/metrics"
 	metricsserver "sigs.k8s.io/controller-runtime/pkg/metrics/server"
 
@@ -53,10 +59,22 @@ import (
 	"sigs.k8s.io/gateway-api-inference-extension/pkg/epp/scheduling/framework/plugins/profile"
 	"sigs.k8s.io/gateway-api-inference-extension/pkg/epp/scheduling/framework/plugins/scorer"
 	"sigs.k8s.io/gateway-api-inference-extension/pkg/epp/server"
+	logutil "sigs.k8s.io/gateway-api-inference-extension/pkg/epp/util/logging"
 	epptestutil "sigs.k8s.io/gateway-api-inference-extension/pkg/epp/util/testing"
 	"sigs.k8s.io/gateway-api-inference-extension/test/integration"
 )
 
+// Global State (Initialized in TestMain)
+var (
+	k8sClient     client.Client
+	testEnv       *envtest.Environment
+	testScheme    = runtime.NewScheme()
+	logger        = zap.New(zap.UseDevMode(true), zap.Level(zapcore.Level(logutil.DEFAULT)))
+	baseResources []*unstructured.Unstructured
+)
+
+const testPoolName = "vllm-llama3-8b-instruct-pool"
+
 // TestHarness encapsulates the environment for a single isolated EPP test run.
 // It manages the lifecycle of the controller manager, the EPP server, and the K8s namespace.
 type TestHarness struct {

test/integration/bbr/util_test.go test/integration/bbr/util.gotest/integration/bbr/util_test.go renamed to test/integration/bbr/util.go

@@ -30,29 +30,24 @@ import (
 	envoyTypePb "github.com/envoyproxy/go-control-plane/envoy/type/v3"
 	"github.com/google/go-cmp/cmp"
 	"github.com/stretchr/testify/require"
-	"go.uber.org/zap/zapcore"
 	"google.golang.org/protobuf/testing/protocmp"
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
-	"k8s.io/apimachinery/pkg/runtime"
 	utilruntime "k8s.io/apimachinery/pkg/util/runtime"
 	"k8s.io/apimachinery/pkg/util/yaml"
 	clientgoscheme "k8s.io/client-go/kubernetes/scheme"
 	ctrl "sigs.k8s.io/controller-runtime"
 	"sigs.k8s.io/controller-runtime/pkg/client"
 	"sigs.k8s.io/controller-runtime/pkg/envtest"
-	"sigs.k8s.io/controller-runtime/pkg/log/zap"
 
 	v1 "sigs.k8s.io/gateway-api-inference-extension/api/v1"
 	"sigs.k8s.io/gateway-api-inference-extension/apix/v1alpha2"
 	"sigs.k8s.io/gateway-api-inference-extension/pkg/epp/metadata"
 	"sigs.k8s.io/gateway-api-inference-extension/pkg/epp/metrics"
-	logutil "sigs.k8s.io/gateway-api-inference-extension/pkg/epp/util/logging"
 	requtil "sigs.k8s.io/gateway-api-inference-extension/pkg/epp/util/request"
 	"sigs.k8s.io/gateway-api-inference-extension/test/integration"
 )
 
 const (
-	testPoolName         = "vllm-llama3-8b-instruct-pool"
 	modelMyModel         = "my-model"
 	modelMyModelTarget   = "my-model-12345"
 	modelSQLLora         = "sql-lora"
@@ -64,15 +59,6 @@ const (
 	modelAfterRewrite    = "rewritten-model"
 )
 
-// Global State (Initialized in TestMain)
-var (
-	k8sClient     client.Client
-	testEnv       *envtest.Environment
-	testScheme    = runtime.NewScheme()
-	logger        = zap.New(zap.UseDevMode(true), zap.Level(zapcore.Level(logutil.DEFAULT)))
-	baseResources []*unstructured.Unstructured
-)
-
 func TestMain(m *testing.M) {
 	ctrl.SetLogger(logger)
 
@@ -446,17 +432,3 @@ func loadBaseResources() []*unstructured.Unstructured {
 	}
 	return objs
 }
-
-// cleanMetric removes indentation from multiline metric strings and ensures a trailing newline exists, which is
-// required by the Prometheus text parser.
-func cleanMetric(s string) string {
-	lines := strings.Split(s, "\n")
-	var cleaned []string
-	for _, l := range lines {
-		trimmed := strings.TrimSpace(l)
-		if trimmed != "" {
-			cleaned = append(cleaned, trimmed)
-		}
-	}
-	return strings.Join(cleaned, "\n") + "\n"
-}

test/integration/epp/harness_test.go test/integration/epp/harness.gotest/integration/epp/harness_test.go renamed to test/integration/epp/harness.go

@@ -180,3 +180,17 @@ func metricReadyPods(count int) string {
 		inference_pool_ready_pods{%s} %d
 		`, labelsToString([]label{{"name", testPoolName}}), count)
 }
+
+// cleanMetric removes indentation from multiline metric strings and ensures a trailing newline exists, which is
+// required by the Prometheus text parser.
+func cleanMetric(s string) string {
+	lines := strings.Split(s, "\n")
+	var cleaned []string
+	for _, l := range lines {
+		trimmed := strings.TrimSpace(l)
+		if trimmed != "" {
+			cleaned = append(cleaned, trimmed)
+		}
+	}
+	return strings.Join(cleaned, "\n") + "\n"
+}