From 176c07dfddbd20994a438a0ce35a68766990cd81 Mon Sep 17 00:00:00 2001 From: Gabe Rosenhouse Date: Wed, 21 Apr 2021 23:50:58 +0000 Subject: [PATCH] fix up test for setting default namespace labels --- pkg/apis/core/v1/defaults_test.go | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/pkg/apis/core/v1/defaults_test.go b/pkg/apis/core/v1/defaults_test.go index b20fee58f61..eb0f00c654d 100644 --- a/pkg/apis/core/v1/defaults_test.go +++ b/pkg/apis/core/v1/defaults_test.go @@ -1417,9 +1417,6 @@ func TestSetDefaultNamespace(t *testing.T) { } func TestSetDefaultNamespaceLabels(t *testing.T) { - // Although this is defaulted to true, it's still worth to enable the feature gate during the test - defer featuregatetesting.SetFeatureGateDuringTest(t, utilfeature.DefaultFeatureGate, features.NamespaceDefaultLabelName, true)() - theNs := "default-ns-labels-are-great" s := &v1.Namespace{ ObjectMeta: metav1.ObjectMeta{ @@ -1432,22 +1429,6 @@ func TestSetDefaultNamespaceLabels(t *testing.T) { if s2.ObjectMeta.Labels[v1.LabelMetadataName] != theNs { t.Errorf("Expected default namespace label value of %v, but got %v", theNs, s2.ObjectMeta.Labels[v1.LabelMetadataName]) } - - // And let's disable the FG and check if it still defaults creating the labels - defer featuregatetesting.SetFeatureGateDuringTest(t, utilfeature.DefaultFeatureGate, features.NamespaceDefaultLabelName, false)() - - theNs = "default-ns-labels-are-not-that-great" - s = &v1.Namespace{ - ObjectMeta: metav1.ObjectMeta{ - Name: theNs, - }, - } - obj2 = roundTrip(t, runtime.Object(s)) - s2 = obj2.(*v1.Namespace) - - if _, ok := s2.ObjectMeta.Labels[v1.LabelMetadataName]; ok { - t.Errorf("Default namespace shouldn't exist here, as the feature gate is disabled %v", s) - } } func TestSetDefaultPodSpecHostNetwork(t *testing.T) {