From fabaf35c7297fb4a4f963b6292c5dd920f468010 Mon Sep 17 00:00:00 2001
From: monopole <jeff.regan@gmail.com>
Date: Fri, 15 Jan 2021 06:09:52 -0800
Subject: [PATCH] Add test covering pod disruption budget treatment.

---
 ...24_test.go => poddisruptionbudget_test.go} | 52 ++++++++++++++++++-
 1 file changed, 50 insertions(+), 2 deletions(-)
 rename api/krusty/{issue3424_test.go => poddisruptionbudget_test.go} (53%)

diff --git a/api/krusty/issue3424_test.go b/api/krusty/poddisruptionbudget_test.go
similarity index 53%
rename from api/krusty/issue3424_test.go
rename to api/krusty/poddisruptionbudget_test.go
index 13dbf09c6..07e0be8e3 100644
--- a/api/krusty/issue3424_test.go
+++ b/api/krusty/poddisruptionbudget_test.go
@@ -10,8 +10,52 @@ import (
 	kusttest_test "sigs.k8s.io/kustomize/api/testutils/kusttest"
 )
 
-// Demonstrate unwanted quotes added to an integer by a strategic merge patch.
-func TestIssue3424Basics(t *testing.T) {
+func TestPodDisruptionBudgetBasics(t *testing.T) {
+	th := kusttest_test.MakeHarness(t)
+	th.WriteF("pdbLiteral.yaml", `
+apiVersion: policy/v1beta1
+kind: PodDisruptionBudget
+metadata:
+  name: pdbLiteral
+spec:
+  maxUnavailable: 90
+`)
+	th.WriteF("pdbPercentage.yaml", `
+apiVersion: policy/v1beta1
+kind: PodDisruptionBudget
+metadata:
+  name: pdbPercentage
+spec:
+  maxUnavailable: 90%
+`)
+	th.WriteK(".", `
+resources:
+- pdbLiteral.yaml
+- pdbPercentage.yaml
+`)
+	m := th.Run(".", th.MakeDefaultOptions())
+	// In a PodDisruptionBudget, the fields maxUnavailable
+	// minAvailable are mutually exclusive, and both can hold
+	// either an integer, i.e. 10, or string that has to be
+	// an int followed by a percent sign, e.g. 10%.
+	th.AssertActualEqualsExpected(m, `
+apiVersion: policy/v1beta1
+kind: PodDisruptionBudget
+metadata:
+  name: pdbLiteral
+spec:
+  maxUnavailable: 90
+---
+apiVersion: policy/v1beta1
+kind: PodDisruptionBudget
+metadata:
+  name: pdbPercentage
+spec:
+  maxUnavailable: 90%
+`)
+}
+
+func TestPodDisruptionBudgetMerging(t *testing.T) {
 	th := kusttest_test.MakeHarness(t)
 	opts := th.MakeDefaultOptions()
 	th.WriteF("pdb-patch.yaml", `
@@ -73,6 +117,10 @@ metadata:
 spec:
   maxUnavailable: %s
 `
+	// In a PodDisruptionBudget, the fields maxUnavailable
+	// minAvailable are mutually exclusive, and both can hold
+	// either an integer, i.e. 10, or string that has to be
+	// an int followed by a percent sign, e.g. 10%.
 	th.AssertActualEqualsExpected(m, opts.IfApiMachineryElseKyaml(
 		fmt.Sprintf(expFmt, `"1"`, `"1"`),
 		fmt.Sprintf(expFmt, `1`, `1`)))