drone/internal/store/database/pullreq.go
Marko Gaćeša a79e187a88 DB & API refactor: Use snake case (#144)
Update DB and API to follow Harness Standard
2022-12-29 13:53:08 -08:00

478 lines
13 KiB
Go

// Copyright 2022 Harness Inc. All rights reserved.
// Use of this source code is governed by the Polyform Free Trial License
// that can be found in the LICENSE.md file for this repository.
package database
import (
"context"
"time"
"github.com/harness/gitness/internal/store"
"github.com/harness/gitness/internal/store/database/dbtx"
"github.com/harness/gitness/types"
"github.com/harness/gitness/types/enum"
"github.com/Masterminds/squirrel"
"github.com/guregu/null"
"github.com/jmoiron/sqlx"
"github.com/pkg/errors"
)
var _ store.PullReqStore = (*PullReqStore)(nil)
// NewPullReqStore returns a new PullReqStore.
func NewPullReqStore(db *sqlx.DB) *PullReqStore {
return &PullReqStore{
db: db,
}
}
// PullReqStore implements store.PullReqStore backed by a relational database.
type PullReqStore struct {
db *sqlx.DB
}
// pullReq is used to fetch pull request data from the database.
// The object should be later re-packed into a different struct to return it as an API response.
type pullReq struct {
ID int64 `db:"pullreq_id"`
Version int64 `db:"pullreq_version"`
Number int64 `db:"pullreq_number"`
CreatedBy int64 `db:"pullreq_created_by"`
Created int64 `db:"pullreq_created"`
Updated int64 `db:"pullreq_updated"`
Edited int64 `db:"pullreq_edited"`
State enum.PullReqState `db:"pullreq_state"`
Title string `db:"pullreq_title"`
Description string `db:"pullreq_description"`
SourceRepoID int64 `db:"pullreq_source_repo_id"`
SourceBranch string `db:"pullreq_source_branch"`
TargetRepoID int64 `db:"pullreq_target_repo_id"`
TargetBranch string `db:"pullreq_target_branch"`
ActivitySeq int64 `db:"pullreq_activity_seq"`
MergedBy null.Int `db:"pullreq_merged_by"`
Merged null.Int `db:"pullreq_merged"`
MergeStrategy null.String `db:"pullreq_merge_strategy"`
AuthorUID string `db:"author_uid"`
AuthorName string `db:"author_name"`
AuthorEmail string `db:"author_email"`
MergerUID null.String `db:"merger_uid"`
MergerName null.String `db:"merger_name"`
MergerEmail null.String `db:"merger_email"`
}
const (
pullReqColumns = `
pullreq_id
,pullreq_version
,pullreq_number
,pullreq_created_by
,pullreq_created
,pullreq_updated
,pullreq_edited
,pullreq_state
,pullreq_title
,pullreq_description
,pullreq_source_repo_id
,pullreq_source_branch
,pullreq_target_repo_id
,pullreq_target_branch
,pullreq_activity_seq
,pullreq_merged_by
,pullreq_merged
,pullreq_merge_strategy
,author.principal_uid as "author_uid"
,author.principal_display_name as "author_name"
,author.principal_email as "author_email"
,merger.principal_uid as "merger_uid"
,merger.principal_display_name as "merger_name"
,merger.principal_email as "merger_email"`
pullReqSelectBase = `
SELECT` + pullReqColumns + `
FROM pullreqs
INNER JOIN principals author on author.principal_id = pullreq_created_by
LEFT JOIN principals merger on merger.principal_id = pullreq_merged_by`
)
// Find finds the pull request by id.
func (s *PullReqStore) Find(ctx context.Context, id int64) (*types.PullReq, error) {
const sqlQuery = pullReqSelectBase + `
WHERE pullreq_id = $1`
db := dbtx.GetAccessor(ctx, s.db)
dst := &pullReq{}
if err := db.GetContext(ctx, dst, sqlQuery, id); err != nil {
return nil, processSQLErrorf(err, "Failed to find pull request")
}
return mapPullReq(dst), nil
}
// FindByNumber finds the pull request by repo ID and pull request number.
func (s *PullReqStore) FindByNumber(ctx context.Context, repoID, number int64) (*types.PullReq, error) {
const sqlQuery = pullReqSelectBase + `
WHERE pullreq_target_repo_id = $1 AND pullreq_number = $2`
db := dbtx.GetAccessor(ctx, s.db)
dst := &pullReq{}
if err := db.GetContext(ctx, dst, sqlQuery, repoID, number); err != nil {
return nil, processSQLErrorf(err, "Failed to find pull request by number")
}
return mapPullReq(dst), nil
}
// Create creates a new pull request.
func (s *PullReqStore) Create(ctx context.Context, pr *types.PullReq) error {
const sqlQuery = `
INSERT INTO pullreqs (
pullreq_version
,pullreq_number
,pullreq_created_by
,pullreq_created
,pullreq_updated
,pullreq_edited
,pullreq_state
,pullreq_title
,pullreq_description
,pullreq_source_repo_id
,pullreq_source_branch
,pullreq_target_repo_id
,pullreq_target_branch
,pullreq_activity_seq
,pullreq_merged_by
,pullreq_merged
,pullreq_merge_strategy
) values (
:pullreq_version
,:pullreq_number
,:pullreq_created_by
,:pullreq_created
,:pullreq_updated
,:pullreq_edited
,:pullreq_state
,:pullreq_title
,:pullreq_description
,:pullreq_source_repo_id
,:pullreq_source_branch
,:pullreq_target_repo_id
,:pullreq_target_branch
,:pullreq_activity_seq
,:pullreq_merged_by
,:pullreq_merged
,:pullreq_merge_strategy
) RETURNING pullreq_id`
db := dbtx.GetAccessor(ctx, s.db)
query, arg, err := db.BindNamed(sqlQuery, mapInternalPullReq(pr))
if err != nil {
return processSQLErrorf(err, "Failed to bind pullReq object")
}
if err = db.QueryRowContext(ctx, query, arg...).Scan(&pr.ID); err != nil {
return processSQLErrorf(err, "Insert query failed")
}
return nil
}
// Update updates the pull request.
func (s *PullReqStore) Update(ctx context.Context, pr *types.PullReq) error {
const sqlQuery = `
UPDATE pullreqs
SET
pullreq_version = :pullreq_version
,pullreq_updated = :pullreq_updated
,pullreq_edited = :pullreq_edited
,pullreq_state = :pullreq_state
,pullreq_title = :pullreq_title
,pullreq_description = :pullreq_description
,pullreq_activity_seq = :pullreq_activity_seq
,pullreq_merged_by = :pullreq_merged_by
,pullreq_merged = :pullreq_merged
,pullreq_merge_strategy = :pullreq_merge_strategy
WHERE pullreq_id = :pullreq_id AND pullreq_version = :pullreq_version - 1`
db := dbtx.GetAccessor(ctx, s.db)
updatedAt := time.Now()
dbPR := mapInternalPullReq(pr)
dbPR.Version++
dbPR.Updated = updatedAt.UnixMilli()
query, arg, err := db.BindNamed(sqlQuery, dbPR)
if err != nil {
return processSQLErrorf(err, "Failed to bind pull request object")
}
result, err := db.ExecContext(ctx, query, arg...)
if err != nil {
return processSQLErrorf(err, "Failed to update pull request")
}
count, err := result.RowsAffected()
if err != nil {
return processSQLErrorf(err, "Failed to get number of updated rows")
}
if count == 0 {
return store.ErrConflict
}
pr.Version = dbPR.Version
pr.Updated = dbPR.Updated
return nil
}
// UpdateActivitySeq updates the pull request's activity sequence.
func (s *PullReqStore) UpdateActivitySeq(ctx context.Context, pr *types.PullReq) (*types.PullReq, error) {
for {
dup := *pr
dup.ActivitySeq++
err := s.Update(ctx, &dup)
if err == nil {
return &dup, nil
}
if !errors.Is(err, store.ErrConflict) {
return nil, err
}
pr, err = s.Find(ctx, pr.ID)
if err != nil {
return nil, err
}
}
}
// Delete the pull request.
func (s *PullReqStore) Delete(ctx context.Context, id int64) error {
const pullReqDelete = `DELETE FROM pullreqs WHERE pullreq_id = $1`
db := dbtx.GetAccessor(ctx, s.db)
if _, err := db.ExecContext(ctx, pullReqDelete, id); err != nil {
return processSQLErrorf(err, "the delete query failed")
}
return nil
}
// LastNumber return the number of the most recent pull request.
func (s *PullReqStore) LastNumber(ctx context.Context, repoID int64) (int64, error) {
const sqlQuery = `select coalesce(max(pullreq_number), 0) from pullreqs where pullreq_target_repo_id = $1 limit 1`
db := dbtx.GetAccessor(ctx, s.db)
row := db.QueryRowContext(ctx, sqlQuery, repoID)
if err := row.Err(); err != nil {
return 0, err
}
var lastNumber int64
if err := row.Scan(&lastNumber); err != nil {
return 0, err
}
return lastNumber, nil
}
// Count of pull requests for a repo.
func (s *PullReqStore) Count(ctx context.Context, repoID int64, opts *types.PullReqFilter) (int64, error) {
stmt := builder.
Select("count(*)").
From("pullreqs").
Where("pullreq_target_repo_id = ?", repoID)
if len(opts.States) == 1 {
stmt = stmt.Where("pullreq_state = ?", opts.States[0])
} else if len(opts.States) > 1 {
stmt = stmt.Where(squirrel.Eq{"pullreq_state": opts.States})
}
if opts.SourceRepoID != 0 {
stmt = stmt.Where("pullreq_source_repo_id = ?", opts.SourceRepoID)
}
if opts.SourceBranch != "" {
stmt = stmt.Where("pullreq_source_branch = ?", opts.SourceBranch)
}
if opts.TargetBranch != "" {
stmt = stmt.Where("pullreq_target_branch = ?", opts.TargetBranch)
}
if opts.CreatedBy != 0 {
stmt = stmt.Where("pullreq_created_by = ?", opts.CreatedBy)
}
sql, args, err := stmt.ToSql()
if err != nil {
return 0, errors.Wrap(err, "Failed to convert query to sql")
}
db := dbtx.GetAccessor(ctx, s.db)
var count int64
err = db.QueryRowContext(ctx, sql, args...).Scan(&count)
if err != nil {
return 0, processSQLErrorf(err, "Failed executing count query")
}
return count, nil
}
// List returns a list of pull requests for a repo.
func (s *PullReqStore) List(ctx context.Context, repoID int64, opts *types.PullReqFilter) ([]*types.PullReq, error) {
stmt := builder.
Select(pullReqColumns).
From("pullreqs").
InnerJoin("principals author on author.principal_id = pullreq_created_by").
LeftJoin("principals merger on merger.principal_id = pullreq_merged_by").
Where("pullreq_target_repo_id = ?", repoID)
if len(opts.States) == 1 {
stmt = stmt.Where("pullreq_state = ?", opts.States[0])
} else if len(opts.States) > 1 {
stmt = stmt.Where(squirrel.Eq{"pullreq_state": opts.States})
}
if opts.SourceRepoID != 0 {
stmt = stmt.Where("pullreq_source_repo_id = ?", opts.SourceRepoID)
}
if opts.SourceBranch != "" {
stmt = stmt.Where("pullreq_source_branch = ?", opts.SourceBranch)
}
if opts.TargetBranch != "" {
stmt = stmt.Where("pullreq_target_branch = ?", opts.TargetBranch)
}
if opts.Query != "" {
stmt = stmt.Where("pullreq_title LIKE ?", "%"+opts.Query+"%")
}
if opts.CreatedBy > 0 {
stmt = stmt.Where("pullreq_created_by = ?", opts.CreatedBy)
}
stmt = stmt.Limit(uint64(limit(opts.Size)))
stmt = stmt.Offset(uint64(offset(opts.Page, opts.Size)))
// NOTE: string concatenation is safe because the
// order attribute is an enum and is not user-defined,
// and is therefore not subject to injection attacks.
switch opts.Sort {
case enum.PullReqSortNumber, enum.PullReqSortNone:
stmt = stmt.OrderBy("pullreq_number " + opts.Order.String())
case enum.PullReqSortCreated:
stmt = stmt.OrderBy("pullreq_created " + opts.Order.String())
case enum.PullReqSortUpdated:
stmt = stmt.OrderBy("pullreq_updated " + opts.Order.String())
}
sql, args, err := stmt.ToSql()
if err != nil {
return nil, errors.Wrap(err, "Failed to convert query to sql")
}
dst := make([]*pullReq, 0)
db := dbtx.GetAccessor(ctx, s.db)
if err = db.SelectContext(ctx, &dst, sql, args...); err != nil {
return nil, processSQLErrorf(err, "Failed executing custom list query")
}
return mapSlicePullReq(dst), nil
}
func mapPullReq(pr *pullReq) *types.PullReq {
m := &types.PullReq{
ID: pr.ID,
Version: pr.Version,
Number: pr.Number,
CreatedBy: pr.CreatedBy,
Created: pr.Created,
Updated: pr.Updated,
Edited: pr.Edited,
State: pr.State,
Title: pr.Title,
Description: pr.Description,
SourceRepoID: pr.SourceRepoID,
SourceBranch: pr.SourceBranch,
TargetRepoID: pr.TargetRepoID,
TargetBranch: pr.TargetBranch,
ActivitySeq: pr.ActivitySeq,
MergedBy: pr.MergedBy.Ptr(),
Merged: pr.Merged.Ptr(),
MergeStrategy: pr.MergeStrategy.Ptr(),
Author: types.PrincipalInfo{},
Merger: nil,
}
m.Author = types.PrincipalInfo{
ID: pr.CreatedBy,
UID: pr.AuthorUID,
Name: pr.AuthorName,
Email: pr.AuthorEmail,
}
if pr.MergedBy.Valid {
m.Merger = &types.PrincipalInfo{
ID: pr.MergedBy.Int64,
UID: pr.MergerUID.String,
Name: pr.MergerName.String,
Email: pr.MergerEmail.String,
}
}
return m
}
func mapInternalPullReq(pr *types.PullReq) *pullReq {
m := &pullReq{
ID: pr.ID,
Version: pr.Version,
Number: pr.Number,
CreatedBy: pr.CreatedBy,
Created: pr.Created,
Updated: pr.Updated,
Edited: pr.Edited,
State: pr.State,
Title: pr.Title,
Description: pr.Description,
SourceRepoID: pr.SourceRepoID,
SourceBranch: pr.SourceBranch,
TargetRepoID: pr.TargetRepoID,
TargetBranch: pr.TargetBranch,
ActivitySeq: pr.ActivitySeq,
MergedBy: null.IntFromPtr(pr.MergedBy),
Merged: null.IntFromPtr(pr.Merged),
MergeStrategy: null.StringFromPtr(pr.MergeStrategy),
}
return m
}
func mapSlicePullReq(prs []*pullReq) []*types.PullReq {
m := make([]*types.PullReq, len(prs))
for i, pr := range prs {
m[i] = mapPullReq(pr)
}
return m
}