return a nil CalendarCompRequest when empty

caldav/caldav_test.go

@@ -0,0 +1,36 @@
+package caldav_test
+
+import (
+	"testing"
+
+	"github.com/emersion/go-webdav/caldav"
+)
+
+func TestCalendarCompRequest_IsEmpty(b *testing.T) {
+	testCases := []struct {
+		Name           string
+		Request        caldav.CalendarCompRequest
+		ExpectedResult bool
+	}{
+		{
+			Name:           "empty",
+			Request:        caldav.CalendarCompRequest{},
+			ExpectedResult: true,
+		},
+		{
+			Name: "has-name",
+			Request: caldav.CalendarCompRequest{
+				Name: "name",
+			},
+			ExpectedResult: false,
+		},
+	}
+
+	for _, tCase := range testCases {
+		b.Run(tCase.Name, func(t *testing.T) {
+			if got, want := tCase.Request.Name == "", tCase.ExpectedResult; got != want { //nolint:scopelint
+				t.Errorf("bad result: %t, expected: %t", got, want)
+			}
+		})
+	}
+}

caldav/client.go

@@ -106,6 +106,10 @@ func (c *Client) FindCalendars(calendarHomeSet string) ([]Calendar, error) {
 }
 
 func encodeCalendarCompReq(c *CalendarCompRequest) (*comp, error) {
+	if c == nil || c.Name == "" {
+		return nil, nil
+	}
+
 	encoded := comp{Name: c.Name}
 
 	if c.AllProps {
@@ -123,7 +127,9 @@ func encodeCalendarCompReq(c *CalendarCompRequest) (*comp, error) {
 		if err != nil {
 			return nil, err
 		}
-		encoded.Comp = append(encoded.Comp, *encodedChild)
+		if encodedChild != nil {
+			encoded.Comp = append(encoded.Comp, *encodedChild)
+		}
 	}
 
 	return &encoded, nil