Moved function to place where it is used.

Author: lamai93

tests/predicates.go

@@ -25,6 +25,7 @@ package tests
 import (
 	"fmt"
 
+	v1 "k8s.io/api/core/v1"
 	api "github.com/arangodb/kube-arangodb/pkg/apis/deployment/v1alpha"
 	"github.com/arangodb/kube-arangodb/pkg/util/k8sutil"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -45,6 +46,22 @@ func deploymentIsReady() func(*api.ArangoDeployment) error {
 	}
 }
 
+func resourcesRequireRotation(wanted, given v1.ResourceRequirements) bool {
+	checkList := func(wanted, given v1.ResourceList) bool {
+		for k, v := range wanted {
+			if gv, ok := given[k]; !ok {
+				return true
+			} else if v.Cmp(gv) != 0 {
+				return true
+			}
+		}
+
+		return false
+	}
+
+	return checkList(wanted.Limits, given.Limits) || checkList(wanted.Requests, given.Requests)
+}
+
 func resourcesAsRequested(kubecli kubernetes.Interface, ns string) func(obj *api.ArangoDeployment) error {
 	return func(obj *api.ArangoDeployment) error {
 		return obj.ForeachServerGroup(func(group api.ServerGroup, spec api.ServerGroupSpec, status *api.MemberStatusList) error {

tests/resources_test.go

@@ -36,22 +36,6 @@ import (
 	"k8s.io/apimachinery/pkg/api/resource"
 )
 
-func resourcesRequireRotation(wanted, given v1.ResourceRequirements) bool {
-	checkList := func(wanted, given v1.ResourceList) bool {
-		for k, v := range wanted {
-			if gv, ok := given[k]; !ok {
-				return true
-			} else if v.Cmp(gv) != 0 {
-				return true
-			}
-		}
-
-		return false
-	}
-
-	return checkList(wanted.Limits, given.Limits) || checkList(wanted.Requests, given.Requests)
-}
-
 func TestResourcesChangeLimitsCluster(t *testing.T) {
 	longOrSkip(t)
 	c := client.MustNewInCluster()