controller: rename secretclient to api since it contains more than se…

…cret utils now Signed-off-by: Muvaffak Onus <me@muvaf.com>
crossplane · Oct 11, 2021 · 1d6e10b · 1d6e10b
1 parent ca81a45
commit 1d6e10b
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 8 deletions.
diff --git a/pkg/controller/secretclient.go → pkg/controller/api.go b/pkg/controller/secretclient.go → pkg/controller/api.go
@@ -81,7 +81,7 @@ func (ac *APICallbacks) Apply(name string) terraform.CallbackFn {
 		if kErr := ac.kube.Get(ctx, nn, tr); kErr != nil {
 			return errors.Wrap(kErr, "cannot get Terraformed resource")
 		}
-		tr.SetConditions(resource.LastOperationCondition(err))
+		tr.SetConditions(resource.AsyncOperationCondition(err))
 		return errors.Wrap(ac.kube.Status().Update(ctx, tr), errStatusUpdate)
 	}
 }
@@ -95,7 +95,7 @@ func (ac *APICallbacks) Destroy(name string) terraform.CallbackFn {
 		if kErr := ac.kube.Get(ctx, nn, tr); kErr != nil {
 			return errors.Wrap(kErr, "cannot get Terraformed resource")
 		}
-		tr.SetConditions(resource.LastOperationCondition(err))
+		tr.SetConditions(resource.AsyncOperationCondition(err))
 		return errors.Wrap(ac.kube.Status().Update(ctx, tr), errStatusUpdate)
 	}
 }
diff --git a/pkg/controller/external.go b/pkg/controller/external.go
@@ -107,7 +107,7 @@ type external struct {
 	callback  CallbackProvider
 }
 
-func (e *external) Observe(ctx context.Context, mg xpresource.Managed) (managed.ExternalObservation, error) {
+func (e *external) Observe(ctx context.Context, mg xpresource.Managed) (managed.ExternalObservation, error) { //nolint:gocyclo
 	// We skip the gocyclo check because most of the operations are straight-forward
 	// and serial.
 	// TODO(muvaf): Look for ways to reduce the cyclomatic complexity without

diff --git a/pkg/controller/external_test.go b/pkg/controller/external_test.go
@@ -181,9 +181,8 @@ func TestConnect(t *testing.T) {
 
 func TestObserve(t *testing.T) {
 	type args struct {
-		w     Workspace
-		async bool
-		obj   xpresource.Managed
+		w   Workspace
+		obj xpresource.Managed
 	}
 	type want struct {
 		obs managed.ExternalObservation

diff --git a/pkg/resource/conditions.go b/pkg/resource/conditions.go
@@ -33,9 +33,9 @@ const (
 	ReasonSuccess        xpv1.ConditionReason = "Success"
 )
 
-// LastOperationCondition returns the condition depending on the content
+// AsyncOperationCondition returns the condition depending on the content
 // of the error.
-func LastOperationCondition(err error) xpv1.Condition {
+func AsyncOperationCondition(err error) xpv1.Condition {
 	switch {
 	case err == nil:
 		return xpv1.Condition{