Add room deletion test

daily/room/delete.go

@@ -52,9 +52,10 @@ func Delete(creds auth.Creds, roomName string) error {
 	}
 	if dr.Deleted {
 		if dr.RoomName != roomName {
-			return fmt.Errorf("requested deletion was of room name '%s', but room reported deleted was '%s'", roomName, dr.RoomName)
+			err := fmt.Errorf("requested deletion was of room name '%s', but room reported deleted was '%s'", roomName, dr.RoomName)
+			return NewErrFailRoomDelete(err)
 		}
 		return nil
 	}
-	return errors.New("failed to delete room")
+	return NewErrFailRoomDelete(errors.New("room not deleted"))
 }

daily/room/errors.go

@@ -6,9 +6,14 @@ import (
 )
 
 var (
-	ErrFailUnmarshal = errors.New("failed to unmarshal response body into Room")
+	ErrFailUnmarshal  = errors.New("failed to unmarshal response body into Room")
+	ErrFailRoomDelete = errors.New("failed to delete room")
 )
 
 func NewErrFailUnmarshal(unmarshalErr error) error {
 	return fmt.Errorf("%s: %w", unmarshalErr, ErrFailUnmarshal)
 }
+
+func NewErrFailRoomDelete(deleteErr error) error {
+	return fmt.Errorf("%s: %w", deleteErr, ErrFailRoomDelete)
+}

daily/room/tests/delete_test.go

@@ -0,0 +1,64 @@
+package tests
+
+import (
+	"github.com/lazeratops/daily-go/daily/auth"
+	"github.com/lazeratops/daily-go/daily/room"
+	"github.com/stretchr/testify/require"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+)
+
+func TestDeleteRoom(t *testing.T) {
+	t.Parallel()
+	testCases := []struct {
+		name               string
+		roomName           string
+		dailyResStatusCode int
+		dailyResBody       string
+		wantErr            error
+	}{
+		{
+			name:               "success",
+			roomName:           "room-0253",
+			dailyResStatusCode: 200,
+			dailyResBody: `
+			{
+			  "deleted": true,
+			  "name": "room-0253"
+			}`,
+		},
+		{
+			name:               "wrong-room-name",
+			roomName:           "room-0253",
+			dailyResStatusCode: 200,
+			dailyResBody: `
+			{
+			  "deleted": true,
+			  "name": "room-0251"
+			}`,
+			wantErr: room.ErrFailRoomDelete,
+		},
+	}
+
+	for _, tc := range testCases {
+		tc := tc
+		t.Run(tc.name, func(t *testing.T) {
+			t.Parallel()
+
+			testServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+				w.WriteHeader(tc.dailyResStatusCode)
+				_, err := w.Write([]byte(tc.dailyResBody))
+				require.NoError(t, err)
+			}))
+
+			defer testServer.Close()
+
+			gotErr := room.Delete(auth.Creds{
+				APIKey: "someKey",
+				APIURL: testServer.URL,
+			}, tc.roomName)
+			require.ErrorIs(t, gotErr, tc.wantErr)
+		})
+	}
+}