Feet/bulk group api

[Akira-256]

グループへのユーザーの複数一括追加とグループの全ユーザーの一括削除

[pikachu0310]

レビュー遅くなってしまってごめんなさい！貢献ありがとうございます！！！
全体的にかなり良く書けていて凄いです！びっくり！
少し直して欲しいところを書いたので、対応お願いします～！

[pikachu0310]

何らかの不具合があった時のために↓の様にエラーハンドリングあった方が良さそうです。OnConflictがあるので、既にグループに居るユーザーを追加しようとしてもエラーで止まることはないです。

		if err := tx.Clauses(clause.OnConflict{
			Columns:   []clause.Column{{Name: "group_id"}, {Name: "user_id"}},
			DoUpdates: clause.AssignmentColumns([]string{"role"}),
		}).Create(&users).Error; err != nil {
			return convertError(err)
		}

[pikachu0310]

RemoveUserGroupMemberで全てのユーザーがグループから削除される挙動は予期されないと思うので、ここは消した方が良さそう。

[pikachu0310]

ちょっとネストが深いので、↓のように早期リターンをしたほうが読みやすいかな～と思ったのですが、好みの範疇だと思うのでどちらでもokです。

// AddUserGroupMember POST /groups/:groupID/members
func (h *Handlers) AddUserGroupMember(c echo.Context) error {
	g := getParamGroup(c)
	bodyBytes, err := io.ReadAll(c.Request().Body)
	if err != nil {
		return err
	}

	var singleReq PostUserGroupMemberRequest
	c.Request().Body = io.NopCloser(bytes.NewBuffer(bodyBytes))
	if err := bindAndValidate(c, &singleReq); err == nil {
		if err := h.Repo.AddUserToGroup(singleReq.ID, g.ID, singleReq.Role); err != nil {
			return herror.InternalServerError(err)
		}
		return c.NoContent(http.StatusNoContent)
	}

	var reqs []PostUserGroupMemberRequest
	c.Request().Body = io.NopCloser(bytes.NewBuffer(bodyBytes))
	if err := bindAndValidate(c, &reqs); err != nil {
		return err
	}
	if len(reqs) == 0 {
		return herror.BadRequest(errors.New("no users provided"))
	}
	users := make([]model.UserGroupMember, len(reqs))
	for i, req := range reqs {
		users[i] = model.UserGroupMember{UserID: req.ID, Role: req.Role, GroupID: g.ID}
	}
	if err := h.Repo.AddUsersToGroup(users, g.ID); err != nil {
		return herror.InternalServerError(err)
	}

	return c.NoContent(http.StatusNoContent)
}

[pikachu0310]

RemoveUserFromGroupにevent.UserGroupMemberRemovedがあるので、RemoveUsersFromGroupでもevent.UserGroupMemberRemovedするべきですね。
各ユーザーの削除を追跡して、それに基づいてイベントを発行する必要があってちょっと大変。

var removedUsers []model.UserGroupMember
if err := tx.Where("group_id = ?", groupID).Find(&members).Error; err != nil {
	return err
}

でユーザー一覧を消す前に取得して、削除に成功したら取得したそれぞれのユーザーに対してイベントの発行

for _, userID := range removedUsers {
repo.hub.Publish(hub.Message{
	Name: event.UserGroupMemberRemoved,
	Fields: hub.Fields{
		"group_id": groupID,
		"user_id":  userID,
	},
})
}

をすれば良さそう。

[pikachu0310]

あと以下二つのテストを書きたいですね！
POST /groups/{groupId}/members にメンバーの配列を入れることが出来るようになる
DELETE /groups/{groupId}/members でグループ内メンバーの全削除ができるようになる
ですが、既に結構大きなPRとなっているので、余力が無ければテストは別機会(別issue)で大丈夫です！

[Akira-256]

すみません、時期も時期なのでテストの方は別issueとして扱ってもらえると助かります。