From 5578211253d8b963fff14ca41d1933d00e03b3d6 Mon Sep 17 00:00:00 2001 From: Jan Chaloupka Date: Sun, 14 Dec 2025 12:24:22 +0100 Subject: [PATCH] refactor(TestFindDuplicatePods): have a pod fully created through BuildTestPod without any edits --- .../plugins/removeduplicates/removeduplicates_test.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/pkg/framework/plugins/removeduplicates/removeduplicates_test.go b/pkg/framework/plugins/removeduplicates/removeduplicates_test.go index 9e4ec06bf..28db8a66a 100644 --- a/pkg/framework/plugins/removeduplicates/removeduplicates_test.go +++ b/pkg/framework/plugins/removeduplicates/removeduplicates_test.go @@ -73,8 +73,10 @@ func TestFindDuplicatePods(t *testing.T) { pod.Namespace = "dev" pod.ObjectMeta.OwnerReferences = ownerRef1 }) - p2 := test.BuildTestPod("p2", 100, 0, node1.Name, nil) - p2.Namespace = "dev" + p2 := test.BuildTestPod("p2", 100, 0, node1.Name, func(pod *v1.Pod) { + pod.Namespace = "dev" + pod.ObjectMeta.OwnerReferences = ownerRef1 + }) p3 := test.BuildTestPod("p3", 100, 0, node1.Name, nil) p3.Namespace = "dev" p4 := test.BuildTestPod("p4", 100, 0, node1.Name, nil) @@ -116,7 +118,6 @@ func TestFindDuplicatePods(t *testing.T) { // ### Evictable Pods ### // Three Pods in the "default" Namespace, bound to same ReplicaSet. 2 should be evicted. - p2.ObjectMeta.OwnerReferences = ownerRef1 p3.ObjectMeta.OwnerReferences = ownerRef1 // Three Pods in the "test" Namespace, bound to same ReplicaSet. 2 should be evicted.