From 791e7f9d0bdd1163e381deeb337d5eafcafd7dba Mon Sep 17 00:00:00 2001 From: Shin'ya UEOKA Date: Thu, 28 Oct 2021 05:20:03 +0000 Subject: [PATCH] test: Assert max unavailable for PDB test cases Kubernetes-commit: 5c76507c3f9dd84dd3a56b39a45f99da1735bfed --- pkg/cmd/create/create_pdb_test.go | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/pkg/cmd/create/create_pdb_test.go b/pkg/cmd/create/create_pdb_test.go index 40537875..614c7f6e 100644 --- a/pkg/cmd/create/create_pdb_test.go +++ b/pkg/cmd/create/create_pdb_test.go @@ -132,8 +132,8 @@ func TestCreatePdb(t *testing.T) { minAvailableNumber := intstr.Parse(podAmountNumber) minAvailablePercent := intstr.Parse(podAmountPercent) - minUnavailableNumber := intstr.Parse(podAmountNumber) - minUnavailablePercent := intstr.Parse(podAmountPercent) + maxUnavailableNumber := intstr.Parse(podAmountNumber) + maxUnavailablePercent := intstr.Parse(podAmountPercent) tests := map[string]struct { options *PodDisruptionBudgetOpts @@ -181,9 +181,9 @@ func TestCreatePdb(t *testing.T) { }, "test-valid-max-unavailable-pods-number": { options: &PodDisruptionBudgetOpts{ - Name: "my-pdb", - Selector: selectorOpts, - MinAvailable: podAmountNumber, + Name: "my-pdb", + Selector: selectorOpts, + MaxUnavailable: podAmountNumber, }, expected: &policyv1.PodDisruptionBudget{ TypeMeta: metav1.TypeMeta{ @@ -194,16 +194,16 @@ func TestCreatePdb(t *testing.T) { Name: "my-pdb", }, Spec: policyv1.PodDisruptionBudgetSpec{ - Selector: selector, - MinAvailable: &minUnavailableNumber, + Selector: selector, + MaxUnavailable: &maxUnavailableNumber, }, }, }, "test-valid-max-unavailable-pods-percentage": { options: &PodDisruptionBudgetOpts{ - Name: "my-pdb", - Selector: selectorOpts, - MinAvailable: podAmountPercent, + Name: "my-pdb", + Selector: selectorOpts, + MaxUnavailable: podAmountPercent, }, expected: &policyv1.PodDisruptionBudget{ TypeMeta: metav1.TypeMeta{ @@ -214,8 +214,8 @@ func TestCreatePdb(t *testing.T) { Name: "my-pdb", }, Spec: policyv1.PodDisruptionBudgetSpec{ - Selector: selector, - MinAvailable: &minUnavailablePercent, + Selector: selector, + MaxUnavailable: &maxUnavailablePercent, }, }, },