feat: add default for new properties in Upstream

go-kong v0.4.0 introduces new properties to Upstream related to HTTPS Active healthc-checks. If these fields are not set in the default filler, it results in a false diff.
Kong · Apr 13, 2019 · 080200d · 080200d
1 parent 0cf17a2
commit 080200d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/utils/constants.go b/utils/constants.go
@@ -31,8 +31,10 @@ var (
 					Interval:     kong.Int(0),
 					Successes:    kong.Int(0),
 				},
-				HTTPPath: kong.String("/"),
-				Timeout:  kong.Int(1),
+				HTTPPath:               kong.String("/"),
+				HTTPSVerifyCertificate: kong.Bool(true),
+				Type:                   kong.String("http"),
+				Timeout:                kong.Int(1),
 				Unhealthy: &kong.Unhealthy{
 					HTTPFailures: kong.Int(0),
 					TCPFailures:  kong.Int(0),

diff --git a/utils/defaulter_test.go b/utils/defaulter_test.go
@@ -225,8 +225,10 @@ func TestUpstreamSetTest(t *testing.T) {
 							Interval:     kong.Int(0),
 							Successes:    kong.Int(0),
 						},
-						HTTPPath: kong.String("/"),
-						Timeout:  kong.Int(1),
+						HTTPPath:               kong.String("/"),
+						HTTPSVerifyCertificate: kong.Bool(true),
+						Type:                   kong.String("http"),
+						Timeout:                kong.Int(1),
 						Unhealthy: &kong.Unhealthy{
 							HTTPFailures: kong.Int(0),
 							TCPFailures:  kong.Int(0),
@@ -277,8 +279,10 @@ func TestUpstreamSetTest(t *testing.T) {
 							Interval:     kong.Int(1),
 							Successes:    kong.Int(0),
 						},
-						HTTPPath: kong.String("/"),
-						Timeout:  kong.Int(1),
+						HTTPPath:               kong.String("/"),
+						HTTPSVerifyCertificate: kong.Bool(true),
+						Type:                   kong.String("http"),
+						Timeout:                kong.Int(1),
 						Unhealthy: &kong.Unhealthy{
 							HTTPFailures: kong.Int(0),
 							TCPFailures:  kong.Int(0),