Fix resource management in upstream tests by adding deferred index manager closure

Dumbris · Dumbris · commit 53fdb8047621 · 2025-06-24T15:37:42.000+03:00
diff --git a/internal/server/upstream_test.go b/internal/server/upstream_test.go
@@ -42,6 +42,7 @@ func TestUpstreamServersHandlerPerformance(t *testing.T) {
 	if err != nil {
 		t.Fatalf("Failed to create index manager: %v", err)
 	}
+	defer indexManager.Close()
 
 	// Create upstream manager
 	upstreamManager := upstream.NewManager(zap.NewNop())
@@ -140,6 +141,7 @@ func TestUpstreamServersListOperation(t *testing.T) {
 	if err != nil {
 		t.Fatalf("Failed to create index manager: %v", err)
 	}
+	defer indexManager.Close()
 
 	// Create upstream manager
 	upstreamManager := upstream.NewManager(zap.NewNop())

Original file line number	Diff line number	Diff line change
`@@ -42,6 +42,7 @@ func TestUpstreamServersHandlerPerformance(t *testing.T) {`
`42`	`42`	`if err != nil {`
`43`	`43`	`t.Fatalf("Failed to create index manager: %v", err)`
`44`	`44`	`}`
	`45`	`+ defer indexManager.Close()`
`45`	`46`
`46`	`47`	`// Create upstream manager`
`47`	`48`	`upstreamManager := upstream.NewManager(zap.NewNop())`
`@@ -140,6 +141,7 @@ func TestUpstreamServersListOperation(t *testing.T) {`
`140`	`141`	`if err != nil {`
`141`	`142`	`t.Fatalf("Failed to create index manager: %v", err)`
`142`	`143`	`}`
	`144`	`+ defer indexManager.Close()`
`143`	`145`
`144`	`146`	`// Create upstream manager`
`145`	`147`	`upstreamManager := upstream.NewManager(zap.NewNop())`