tests: add mockery for Konnect SDK

.gitattributes

@@ -1,2 +1,3 @@
 **/zz_generated*.go linguist-generated=true
 pkg/clientset/** linguist-generated=true
+controller/konnect/ops/*_mock_test.go linguist-generated=true

.golangci.yaml

@@ -62,6 +62,13 @@ linters-settings:
 
       - pkg: github.com/kong/gateway-operator/internal/types
         alias: gwtypes
+
+      - pkg: github.com/Kong/sdk-konnect-go/models/components
+        alias: sdkkonnectgocomp
+      - pkg: github.com/Kong/sdk-konnect-go/models/operations
+        alias: sdkkonnectgoops
+      - pkg: github.com/Kong/sdk-konnect-go/models/sdkerrors
+        alias: sdkkonnectgoerrs
   revive:
     rules:
       - name: errorf

.mockery.yaml

@@ -0,0 +1,14 @@
+quiet: False
+inpackage: True
+disable-version-string: True
+with-expecter: True
+
+filename: "{{.InterfaceName | lower}}_mock_test.go"
+dir: "{{.InterfaceDir}}"
+mockname: "Mock{{.InterfaceName}}"
+outpkg: "{{.PackageName}}"
+
+packages:
+  github.com/kong/gateway-operator/controller/konnect/ops:
+    interfaces:
+      ControlPlaneSDK:

.tools_versions.yaml

@@ -14,3 +14,5 @@ dlv: "1.23.0"
 gotestsum: "1.12.0"
 # renovate: datasource=github-releases depName=elastic/crd-ref-docs
 crd-ref-docs: "0.1.0"
+# renovate: datasource=github-releases depName=vektra/mockery
+mockery: "2.44.2"

Makefile

@@ -126,6 +126,13 @@ skaffold: mise yq ## Download skaffold locally if necessary.
 	@$(MISE) plugin install --yes -q skaffold
 	@$(MISE) install -q skaffold@$(SKAFFOLD_VERSION)
 
+MOCKERY_VERSION = $(shell $(YQ) -r '.mockery' < $(TOOLS_VERSIONS_FILE))
+MOCKERY = $(PROJECT_DIR)/bin/installs/mockery/$(MOCKERY_VERSION)/bin/mockery
+.PHONY: mockery
+mockery: mise yq ## Download mockery locally if necessary.
+	@$(MISE) plugin install --yes -q mockery https://github.com/cabify/asdf-mockery.git
+	@$(MISE) install -q mockery@$(MOCKERY_VERSION)
+
 # ------------------------------------------------------------------------------
 # Build
 # ------------------------------------------------------------------------------
@@ -191,7 +198,7 @@ verify.generators: verify.repo generate verify.diff
 API_DIR ?= api
 
 .PHONY: generate
-generate: generate.api generate.clientsets generate.rbacs generate.gateway-api-urls generate.docs generate.k8sio-gomod-replace generate.testcases-registration generate.kic-webhook-config
+generate: generate.api generate.clientsets generate.rbacs generate.gateway-api-urls generate.docs generate.k8sio-gomod-replace generate.testcases-registration generate.kic-webhook-config generate.mocks
 
 .PHONY: generate.api
 generate.api: controller-gen
@@ -388,6 +395,11 @@ test.conformance:
 test.samples: kustomize
 	find ./config/samples -not -name "kustomization.*" -type f | sort | xargs -I{} bash -c "kubectl apply -f {}; kubectl delete -f {}"
 
+# https://github.com/vektra/mockery/issues/803#issuecomment-2287198024
+.PHONY: generate.mocks
+generate.mocks: mockery
+	GODEBUG=gotypesalias=0 $(MOCKERY)
+
 # ------------------------------------------------------------------------------
 # Gateway API
 # ------------------------------------------------------------------------------

controller/konnect/conditions.go → controller/konnect/conditions/conditions.go

@@ -1,4 +1,4 @@
-package konnect
+package conditions
 
 // TODO(pmalek): move this to Konnect API directory so that it's part of the API contract.
 // https://github.com/Kong/kubernetes-configuration/issues/14

controller/konnect/constraints.go → controller/konnect/constraints/constraints.go

@@ -1,4 +1,4 @@
-package konnect
+package constraints
 
 import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -24,9 +24,9 @@ type SupportedKonnectEntityType interface {
 }
 
 // EntityType is an interface that all Konnect entity types must implement.
-// Separating this from SupportedKonnectEntityType allows us to use EntityType
+// Separating this from constraints.SupportedKonnectEntityType allows us to use EntityType
 // where client.Object is required, since it embeds client.Object and uses pointer
-// to refer to the SupportedKonnectEntityType.
+// to refer to the constraints.SupportedKonnectEntityType.
 type EntityType[T SupportedKonnectEntityType] interface {
 	*T
 	// Kubernetes Object methods

controller/konnect/constraints/entitytypename.go

@@ -0,0 +1,7 @@
+package constraints
+
+// EntityTypeName returns the name of the entity type.
+func EntityTypeName[T SupportedKonnectEntityType]() string {
+	var e T
+	return e.GetTypeName()
+}

controller/konnect/errors.go

@@ -6,25 +6,6 @@ import (
 	"k8s.io/apimachinery/pkg/types"
 )
 
-// FailedKonnectOpError is an error type that is returned when an operation against
-// Konnect API fails.
-type FailedKonnectOpError[T SupportedKonnectEntityType] struct {
-	Op  Op
-	Err error
-}
-
-// Error implements the error interface.
-func (e FailedKonnectOpError[T]) Error() string {
-	return fmt.Sprintf("failed to %s %s on Konnect: %v",
-		e.Op, entityTypeName[T](), e.Err,
-	)
-}
-
-// Unwrap returns the underlying error.
-func (e FailedKonnectOpError[T]) Unwrap() error {
-	return e.Err
-}
-
 // ReferencedControlPlaneDoesNotExistError is an error type that is returned when
 // a Konnect entity references a KonnectControlPlane that does not exist.
 type ReferencedControlPlaneDoesNotExistError struct {

controller/konnect/ops/controlplane.go

@@ -0,0 +1,15 @@
+package ops
+
+import (
+	"context"
+
+	sdkkonnectgocomp "github.com/Kong/sdk-konnect-go/models/components"
+	sdkkonnectgoops "github.com/Kong/sdk-konnect-go/models/operations"
+)
+
+// ControlPlaneSDK is the interface for the Konnect ControlPlaneSDK SDK.
+type ControlPlaneSDK interface {
+	CreateControlPlane(ctx context.Context, req sdkkonnectgocomp.CreateControlPlaneRequest, opts ...sdkkonnectgoops.Option) (*sdkkonnectgoops.CreateControlPlaneResponse, error)
+	DeleteControlPlane(ctx context.Context, id string, opts ...sdkkonnectgoops.Option) (*sdkkonnectgoops.DeleteControlPlaneResponse, error)
+	UpdateControlPlane(ctx context.Context, id string, req sdkkonnectgocomp.UpdateControlPlaneRequest, opts ...sdkkonnectgoops.Option) (*sdkkonnectgoops.UpdateControlPlaneResponse, error)
+}