fix: snyk errors and missing mock services (#540)

The build check were failing because of 3rd party dependency coming soon
from upctl deploy starter-kit.
Changes in the sdk made the mock service used in the testing to fail
because some classes where missing.
list_test.go, part of the server tests was failing because of changes
made for the floating ip address in the sdk.
I manually upgraded to latest sdk release

Author: paketeserrano

.snyk

@@ -0,0 +1,10 @@
+version: v1.25.0
+ignore:
+  "snyk:lic:golang:github.com:hashicorp:errwrap:MPL-2.0":
+    - "*":
+        reason: "Allowed MPL-2.0 license for errwrap (transitive via Helm SDK for upctl stacks)"
+        expires: 2027-01-01T00:00:00.000Z
+  "snyk:lic:golang:github.com:hashicorp:go-multierror:MPL-2.0":
+    - "*":
+        reason: "Allowed MPL-2.0 license for go-multierror (transitive via Helm SDK for upctl stacks)"
+        expires: 2027-01-01T00:00:00.000Z

go.mod

@@ -5,7 +5,7 @@ go 1.25.0
 require (
 	github.com/UpCloudLtd/progress v1.0.3
 	github.com/UpCloudLtd/upcloud-go-api/credentials v0.1.1
-	github.com/UpCloudLtd/upcloud-go-api/v8 v8.26.0
+	github.com/UpCloudLtd/upcloud-go-api/v8 v8.28.0
 	github.com/adrg/xdg v0.5.3
 	github.com/asaskevich/govalidator v0.0.0-20210307081110-f21760c49a8d
 	github.com/gemalto/flume v1.0.0

go.sum

@@ -399,8 +399,8 @@ github.com/UpCloudLtd/progress v1.0.3 h1:8SfntHkBPyQc5BL3946Bgi9KYnQOxa5RR2EKdad
 github.com/UpCloudLtd/progress v1.0.3/go.mod h1:iGxOnb9HvHW0yrLGUjHr0lxHhn7TehgWwh7a8NqK6iQ=
 github.com/UpCloudLtd/upcloud-go-api/credentials v0.1.1 h1:eTfQsv58ufALOk9BZ7WbS/i7pMUD11RnYYpRPsz0LdI=
 github.com/UpCloudLtd/upcloud-go-api/credentials v0.1.1/go.mod h1:7OtVs2UqtfvjkC1HfE+Oud0MnbMv7qUWnbEgxnTAqts=
-github.com/UpCloudLtd/upcloud-go-api/v8 v8.26.0 h1:CmbgIINYPiQ0EIwihpUtqz9PZQEiNQfScNwvQKIJEBQ=
-github.com/UpCloudLtd/upcloud-go-api/v8 v8.26.0/go.mod h1:NBh1d/ip1bhdAIhuPWbyPme7tbLzDTV7dhutUmU1vg8=
+github.com/UpCloudLtd/upcloud-go-api/v8 v8.28.0 h1:51v7vnzSlTtEHP7lzyudLRgEQqd53+SADCrUaSX/Zdg=
+github.com/UpCloudLtd/upcloud-go-api/v8 v8.28.0/go.mod h1:NBh1d/ip1bhdAIhuPWbyPme7tbLzDTV7dhutUmU1vg8=
 github.com/adrg/xdg v0.5.3 h1:xRnxJXne7+oWDatRhR1JLnvuccuIeCoBu2rtuLqQB78=
 github.com/adrg/xdg v0.5.3/go.mod h1:nlTsY+NNiCBGCK2tpm09vRqfVzrc2fLmXGpBLF0zlTQ=
 github.com/ansel1/merry v1.5.0/go.mod h1:wUy/yW0JX0ix9GYvUbciq+bi3jW/vlKPlbpI7qdZpOw=

internal/commands/server/list_test.go

@@ -37,6 +37,7 @@ const expectedJSONOutput = `
           "family": "",
           "part_of_plan": "no",
           "ptr_record": "",
+		  "release_policy": "",
           "server": "",
           "mac": "",
           "floating": "yes",
@@ -49,6 +50,7 @@ const expectedJSONOutput = `
           "family": "",
           "part_of_plan": "no",
           "ptr_record": "",
+		  "release_policy": "",
           "server": "",
           "mac": "",
           "floating": "no",
@@ -61,6 +63,7 @@ const expectedJSONOutput = `
           "family": "",
           "part_of_plan": "no",
           "ptr_record": "",
+		  "release_policy": "",
           "server": "",
           "mac": "",
           "floating": "no",
@@ -73,6 +76,7 @@ const expectedJSONOutput = `
           "family": "",
           "part_of_plan": "no",
           "ptr_record": "",
+		  "release_policy": "",
           "server": "",
           "mac": "",
           "floating": "no",

internal/mock/mock.go

@@ -16,6 +16,30 @@ type Service struct {
 	mock.Mock
 }
 
+// AttachLoadBalancerIPAddress implements service.LoadBalancer.
+func (m *Service) AttachLoadBalancerIPAddress(ctx context.Context, r *request.AttachLoadBalancerIPAddressRequest) (upcloud.LoadBalancerFloatingIPAddress, error) {
+	return upcloud.LoadBalancerFloatingIPAddress{}, nil
+}
+
+// GetLoadBalancerIPAddresses implements service.LoadBalancer.
+func (m *Service) GetLoadBalancerIPAddresses(ctx context.Context, r *request.GetLoadBalancerIPAddressesRequest) ([]upcloud.LoadBalancerFloatingIPAddress, error) {
+	return []upcloud.LoadBalancerFloatingIPAddress{}, nil
+}
+
+// RemoveLoadBalancerIPAddress implements service.LoadBalancer.
+func (m *Service) RemoveLoadBalancerIPAddress(ctx context.Context, r *request.RemoveLoadBalancerIPAddressRequest) error {
+	return nil
+}
+
+// GetAllManagedDatabases implements service.AllServices (or the interface that declares it).
+func (m *Service) GetAllManagedDatabases(_ context.Context) ([]upcloud.ManagedDatabase, error) {
+	args := m.Called()
+	if args[0] == nil {
+		return nil, args.Error(1)
+	}
+	return args[0].([]upcloud.ManagedDatabase), args.Error(1)
+}
+
 func (m *Service) CreateToken(_ context.Context, r *request.CreateTokenRequest) (*upcloud.Token, error) {
 	args := m.Called(r)
 	if args[0] == nil {