drone/app/api/controller/pullreq/reviewer_combined_list.go
Akhilesh Pandey 0578be8afe feat: [CODE-1941]: user group changes (#2134)
* feat: [CODE-1941]: remove usergroup filter from request
* feat: [CODE-1941]: decrement page for harness-core
* feat: [CODE-1941]: update pagination request
* feat: [CODE-1941]: change interface from find to list
* feat: [CODE-1941]: change reviewer add permissions
* feat: [CODE-1941]: comments
* feat: [CODE-1941]: comments
* fix wire issues
* feat: [CODE-1941]: remove users from the db table usergroups
* feat: [CODE-1941]: annotate errors
* feat: [CODE-1941]: annotate errors
* feat: [CODE-1941]: sql style
* feat: [CODE-1941]: revert common.go change for parse limit
* feat: [CODE-1941]: move usergroup listing to space level
* feat: [CODE-1941]: update usergroup listing and update delete to push permission
* feat: [CODE-1941]: Update combined listing
* feat: [CODE-1941]: Update combined listing
* feat: [CODE-1941]: format
* feat: [CODE-1941]: rebase
* feat: [CODE-1941]: separate db struct for usergroup.go
* feat: [CODE-1941]: change usergroup filter to default query and review comments
* feat: [CODE-1941]: change usergroup filter to default
* feat: [CODE-1941]: change return type for search service interface
* feat: [CODE-1941]: change return type for search service interface
* feat: [CODE-1941]: change return type for search service interface
* feat: [CODE-1941]: Search Service interface and wire changes
* feat: [CODE-1941]: Search Service interface and wire changes
* feat: [CODE-1941]: Search UserGroups Handler change - codeformat
* feat: [CODE-1941]: Search UserGroups Handler change - codeformat
* feat: [CODE-1941]: Search UserGroups Handler change - codeformat
* feat: [CODE-1941]: Search UserGroups Handler change
* feat: [CODE-1941]: Big change: New LOGIC for UserGroup Reviewers
* feat: [CODE-1941]: DB changes
* feat: [CODE-1941]: listing api
* feat: [CODE-1941]: mapping methods
* feat: [CODE-1941]: list method for pullreq_reviewers db
* feat: [CODE-1941]: database queries
* feat: [CODE-1941]: database queries
* feat: [CODE-2058]: dbstore changes
* feat: [CODE-1941]: Controller and Database code
* feat: [CODE-1941]: update reviewer_add.go code
* feat: [CODE-1941]: update reviewers and usergroup db migrations
* feat: [CODE-1941]: Update API ops
* feat: [CODE-1941]: Update DB ops for reviewers
* feat: [CODE-1941]: db migration update
* feat: [CODE-1941]: db migration update
* feat: [CODE-1941]: code comments
* feat: [CODE-1941]: update migration numbering
* feat: [CODE-1941]: Add postgresql migrations
* feat: [CODE-1941]: Update DB ops for reviewers

feat: [CODE-1941]: Update API ops

feat: [CODE-1941]: update reviewers and usergroup db migrations

feat: [CODE-1941]: update reviewer_add.go code

feat: [CODE-1941]: break pr

feat: [CODE-1941]: break pr

feat: [CODE-1941]: Update migration numbering
* feat: [CODE-1941]: create table usergroups
* feat: [CODE-1941]: table migrations
* feat: [CODE-1941]: user group changes
2024-08-26 15:35:27 +00:00

107 lines
3.3 KiB
Go

// Copyright 2023 Harness, Inc.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package pullreq
import (
"context"
"errors"
"fmt"
"github.com/harness/gitness/app/auth"
"github.com/harness/gitness/store"
"github.com/harness/gitness/types"
"github.com/harness/gitness/types/enum"
)
type CombinedListResponse struct {
Reviewers []*types.PullReqReviewer `json:"reviewers"`
UserGroupReviewers []*types.UserGroupReviewer `json:"user_group_reviewers"`
}
// ReviewersListCombined returns the combined reviewer list for the pull request.
func (c *Controller) ReviewersListCombined(
ctx context.Context,
session *auth.Session,
repoRef string,
prNum int64,
) (*CombinedListResponse, error) {
repo, err := c.getRepoCheckAccess(ctx, session, repoRef, enum.PermissionRepoView)
if err != nil {
return nil, fmt.Errorf("failed to acquire access to repo: %w", err)
}
pr, err := c.pullreqStore.FindByNumber(ctx, repo.ID, prNum)
if err != nil {
return nil, fmt.Errorf("failed to find pull request by number: %w", err)
}
reviewers, err := c.reviewerStore.List(ctx, pr.ID)
if err != nil {
return nil, fmt.Errorf("failed to list reviewers: %w", err)
}
reviewersMap := createReviewerMap(reviewers)
userGroupReviewers, err := c.userGroupReviewerStore.List(ctx, pr.ID)
if err != nil && !errors.Is(err, store.ErrResourceNotFound) {
return nil, fmt.Errorf("failed to list user group reviewers: %w", err)
}
for _, userGroupReviewer := range userGroupReviewers {
userGroup, err := c.userGroupStore.Find(ctx, userGroupReviewer.UserGroupID)
if err != nil {
return nil, fmt.Errorf("failed to find user group: %w", err)
}
usersUIDs, err := c.userGroupService.ListUsers(ctx, session, userGroup)
if err != nil {
return nil, fmt.Errorf("failed to find users belonging: %w", err)
}
if usersUIDs == nil {
continue
}
users, err := c.principalStore.FindManyByUID(ctx, usersUIDs)
if err != nil {
return nil, fmt.Errorf("failed to find user group users: %w", err)
}
if len(users) == 0 {
continue
}
var userGroupReviewerDecisions []types.UserGroupReviewerDecision
for _, user := range users {
if reviewer, ok := reviewersMap[user.ID]; ok {
userGroupReviewerDecisions = append(userGroupReviewerDecisions, types.UserGroupReviewerDecision{
ReviewDecision: reviewer.ReviewDecision,
SHA: reviewer.SHA,
Reviewer: reviewer.Reviewer,
})
}
}
userGroupReviewer.Reviewers = userGroupReviewerDecisions
}
return &CombinedListResponse{
Reviewers: reviewers,
UserGroupReviewers: userGroupReviewers,
}, nil
}
func createReviewerMap(reviewers []*types.PullReqReviewer) map[int64]*types.PullReqReviewer {
reviewerMap := make(map[int64]*types.PullReqReviewer)
for _, reviewer := range reviewers {
reviewerMap[reviewer.PrincipalID] = reviewer
}
return reviewerMap
}