From d590e1961d9bd9121a63861912682d85a3d1dff7 Mon Sep 17 00:00:00 2001 From: Daniel Lipovetsky Date: Wed, 20 Mar 2024 09:06:59 -0700 Subject: [PATCH] test: Remove redundant test case from httpproxy handler The "generic worker" test case already covers this. --- .../httpproxy/tests/generate_patches.go | 28 ------------------- 1 file changed, 28 deletions(-) diff --git a/pkg/handlers/generic/mutation/httpproxy/tests/generate_patches.go b/pkg/handlers/generic/mutation/httpproxy/tests/generate_patches.go index 61c6c5d34..79cb4f2e1 100644 --- a/pkg/handlers/generic/mutation/httpproxy/tests/generate_patches.go +++ b/pkg/handlers/generic/mutation/httpproxy/tests/generate_patches.go @@ -30,34 +30,6 @@ func TestGeneratePatches( capitest.PatchTestDef{ Name: "unset variable", }, - capitest.PatchTestDef{ - Name: "http proxy set for KubeadmConfigTemplate default-worker", - Vars: []runtimehooksv1.Variable{ - capitest.VariableWithValue( - variableName, - v1alpha1.HTTPProxy{ - HTTP: "http://example.com", - HTTPS: "https://example.com", - AdditionalNo: []string{"no-proxy.example.com"}, - }, - variablePath..., - ), - capitest.VariableWithValue( - "builtin", - map[string]any{ - "machineDeployment": map[string]any{ - "class": "default-worker", - }, - }, - ), - }, - RequestItem: request.NewKubeadmConfigTemplateRequestItem(""), - ExpectedPatchMatchers: []capitest.JSONPatchMatcher{{ - Operation: "add", - Path: "/spec/template/spec/files", - ValueMatcher: gomega.HaveLen(2), - }}, - }, capitest.PatchTestDef{ Name: "http proxy set for KubeadmConfigTemplate generic worker", Vars: []runtimehooksv1.Variable{