fix(objsto): omit empty labels when creating managed object storage (#281)

Author: kangasta

CHANGELOG.md

@@ -8,6 +8,9 @@ See updating [Changelog example here](https://keepachangelog.com/en/1.0.0/)
 ### Added
 - Managed Database sub-properties support. E.g., PostgreSQL property `timescaledb` is of type `object` and has `max_background_workers` sub-property.
 
+### Fixed
+- Managed Object Storage: omit empty labels slice when creating managed object storage instance
+
 ## [6.10.0]
 
 ### Added

upcloud/request/managed_object_storage.go

@@ -37,7 +37,7 @@ func (r *GetManagedObjectStorageRegionRequest) RequestURL() string {
 // CreateManagedObjectStorageRequest represents a request for creating a new Managed Object Storage service
 type CreateManagedObjectStorageRequest struct {
 	ConfiguredStatus upcloud.ManagedObjectStorageConfiguredStatus `json:"configured_status"`
-	Labels           []upcloud.Label                              `json:"labels"`
+	Labels           []upcloud.Label                              `json:"labels,omitempty"`
 	Networks         []upcloud.ManagedObjectStorageNetwork        `json:"networks"`
 	Region           string                                       `json:"region"`
 	Users            []ManagedObjectStorageUser                   `json:"users"`

upcloud/request/managed_object_storage_test.go

@@ -1,6 +1,7 @@
 package request
 
 import (
+	"encoding/json"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -23,6 +24,24 @@ func TestCreateManagedObjectStorageRequest_RequestURL(t *testing.T) {
 	assert.Equal(t, "/object-storage-2", req.RequestURL())
 }
 
+func TestCreateManagedObjectStorageRequest_MarshalJSON(t *testing.T) {
+	t.Run("TestMinimal", func(t *testing.T) {
+		req := CreateManagedObjectStorageRequest{
+			Region: "europe-1",
+		}
+		d, err := json.Marshal(&req)
+		assert.NoError(t, err)
+
+		const expected = `{
+			"configured_status":"",
+			"networks":null,
+			"region":"europe-1",
+			"users":null
+		}`
+		assert.JSONEq(t, expected, string(d))
+	})
+}
+
 func TestGetManagedObjectStoragesRequest_RequestURL(t *testing.T) {
 	req := &GetManagedObjectStoragesRequest{}
 	assert.Equal(t, "/object-storage-2", req.RequestURL())