OMG goland do your go fmt thing

Signed-off-by: Raffaele Di Fazio <raffo@github.com>
kubernetes-sigs · Nov 22, 2020 · 904bf83 · 904bf83
1 parent 6803d57
commit 904bf83
Show file tree

Hide file tree

Showing 5 changed files with 34 additions and 35 deletions.
diff --git a/plan/plan_test.go b/plan/plan_test.go
@@ -688,15 +688,15 @@ func TestNormalizeDNSName(t *testing.T) {
 }
 
 func TestShouldUpdateProviderSpecific(tt *testing.T) {
-	comparator := func (name, previous, current string) bool {
+	comparator := func(name, previous, current string) bool {
 		return previous == current
 	}
 	for _, test := range []struct {
-		name string
-		current *endpoint.Endpoint
-		desired *endpoint.Endpoint
+		name               string
+		current            *endpoint.Endpoint
+		desired            *endpoint.Endpoint
 		propertyComparator func(name, previous, current string) bool
-		shouldUpdate bool
+		shouldUpdate       bool
 	}{
 		{
 			name: "skip AWS target health",
@@ -713,7 +713,7 @@ func TestShouldUpdateProviderSpecific(tt *testing.T) {
 				},
 			},
 			propertyComparator: comparator,
-			shouldUpdate: false,
+			shouldUpdate:       false,
 		},
 		{
 			name: "custom property unchanged",
@@ -728,7 +728,7 @@ func TestShouldUpdateProviderSpecific(tt *testing.T) {
 				},
 			},
 			propertyComparator: comparator,
-			shouldUpdate: false,
+			shouldUpdate:       false,
 		},
 		{
 			name: "custom property value changed",
@@ -743,7 +743,7 @@ func TestShouldUpdateProviderSpecific(tt *testing.T) {
 				},
 			},
 			propertyComparator: comparator,
-			shouldUpdate: true,
+			shouldUpdate:       true,
 		},
 		{
 			name: "custom property key changed",
@@ -758,7 +758,7 @@ func TestShouldUpdateProviderSpecific(tt *testing.T) {
 				},
 			},
 			propertyComparator: comparator,
-			shouldUpdate: true,
+			shouldUpdate:       true,
 		},
 		{
 			name: "desired has same key and value as current but not comparator is set",
@@ -805,8 +805,8 @@ func TestShouldUpdateProviderSpecific(tt *testing.T) {
 	} {
 		tt.Run(test.name, func(t *testing.T) {
 			plan := &Plan{
-				Current:  []*endpoint.Endpoint{test.current},
-				Desired:  []*endpoint.Endpoint{test.desired},
+				Current:            []*endpoint.Endpoint{test.current},
+				Desired:            []*endpoint.Endpoint{test.desired},
 				PropertyComparator: test.propertyComparator,
 			}
 			b := plan.shouldUpdateProviderSpecific(test.desired, test.current)

diff --git a/provider/aws/aws.go b/provider/aws/aws.go
@@ -205,7 +205,6 @@ func NewAWSProvider(awsConfig AWSConfig) (*AWSProvider, error) {
 	return provider, nil
 }
 
-
 func (p *AWSProvider) PropertyValuesEqual(name string, previous string, current string) bool {
 	if name == "aws/evaluate-target-health" {
 		return true

diff --git a/provider/aws/aws_test.go b/provider/aws/aws_test.go
@@ -1101,41 +1101,41 @@ func TestAWSSuitableZones(t *testing.T) {
 }
 
 func TestAWSHealthTargetAnnotation(tt *testing.T) {
-	comparator := func (name, previous, current string) bool {
+	comparator := func(name, previous, current string) bool {
 		return previous == current
 	}
 	for _, test := range []struct {
-		name string
-		current *endpoint.Endpoint
-		desired *endpoint.Endpoint
+		name               string
+		current            *endpoint.Endpoint
+		desired            *endpoint.Endpoint
 		propertyComparator func(name, previous, current string) bool
-		shouldUpdate bool
+		shouldUpdate       bool
 	}{
 		{
 			name: "skip AWS target health",
 			current: &endpoint.Endpoint{
 				RecordType: "A",
-				DNSName: "foo.com",
+				DNSName:    "foo.com",
 				ProviderSpecific: []endpoint.ProviderSpecificProperty{
 					{Name: "aws/evaluate-target-health", Value: "true"},
 				},
 			},
 			desired: &endpoint.Endpoint{
-				DNSName: "foo.com",
+				DNSName:    "foo.com",
 				RecordType: "A",
 				ProviderSpecific: []endpoint.ProviderSpecificProperty{
 					{Name: "aws/evaluate-target-health", Value: "false"},
 				},
 			},
 			propertyComparator: comparator,
-			shouldUpdate: false,
+			shouldUpdate:       false,
 		},
 	} {
 		tt.Run(test.name, func(t *testing.T) {
 			provider := &AWSProvider{}
 			plan := &plan.Plan{
-				Current:  []*endpoint.Endpoint{test.current},
-				Desired:  []*endpoint.Endpoint{test.desired},
+				Current:            []*endpoint.Endpoint{test.current},
+				Desired:            []*endpoint.Endpoint{test.desired},
 				PropertyComparator: provider.PropertyValuesEqual,
 			}
 			plan = plan.Calculate()

diff --git a/source/ingress_test.go b/source/ingress_test.go
@@ -997,31 +997,31 @@ func testIngressEndpoints(t *testing.T) {
 			},
 		},
 		{
-			title:                    "ignore tls section",
-			targetNamespace:          "",
-			ignoreIngressTLSSpec:     true,
+			title:                "ignore tls section",
+			targetNamespace:      "",
+			ignoreIngressTLSSpec: true,
 			ingressItems: []fakeIngress{
 				{
-					name:      "fake1",
-					namespace: namespace,
+					name:        "fake1",
+					namespace:   namespace,
 					tlsdnsnames: [][]string{{"example.org"}},
 					ips:         []string{"1.2.3.4"},
 				},
-		  },
+			},
 			expected: []*endpoint.Endpoint{},
 		},
 		{
-			title:                    "reading tls section",
-			targetNamespace:          "",
-			ignoreIngressTLSSpec:     false,
+			title:                "reading tls section",
+			targetNamespace:      "",
+			ignoreIngressTLSSpec: false,
 			ingressItems: []fakeIngress{
 				{
-					name:      "fake1",
-					namespace: namespace,
+					name:        "fake1",
+					namespace:   namespace,
 					tlsdnsnames: [][]string{{"example.org"}},
 					ips:         []string{"1.2.3.4"},
 				},
-		  },
+			},
 			expected: []*endpoint.Endpoint{
 				{
 					DNSName: "example.org",

diff --git a/source/store_test.go b/source/store_test.go
@@ -157,5 +157,5 @@ func TestByNames(t *testing.T) {
 }
 
 var minimalConfig = &Config{
-	ContourLoadBalancerService:  "heptio-contour/contour",
+	ContourLoadBalancerService: "heptio-contour/contour",
 }