Extends the router client with the missing update functionality

.gitignore

@@ -1,3 +1,4 @@
 .idea/
+.env
 /vendor/
 bin/

router.go

@@ -43,14 +43,20 @@ type RouterCreateRequest struct {
 	InternetGateway bool   `json:"internet_gateway"`
 }
 
-// RouterUpdateRequest is not implemented yet because the API is not implemented yet
-type RouterUpdateRequest struct{}
+type RouterUpdateRequest struct {
+	ZonalResourceRequest
+	TaggedResourceRequest
+	Name string `json:"name,omitempty"`
+	// bool + omitempty: If we don't use a pointer to bool here, the value `false` causes the attribute to be omitted.
+	// This means setting the `internet_gateway` to false would cause an empty request body and thus a 400 Bad Request.
+	InternetGateway *bool `json:"internet_gateway,omitempty"`
+}
 
 type RouterService interface {
 	GenericCreateService[Router, RouterCreateRequest]
 	GenericGetService[Router]
 	GenericListService[Router]
-	// GenericUpdateService[Router, RouterUpdateRequest]
+	GenericUpdateService[Router, RouterUpdateRequest]
 	GenericDeleteService[Router]
 	GenericWaitForService[Router]
 	// CreateInterface creates a new interface attached to this router

router_test.go

@@ -1,6 +1,7 @@
 package cloudscale
 
 import (
+	"context"
 	"encoding/json"
 	"fmt"
 	"net/http"
@@ -48,6 +49,40 @@ func TestRouters_Create(t *testing.T) {
 	}
 }
 
+func TestRouters_Update(t *testing.T) {
+	setup()
+	defer teardown()
+
+	routerID := "cfde831a-4e87-4a75-960f-89b0148aa2cc"
+	newRouterName := "new-router-name"
+	updateRouterRequest := &RouterUpdateRequest{
+		Name: newRouterName,
+	}
+
+	mux.HandleFunc("/v1/routers/cfde831a-4e87-4a75-960f-89b0148aa2cc", func(w http.ResponseWriter, r *http.Request) {
+		testHTTPMethod(t, r, http.MethodPatch)
+
+		expected := map[string]any{
+			"name": newRouterName,
+		}
+
+		var v map[string]any
+		err := json.NewDecoder(r.Body).Decode(&v)
+		if err != nil {
+			t.Fatalf("decode json: %v", err)
+		}
+
+		if !reflect.DeepEqual(v, expected) {
+			t.Errorf("Request body = %#v, expected %#v", v, expected)
+		}
+	})
+
+	err := client.Routers.Update(context.TODO(), routerID, updateRouterRequest)
+	if err != nil {
+		t.Errorf("Routers.Update returned error: %v", err)
+	}
+}
+
 func TestRouters_Get(t *testing.T) {
 	setup()
 	defer teardown()

test/integration/cloudscale_test.go

@@ -260,7 +260,7 @@ func DeleteRemainingRouters() bool {
 			}
 			err = client.Routers.Delete(context.Background(), router.UUID)
 			if err != nil {
-				log.Fatalf("Routers.Delete returned error %s\n", err)
+				log.Fatalf("Routers.Delete returned error: %s\n", err)
 			}
 		}
 	}

test/integration/helper_test.go

@@ -4,13 +4,9 @@ package integration
 
 import (
 	"context"
-	"errors"
 	"math/rand"
 	"reflect"
 	"testing"
-	"time"
-
-	"github.com/cenkalti/backoff/v5"
 
 	"github.com/cloudscale-ch/cloudscale-go-sdk/v10"
 )
@@ -50,23 +46,3 @@ func assertEqual(t *testing.T, expected any, actual any) {
 		t.Errorf("Assertion failed:\nexpected: %#v\n  actual: %#v", expected, actual)
 	}
 }
-
-// waitForDeleted calls existsFunc in a backoff loop until exists is false.
-func waitForDeleted(ctx context.Context, existsFunc func() (exists bool, err error)) error {
-	options := []backoff.RetryOption{
-		backoff.WithBackOff(backoff.NewConstantBackOff(2 * time.Second)),
-		backoff.WithMaxElapsedTime(5 * time.Minute),
-	}
-
-	_, err := backoff.Retry(ctx, func() (struct{}, error) {
-		exists, err := existsFunc()
-		if !exists {
-			return struct{}{}, nil
-		}
-		if err == nil {
-			return struct{}{}, errors.New("resource not deleted yet")
-		}
-		return struct{}{}, err
-	}, options...)
-	return err
-}