Run golangci-lint run --fix

This commit is entirely generated by earthly +reviewable

Signed-off-by: Nic Cope <[email protected]>

Author: negz

apis/common/v1/condition.go

@@ -126,6 +126,7 @@ func IsSystemConditionType(t ConditionType) bool {
 	case TypeReady, TypeSynced, TypeHealthy:
 		return true
 	}
+
 	return false
 }
 
@@ -150,6 +151,7 @@ type ConditionedStatus struct {
 func NewConditionedStatus(c ...Condition) *ConditionedStatus {
 	s := &ConditionedStatus{}
 	s.SetConditions(c...)
+
 	return s
 }
 
@@ -171,6 +173,7 @@ func (s *ConditionedStatus) GetCondition(ct ConditionType) Condition {
 func (s *ConditionedStatus) SetConditions(c ...Condition) {
 	for _, cond := range c {
 		exists := false
+
 		for i, existing := range s.Conditions {
 			if existing.Type != cond.Type {
 				continue
@@ -184,6 +187,7 @@ func (s *ConditionedStatus) SetConditions(c ...Condition) {
 			s.Conditions[i] = cond
 			exists = true
 		}
+
 		if !exists {
 			s.Conditions = append(s.Conditions, cond)
 		}

apis/common/v1/connection_details.go

@@ -72,6 +72,7 @@ func (in *ConnectionSecretMetadata) SetOwnerUID(uid types.UID) {
 	if in.Labels == nil {
 		in.Labels = map[string]string{}
 	}
+
 	in.Labels[LabelKeyOwnerUID] = string(uid)
 }
 
@@ -80,6 +81,7 @@ func (in *ConnectionSecretMetadata) GetOwnerUID() string {
 	if u, ok := in.Labels[LabelKeyOwnerUID]; ok {
 		return u
 	}
+
 	return ""
 }
 

apis/common/v1/merge.go

@@ -40,9 +40,11 @@ func (mo *MergeOptions) MergoConfiguration() []func(*mergo.Config) {
 	if mo.KeepMapValues != nil && *mo.KeepMapValues {
 		config = config[:0]
 	}
+
 	if mo.AppendSlice != nil && *mo.AppendSlice {
 		config = append(config, mergo.WithAppendSlice)
 	}
+
 	return config
 }
 

apis/common/v1/merge_test.go

@@ -33,12 +33,15 @@ func (arr mergoOptArr) names() []string {
 	for i, opt := range arr {
 		names[i] = runtime.FuncForPC(reflect.ValueOf(opt).Pointer()).Name()
 	}
+
 	sort.Strings(names)
+
 	return names
 }
 
 func TestMergoConfiguration(t *testing.T) {
 	valTrue := true
+
 	tests := map[string]struct {
 		mo   *MergeOptions
 		want mergoOptArr

apis/common/v1/resource.go

@@ -101,6 +101,7 @@ func (p *Policy) IsResolutionPolicyOptional() bool {
 	if p == nil || p.Resolution == nil {
 		return false
 	}
+
 	return *p.Resolution == ResolutionPolicyOptional
 }
 
@@ -109,6 +110,7 @@ func (p *Policy) IsResolvePolicyAlways() bool {
 	if p == nil || p.Resolve == nil {
 		return false
 	}
+
 	return *p.Resolve == ResolvePolicyAlways
 }
 

pkg/certificates/certificates.go

@@ -36,12 +36,14 @@ const (
 func LoadMTLSConfig(caPath, certPath, keyPath string, isServer bool) (*tls.Config, error) {
 	tlsCertFilePath := filepath.Clean(certPath)
 	tlsKeyFilePath := filepath.Clean(keyPath)
+
 	certificate, err := tls.LoadX509KeyPair(tlsCertFilePath, tlsKeyFilePath)
 	if err != nil {
 		return nil, errors.Wrap(err, errLoadCert)
 	}
 
 	caCertFilePath := filepath.Clean(caPath)
+
 	ca, err := os.ReadFile(caCertFilePath)
 	if err != nil {
 		return nil, errors.Wrap(err, errLoadCA)

pkg/certificates/certificates_test.go

@@ -27,10 +27,12 @@ func TestLoad(t *testing.T) {
 		certsFolderPath         string
 		requireClientValidation bool
 	}
+
 	type want struct {
 		err error
 		out *tls.Config
 	}
+
 	cases := map[string]struct {
 		reason string
 		args
@@ -92,16 +94,16 @@ func TestLoad(t *testing.T) {
 	}
 	for name, tc := range cases {
 		t.Run(name, func(t *testing.T) {
-			certsFolderPath := tc.args.certsFolderPath
-			requireClient := tc.args.requireClientValidation
+			certsFolderPath := tc.certsFolderPath
+			requireClient := tc.requireClientValidation
 
 			cfg, err := LoadMTLSConfig(filepath.Join(certsFolderPath, caCertFileName), filepath.Join(certsFolderPath, tlsCertFileName), filepath.Join(certsFolderPath, tlsKeyFileName), requireClient)
-			if diff := cmp.Diff(tc.want.err, err, test.EquateErrors()); diff != "" {
+			if diff := cmp.Diff(tc.err, err, test.EquateErrors()); diff != "" {
 				t.Errorf("\n%s\nLoad(...): -want error, +got error:\n%s", tc.reason, diff)
 			}
 
 			if requireClient {
-				if diff := cmp.Diff(tc.want.out.ClientAuth, cfg.ClientAuth); diff != "" {
+				if diff := cmp.Diff(tc.out.ClientAuth, cfg.ClientAuth); diff != "" {
 					t.Errorf("\n%s\nLoad(...): -want, +got:\n%s", tc.reason, diff)
 				}
 			}

pkg/conditions/manager.go

@@ -68,5 +68,6 @@ func (c *observedGenerationPropagationConditionSet) MarkConditions(condition ...
 	for i := range condition {
 		condition[i].ObservedGeneration = c.o.GetGeneration()
 	}
+
 	c.o.SetConditions(condition...)
 }

pkg/conditions/manager_test.go

@@ -80,6 +80,7 @@ func TestOGConditionSetMark(t *testing.T) {
 			ut := newManaged(42, tt.start...)
 			c := manager.For(ut)
 			c.MarkConditions(tt.mark...)
+
 			if diff := cmp.Diff(tt.want, ut.Conditions, test.EquateConditions(), cmpopts.EquateApproxTime(1*time.Second)); diff != "" {
 				t.Errorf("\nReason: %s\n-want, +got:\n%s", tt.reason, diff)
 			}
@@ -129,5 +130,6 @@ func newManaged(generation int64, conditions ...xpv1.Condition) *fake.Managed {
 	mg := &fake.Managed{}
 	mg.Generation = generation
 	mg.SetConditions(conditions...)
+
 	return mg
 }

pkg/connection/fake/mocks.go

@@ -74,10 +74,13 @@ func (s *StoreConfig) GetObjectKind() schema.ObjectKind {
 // DeepCopyObject returns a copy of the object as runtime.Object.
 func (s *StoreConfig) DeepCopyObject() runtime.Object {
 	out := &StoreConfig{}
+
 	j, err := json.Marshal(s)
 	if err != nil {
 		panic(err)
 	}
+
 	_ = json.Unmarshal(j, out)
+
 	return out
 }