mirror of https://github.com/go-gitea/gitea.git
Merge 79c014e003 into 01351cc6c7
commit
2358856466
@ -0,0 +1,25 @@
|
|||||||
|
// Copyright 2025 The Gitea Authors. All rights reserved.
|
||||||
|
// SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
package v1_26
|
||||||
|
|
||||||
|
import (
|
||||||
|
"code.gitea.io/gitea/modules/timeutil"
|
||||||
|
|
||||||
|
"xorm.io/xorm"
|
||||||
|
)
|
||||||
|
|
||||||
|
func AddProjectWorkflow(x *xorm.Engine) error {
|
||||||
|
type ProjectWorkflow struct {
|
||||||
|
ID int64
|
||||||
|
ProjectID int64 `xorm:"INDEX"`
|
||||||
|
WorkflowEvent string `xorm:"INDEX"`
|
||||||
|
WorkflowFilters string `xorm:"TEXT JSON"`
|
||||||
|
WorkflowActions string `xorm:"TEXT JSON"`
|
||||||
|
Enabled bool `xorm:"DEFAULT true"`
|
||||||
|
CreatedUnix timeutil.TimeStamp `xorm:"created"`
|
||||||
|
UpdatedUnix timeutil.TimeStamp `xorm:"updated"`
|
||||||
|
}
|
||||||
|
|
||||||
|
return x.Sync(&ProjectWorkflow{})
|
||||||
|
}
|
||||||
@ -0,0 +1,241 @@
|
|||||||
|
// Copyright 2025 The Gitea Authors. All rights reserved.
|
||||||
|
// SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
package project
|
||||||
|
|
||||||
|
import (
|
||||||
|
"context"
|
||||||
|
"fmt"
|
||||||
|
|
||||||
|
"code.gitea.io/gitea/models/db"
|
||||||
|
"code.gitea.io/gitea/modules/log"
|
||||||
|
"code.gitea.io/gitea/modules/timeutil"
|
||||||
|
)
|
||||||
|
|
||||||
|
type WorkflowEvent string
|
||||||
|
|
||||||
|
const (
|
||||||
|
WorkflowEventItemOpened WorkflowEvent = "item_opened"
|
||||||
|
WorkflowEventItemAddedToProject WorkflowEvent = "item_added_to_project"
|
||||||
|
WorkflowEventItemRemovedFromProject WorkflowEvent = "item_removed_from_project"
|
||||||
|
WorkflowEventItemReopened WorkflowEvent = "item_reopened"
|
||||||
|
WorkflowEventItemClosed WorkflowEvent = "item_closed"
|
||||||
|
WorkflowEventItemColumnChanged WorkflowEvent = "item_column_changed"
|
||||||
|
WorkflowEventCodeChangesRequested WorkflowEvent = "code_changes_requested"
|
||||||
|
WorkflowEventCodeReviewApproved WorkflowEvent = "code_review_approved"
|
||||||
|
WorkflowEventPullRequestMerged WorkflowEvent = "pull_request_merged"
|
||||||
|
)
|
||||||
|
|
||||||
|
var workflowEvents = []WorkflowEvent{
|
||||||
|
WorkflowEventItemOpened,
|
||||||
|
WorkflowEventItemAddedToProject,
|
||||||
|
WorkflowEventItemRemovedFromProject,
|
||||||
|
WorkflowEventItemReopened,
|
||||||
|
WorkflowEventItemClosed,
|
||||||
|
WorkflowEventItemColumnChanged,
|
||||||
|
WorkflowEventCodeChangesRequested,
|
||||||
|
WorkflowEventCodeReviewApproved,
|
||||||
|
WorkflowEventPullRequestMerged,
|
||||||
|
}
|
||||||
|
|
||||||
|
func GetWorkflowEvents() []WorkflowEvent {
|
||||||
|
return workflowEvents
|
||||||
|
}
|
||||||
|
|
||||||
|
func IsValidWorkflowEvent(event string) bool {
|
||||||
|
for _, we := range workflowEvents {
|
||||||
|
if we.EventID() == event {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
|
func (we WorkflowEvent) LangKey() string {
|
||||||
|
switch we {
|
||||||
|
case WorkflowEventItemOpened:
|
||||||
|
return "projects.workflows.event.item_opened"
|
||||||
|
case WorkflowEventItemAddedToProject:
|
||||||
|
return "projects.workflows.event.item_added_to_project"
|
||||||
|
case WorkflowEventItemRemovedFromProject:
|
||||||
|
return "projects.workflows.event.item_removed_from_project"
|
||||||
|
case WorkflowEventItemReopened:
|
||||||
|
return "projects.workflows.event.item_reopened"
|
||||||
|
case WorkflowEventItemClosed:
|
||||||
|
return "projects.workflows.event.item_closed"
|
||||||
|
case WorkflowEventItemColumnChanged:
|
||||||
|
return "projects.workflows.event.item_column_changed"
|
||||||
|
case WorkflowEventCodeChangesRequested:
|
||||||
|
return "projects.workflows.event.code_changes_requested"
|
||||||
|
case WorkflowEventCodeReviewApproved:
|
||||||
|
return "projects.workflows.event.code_review_approved"
|
||||||
|
case WorkflowEventPullRequestMerged:
|
||||||
|
return "projects.workflows.event.pull_request_merged"
|
||||||
|
default:
|
||||||
|
return string(we)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func (we WorkflowEvent) EventID() string {
|
||||||
|
return string(we)
|
||||||
|
}
|
||||||
|
|
||||||
|
type WorkflowFilterType string
|
||||||
|
|
||||||
|
const (
|
||||||
|
WorkflowFilterTypeIssueType WorkflowFilterType = "issue_type" // issue, pull_request, etc.
|
||||||
|
WorkflowFilterTypeSourceColumn WorkflowFilterType = "source_column" // source column for item_column_changed event
|
||||||
|
WorkflowFilterTypeTargetColumn WorkflowFilterType = "target_column" // target column for item_column_changed event
|
||||||
|
WorkflowFilterTypeLabels WorkflowFilterType = "labels" // filter by issue/PR labels
|
||||||
|
)
|
||||||
|
|
||||||
|
type WorkflowFilter struct {
|
||||||
|
Type WorkflowFilterType `json:"type"`
|
||||||
|
Value string `json:"value"`
|
||||||
|
}
|
||||||
|
|
||||||
|
type WorkflowActionType string
|
||||||
|
|
||||||
|
const (
|
||||||
|
WorkflowActionTypeColumn WorkflowActionType = "column" // add the item to the project's column
|
||||||
|
WorkflowActionTypeAddLabels WorkflowActionType = "add_labels" // choose one or more labels
|
||||||
|
WorkflowActionTypeRemoveLabels WorkflowActionType = "remove_labels" // choose one or more labels
|
||||||
|
WorkflowActionTypeIssueState WorkflowActionType = "issue_state" // change the issue state (reopen/close)
|
||||||
|
)
|
||||||
|
|
||||||
|
type WorkflowAction struct {
|
||||||
|
Type WorkflowActionType `json:"type"`
|
||||||
|
Value string `json:"value"`
|
||||||
|
}
|
||||||
|
|
||||||
|
// WorkflowEventCapabilities defines what filters and actions are available for each event
|
||||||
|
type WorkflowEventCapabilities struct {
|
||||||
|
AvailableFilters []WorkflowFilterType `json:"available_filters"`
|
||||||
|
AvailableActions []WorkflowActionType `json:"available_actions"`
|
||||||
|
}
|
||||||
|
|
||||||
|
// GetWorkflowEventCapabilities returns the capabilities for each workflow event
|
||||||
|
func GetWorkflowEventCapabilities() map[WorkflowEvent]WorkflowEventCapabilities {
|
||||||
|
return map[WorkflowEvent]WorkflowEventCapabilities{
|
||||||
|
WorkflowEventItemOpened: {
|
||||||
|
AvailableFilters: []WorkflowFilterType{WorkflowFilterTypeIssueType, WorkflowFilterTypeLabels},
|
||||||
|
AvailableActions: []WorkflowActionType{WorkflowActionTypeColumn, WorkflowActionTypeAddLabels},
|
||||||
|
},
|
||||||
|
WorkflowEventItemAddedToProject: {
|
||||||
|
AvailableFilters: []WorkflowFilterType{WorkflowFilterTypeIssueType, WorkflowFilterTypeLabels},
|
||||||
|
AvailableActions: []WorkflowActionType{WorkflowActionTypeColumn, WorkflowActionTypeAddLabels, WorkflowActionTypeRemoveLabels, WorkflowActionTypeIssueState},
|
||||||
|
},
|
||||||
|
WorkflowEventItemRemovedFromProject: {
|
||||||
|
AvailableFilters: []WorkflowFilterType{WorkflowFilterTypeIssueType, WorkflowFilterTypeLabels},
|
||||||
|
AvailableActions: []WorkflowActionType{WorkflowActionTypeAddLabels, WorkflowActionTypeRemoveLabels, WorkflowActionTypeIssueState},
|
||||||
|
},
|
||||||
|
WorkflowEventItemReopened: {
|
||||||
|
AvailableFilters: []WorkflowFilterType{WorkflowFilterTypeIssueType, WorkflowFilterTypeLabels},
|
||||||
|
AvailableActions: []WorkflowActionType{WorkflowActionTypeColumn, WorkflowActionTypeAddLabels, WorkflowActionTypeRemoveLabels},
|
||||||
|
},
|
||||||
|
WorkflowEventItemClosed: {
|
||||||
|
AvailableFilters: []WorkflowFilterType{WorkflowFilterTypeIssueType, WorkflowFilterTypeLabels},
|
||||||
|
AvailableActions: []WorkflowActionType{WorkflowActionTypeColumn, WorkflowActionTypeAddLabels, WorkflowActionTypeRemoveLabels},
|
||||||
|
},
|
||||||
|
WorkflowEventItemColumnChanged: {
|
||||||
|
AvailableFilters: []WorkflowFilterType{WorkflowFilterTypeIssueType, WorkflowFilterTypeSourceColumn, WorkflowFilterTypeTargetColumn, WorkflowFilterTypeLabels},
|
||||||
|
AvailableActions: []WorkflowActionType{WorkflowActionTypeAddLabels, WorkflowActionTypeRemoveLabels, WorkflowActionTypeIssueState},
|
||||||
|
},
|
||||||
|
WorkflowEventCodeChangesRequested: {
|
||||||
|
AvailableFilters: []WorkflowFilterType{WorkflowFilterTypeLabels}, // only applies to pull requests
|
||||||
|
AvailableActions: []WorkflowActionType{WorkflowActionTypeColumn, WorkflowActionTypeAddLabels, WorkflowActionTypeRemoveLabels},
|
||||||
|
},
|
||||||
|
WorkflowEventCodeReviewApproved: {
|
||||||
|
AvailableFilters: []WorkflowFilterType{WorkflowFilterTypeLabels}, // only applies to pull requests
|
||||||
|
AvailableActions: []WorkflowActionType{WorkflowActionTypeColumn, WorkflowActionTypeAddLabels, WorkflowActionTypeRemoveLabels},
|
||||||
|
},
|
||||||
|
WorkflowEventPullRequestMerged: {
|
||||||
|
AvailableFilters: []WorkflowFilterType{WorkflowFilterTypeLabels}, // only applies to pull requests
|
||||||
|
AvailableActions: []WorkflowActionType{WorkflowActionTypeColumn, WorkflowActionTypeAddLabels, WorkflowActionTypeRemoveLabels},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
type Workflow struct {
|
||||||
|
ID int64
|
||||||
|
ProjectID int64 `xorm:"INDEX"`
|
||||||
|
Project *Project `xorm:"-"`
|
||||||
|
WorkflowEvent WorkflowEvent `xorm:"INDEX"`
|
||||||
|
WorkflowFilters []WorkflowFilter `xorm:"TEXT JSON"`
|
||||||
|
WorkflowActions []WorkflowAction `xorm:"TEXT JSON"`
|
||||||
|
Enabled bool `xorm:"DEFAULT true"`
|
||||||
|
CreatedUnix timeutil.TimeStamp `xorm:"created"`
|
||||||
|
UpdatedUnix timeutil.TimeStamp `xorm:"updated"`
|
||||||
|
}
|
||||||
|
|
||||||
|
// TableName overrides the table name used by ProjectWorkflow to `project_workflow`
|
||||||
|
func (Workflow) TableName() string {
|
||||||
|
return "project_workflow"
|
||||||
|
}
|
||||||
|
|
||||||
|
func (p *Workflow) LoadProject(ctx context.Context) error {
|
||||||
|
if p.Project != nil || p.ProjectID <= 0 {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
project, err := GetProjectByID(ctx, p.ProjectID)
|
||||||
|
if err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
p.Project = project
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
|
func (p *Workflow) Link(ctx context.Context) string {
|
||||||
|
if err := p.LoadProject(ctx); err != nil {
|
||||||
|
log.Error("ProjectWorkflow Link: %v", err)
|
||||||
|
return ""
|
||||||
|
}
|
||||||
|
return p.Project.Link(ctx) + fmt.Sprintf("/workflows/%d", p.ID)
|
||||||
|
}
|
||||||
|
|
||||||
|
func init() {
|
||||||
|
db.RegisterModel(new(Workflow))
|
||||||
|
}
|
||||||
|
|
||||||
|
func FindWorkflowsByProjectID(ctx context.Context, projectID int64) ([]*Workflow, error) {
|
||||||
|
workflows := make([]*Workflow, 0)
|
||||||
|
if err := db.GetEngine(ctx).Where("project_id=?", projectID).Find(&workflows); err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
return workflows, nil
|
||||||
|
}
|
||||||
|
|
||||||
|
func GetWorkflowByID(ctx context.Context, id int64) (*Workflow, error) {
|
||||||
|
p, exist, err := db.GetByID[Workflow](ctx, id)
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
if !exist {
|
||||||
|
return nil, db.ErrNotExist{Resource: "ProjectWorkflow", ID: id}
|
||||||
|
}
|
||||||
|
return p, nil
|
||||||
|
}
|
||||||
|
|
||||||
|
func CreateWorkflow(ctx context.Context, wf *Workflow) error {
|
||||||
|
return db.Insert(ctx, wf)
|
||||||
|
}
|
||||||
|
|
||||||
|
func UpdateWorkflow(ctx context.Context, wf *Workflow) error {
|
||||||
|
_, err := db.GetEngine(ctx).ID(wf.ID).Cols("workflow_filters", "workflow_actions").Update(wf)
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
|
||||||
|
func DeleteWorkflow(ctx context.Context, id int64) error {
|
||||||
|
_, err := db.GetEngine(ctx).ID(id).Delete(&Workflow{})
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
|
||||||
|
func EnableWorkflow(ctx context.Context, id int64) error {
|
||||||
|
_, err := db.GetEngine(ctx).ID(id).Cols("enabled").Update(&Workflow{Enabled: true})
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
|
||||||
|
func DisableWorkflow(ctx context.Context, id int64) error {
|
||||||
|
_, err := db.GetEngine(ctx).ID(id).Cols("enabled").Update(&Workflow{Enabled: false})
|
||||||
|
return err
|
||||||
|
}
|
||||||
@ -0,0 +1,310 @@
|
|||||||
|
// Copyright 2025 The Gitea Authors. All rights reserved.
|
||||||
|
// SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
package project
|
||||||
|
|
||||||
|
import (
|
||||||
|
"strconv"
|
||||||
|
"testing"
|
||||||
|
|
||||||
|
"code.gitea.io/gitea/models/db"
|
||||||
|
"code.gitea.io/gitea/models/unittest"
|
||||||
|
|
||||||
|
"github.com/stretchr/testify/assert"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestIsValidWorkflowEvent(t *testing.T) {
|
||||||
|
tests := []struct {
|
||||||
|
event string
|
||||||
|
valid bool
|
||||||
|
}{
|
||||||
|
{string(WorkflowEventItemOpened), true},
|
||||||
|
{string(WorkflowEventItemClosed), true},
|
||||||
|
{string(WorkflowEventItemReopened), true},
|
||||||
|
{string(WorkflowEventItemAddedToProject), true},
|
||||||
|
{string(WorkflowEventItemRemovedFromProject), true},
|
||||||
|
{string(WorkflowEventItemColumnChanged), true},
|
||||||
|
{string(WorkflowEventCodeChangesRequested), true},
|
||||||
|
{string(WorkflowEventCodeReviewApproved), true},
|
||||||
|
{string(WorkflowEventPullRequestMerged), true},
|
||||||
|
{"invalid_event", false},
|
||||||
|
{"", false},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, tt := range tests {
|
||||||
|
t.Run(tt.event, func(t *testing.T) {
|
||||||
|
result := IsValidWorkflowEvent(tt.event)
|
||||||
|
assert.Equal(t, tt.valid, result, "Event: %s", tt.event)
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestWorkflowEventLangKey(t *testing.T) {
|
||||||
|
tests := []struct {
|
||||||
|
event WorkflowEvent
|
||||||
|
langKey string
|
||||||
|
}{
|
||||||
|
{WorkflowEventItemOpened, "projects.workflows.event.item_opened"},
|
||||||
|
{WorkflowEventItemClosed, "projects.workflows.event.item_closed"},
|
||||||
|
{WorkflowEventItemReopened, "projects.workflows.event.item_reopened"},
|
||||||
|
{WorkflowEventItemAddedToProject, "projects.workflows.event.item_added_to_project"},
|
||||||
|
{WorkflowEventItemRemovedFromProject, "projects.workflows.event.item_removed_from_project"},
|
||||||
|
{WorkflowEventItemColumnChanged, "projects.workflows.event.item_column_changed"},
|
||||||
|
{WorkflowEventCodeChangesRequested, "projects.workflows.event.code_changes_requested"},
|
||||||
|
{WorkflowEventCodeReviewApproved, "projects.workflows.event.code_review_approved"},
|
||||||
|
{WorkflowEventPullRequestMerged, "projects.workflows.event.pull_request_merged"},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, tt := range tests {
|
||||||
|
t.Run(string(tt.event), func(t *testing.T) {
|
||||||
|
result := tt.event.LangKey()
|
||||||
|
assert.Equal(t, tt.langKey, result)
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestGetWorkflowEventCapabilities(t *testing.T) {
|
||||||
|
capabilities := GetWorkflowEventCapabilities()
|
||||||
|
|
||||||
|
// Verify all events have capabilities
|
||||||
|
assert.Len(t, capabilities, 9, "Should have capabilities for all 9 workflow events")
|
||||||
|
|
||||||
|
// Test item_opened event
|
||||||
|
itemOpenedCap := capabilities[WorkflowEventItemOpened]
|
||||||
|
assert.Contains(t, itemOpenedCap.AvailableFilters, WorkflowFilterTypeIssueType)
|
||||||
|
assert.Contains(t, itemOpenedCap.AvailableFilters, WorkflowFilterTypeLabels)
|
||||||
|
assert.Contains(t, itemOpenedCap.AvailableActions, WorkflowActionTypeColumn)
|
||||||
|
assert.Contains(t, itemOpenedCap.AvailableActions, WorkflowActionTypeAddLabels)
|
||||||
|
|
||||||
|
// Test item_column_changed event (should have the most filters)
|
||||||
|
columnChangedCap := capabilities[WorkflowEventItemColumnChanged]
|
||||||
|
assert.Contains(t, columnChangedCap.AvailableFilters, WorkflowFilterTypeIssueType)
|
||||||
|
assert.Contains(t, columnChangedCap.AvailableFilters, WorkflowFilterTypeSourceColumn)
|
||||||
|
assert.Contains(t, columnChangedCap.AvailableFilters, WorkflowFilterTypeTargetColumn)
|
||||||
|
assert.Contains(t, columnChangedCap.AvailableFilters, WorkflowFilterTypeLabels)
|
||||||
|
|
||||||
|
// Test code review events (should not have issue state action)
|
||||||
|
codeReviewCap := capabilities[WorkflowEventCodeReviewApproved]
|
||||||
|
assert.NotContains(t, codeReviewCap.AvailableActions, WorkflowActionTypeIssueState)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestCreateWorkflow(t *testing.T) {
|
||||||
|
assert.NoError(t, unittest.PrepareTestDatabase())
|
||||||
|
|
||||||
|
// Get an existing project from fixtures
|
||||||
|
project := unittest.AssertExistsAndLoadBean(t, &Project{ID: 1})
|
||||||
|
|
||||||
|
// Create a column for the project
|
||||||
|
column := &Column{
|
||||||
|
Title: "Test Column",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
err := NewColumn(t.Context(), column)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create a workflow
|
||||||
|
workflow := &Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []WorkflowFilter{
|
||||||
|
{
|
||||||
|
Type: WorkflowFilterTypeIssueType,
|
||||||
|
Value: "issue",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
WorkflowActions: []WorkflowAction{
|
||||||
|
{
|
||||||
|
Type: WorkflowActionTypeColumn,
|
||||||
|
Value: strconv.FormatInt(column.ID, 10),
|
||||||
|
},
|
||||||
|
},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
|
||||||
|
err = CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.NotZero(t, workflow.ID, "Workflow ID should be set after creation")
|
||||||
|
|
||||||
|
// Verify the workflow was created
|
||||||
|
createdWorkflow, err := GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Equal(t, project.ID, createdWorkflow.ProjectID)
|
||||||
|
assert.Equal(t, WorkflowEventItemOpened, createdWorkflow.WorkflowEvent)
|
||||||
|
assert.True(t, createdWorkflow.Enabled)
|
||||||
|
assert.Len(t, createdWorkflow.WorkflowFilters, 1)
|
||||||
|
assert.Len(t, createdWorkflow.WorkflowActions, 1)
|
||||||
|
assert.Equal(t, WorkflowFilterTypeIssueType, createdWorkflow.WorkflowFilters[0].Type)
|
||||||
|
assert.Equal(t, "issue", createdWorkflow.WorkflowFilters[0].Value)
|
||||||
|
assert.Equal(t, WorkflowActionTypeColumn, createdWorkflow.WorkflowActions[0].Type)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestUpdateWorkflow(t *testing.T) {
|
||||||
|
assert.NoError(t, unittest.PrepareTestDatabase())
|
||||||
|
|
||||||
|
// Get an existing project from fixtures
|
||||||
|
project := unittest.AssertExistsAndLoadBean(t, &Project{ID: 1})
|
||||||
|
|
||||||
|
// Create a workflow
|
||||||
|
workflow := &Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []WorkflowFilter{},
|
||||||
|
WorkflowActions: []WorkflowAction{},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err := CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Update the workflow
|
||||||
|
workflow.WorkflowFilters = []WorkflowFilter{
|
||||||
|
{
|
||||||
|
Type: WorkflowFilterTypeIssueType,
|
||||||
|
Value: "pull_request",
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
err = UpdateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Verify the update
|
||||||
|
updatedWorkflow, err := GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, updatedWorkflow.Enabled)
|
||||||
|
assert.Len(t, updatedWorkflow.WorkflowFilters, 1)
|
||||||
|
assert.Equal(t, "pull_request", updatedWorkflow.WorkflowFilters[0].Value)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestDeleteWorkflow(t *testing.T) {
|
||||||
|
assert.NoError(t, unittest.PrepareTestDatabase())
|
||||||
|
|
||||||
|
// Get an existing project from fixtures
|
||||||
|
project := unittest.AssertExistsAndLoadBean(t, &Project{ID: 1})
|
||||||
|
|
||||||
|
// Create a workflow
|
||||||
|
workflow := &Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []WorkflowFilter{},
|
||||||
|
WorkflowActions: []WorkflowAction{},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err := CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
workflowID := workflow.ID
|
||||||
|
|
||||||
|
// Delete the workflow
|
||||||
|
err = DeleteWorkflow(t.Context(), workflowID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Verify it was deleted
|
||||||
|
_, err = GetWorkflowByID(t.Context(), workflowID)
|
||||||
|
assert.Error(t, err)
|
||||||
|
assert.True(t, db.IsErrNotExist(err), "Should return ErrNotExist")
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestEnableDisableWorkflow(t *testing.T) {
|
||||||
|
assert.NoError(t, unittest.PrepareTestDatabase())
|
||||||
|
|
||||||
|
// Get an existing project from fixtures
|
||||||
|
project := unittest.AssertExistsAndLoadBean(t, &Project{ID: 1})
|
||||||
|
|
||||||
|
// Create a workflow (enabled by default)
|
||||||
|
workflow := &Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []WorkflowFilter{},
|
||||||
|
WorkflowActions: []WorkflowAction{},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err := CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Test Disable
|
||||||
|
err = DisableWorkflow(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
disabledWorkflow, err := GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.False(t, disabledWorkflow.Enabled)
|
||||||
|
|
||||||
|
// Test Enable
|
||||||
|
err = EnableWorkflow(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
enabledWorkflow, err := GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, enabledWorkflow.Enabled)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestFindWorkflowsByProjectID(t *testing.T) {
|
||||||
|
assert.NoError(t, unittest.PrepareTestDatabase())
|
||||||
|
|
||||||
|
// Get an existing project from fixtures
|
||||||
|
project := unittest.AssertExistsAndLoadBean(t, &Project{ID: 1})
|
||||||
|
|
||||||
|
// Create multiple workflows
|
||||||
|
workflow1 := &Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []WorkflowFilter{},
|
||||||
|
WorkflowActions: []WorkflowAction{},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err := CreateWorkflow(t.Context(), workflow1)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
workflow2 := &Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: WorkflowEventItemClosed,
|
||||||
|
WorkflowFilters: []WorkflowFilter{},
|
||||||
|
WorkflowActions: []WorkflowAction{},
|
||||||
|
Enabled: false,
|
||||||
|
}
|
||||||
|
err = CreateWorkflow(t.Context(), workflow2)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Find all workflows for the project
|
||||||
|
workflows, err := FindWorkflowsByProjectID(t.Context(), project.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Len(t, workflows, 2)
|
||||||
|
|
||||||
|
// Verify the workflows
|
||||||
|
assert.Equal(t, WorkflowEventItemOpened, workflows[0].WorkflowEvent)
|
||||||
|
assert.True(t, workflows[0].Enabled)
|
||||||
|
assert.Equal(t, WorkflowEventItemClosed, workflows[1].WorkflowEvent)
|
||||||
|
assert.False(t, workflows[1].Enabled)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestWorkflowLoadProject(t *testing.T) {
|
||||||
|
assert.NoError(t, unittest.PrepareTestDatabase())
|
||||||
|
|
||||||
|
// Get an existing project from fixtures
|
||||||
|
project := unittest.AssertExistsAndLoadBean(t, &Project{ID: 1})
|
||||||
|
|
||||||
|
// Create a workflow
|
||||||
|
workflow := &Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []WorkflowFilter{},
|
||||||
|
WorkflowActions: []WorkflowAction{},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err := CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Get the workflow
|
||||||
|
loadedWorkflow, err := GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Nil(t, loadedWorkflow.Project)
|
||||||
|
|
||||||
|
// Load the project
|
||||||
|
err = loadedWorkflow.LoadProject(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.NotNil(t, loadedWorkflow.Project)
|
||||||
|
assert.Equal(t, project.ID, loadedWorkflow.Project.ID)
|
||||||
|
|
||||||
|
// Load again should not error
|
||||||
|
err = loadedWorkflow.LoadProject(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
}
|
||||||
@ -0,0 +1,624 @@
|
|||||||
|
// Copyright 2025 The Gitea Authors. All rights reserved.
|
||||||
|
// SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
package projects
|
||||||
|
|
||||||
|
import (
|
||||||
|
"io"
|
||||||
|
"net/http"
|
||||||
|
"strconv"
|
||||||
|
"strings"
|
||||||
|
|
||||||
|
"code.gitea.io/gitea/models/db"
|
||||||
|
issues_model "code.gitea.io/gitea/models/issues"
|
||||||
|
project_model "code.gitea.io/gitea/models/project"
|
||||||
|
"code.gitea.io/gitea/modules/json"
|
||||||
|
"code.gitea.io/gitea/modules/log"
|
||||||
|
"code.gitea.io/gitea/modules/templates"
|
||||||
|
"code.gitea.io/gitea/services/context"
|
||||||
|
project_service "code.gitea.io/gitea/services/projects"
|
||||||
|
)
|
||||||
|
|
||||||
|
var (
|
||||||
|
tmplRepoWorkflows = templates.TplName("repo/projects/workflows")
|
||||||
|
tmplOrgWorkflows = templates.TplName("org/projects/workflows")
|
||||||
|
)
|
||||||
|
|
||||||
|
// convertFormToFilters converts form filters to WorkflowFilter objects
|
||||||
|
func convertFormToFilters(formFilters map[string]any) []project_model.WorkflowFilter {
|
||||||
|
filters := make([]project_model.WorkflowFilter, 0)
|
||||||
|
|
||||||
|
for key, value := range formFilters {
|
||||||
|
switch key {
|
||||||
|
case "labels":
|
||||||
|
// Handle labels array
|
||||||
|
if labelInterfaces, ok := value.([]any); ok && len(labelInterfaces) > 0 {
|
||||||
|
for _, labelInterface := range labelInterfaces {
|
||||||
|
if label, ok := labelInterface.(string); ok && label != "" {
|
||||||
|
filters = append(filters, project_model.WorkflowFilter{
|
||||||
|
Type: project_model.WorkflowFilterTypeLabels,
|
||||||
|
Value: label,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
default:
|
||||||
|
// Handle string values (issue_type, column)
|
||||||
|
if strValue, ok := value.(string); ok && strValue != "" {
|
||||||
|
filters = append(filters, project_model.WorkflowFilter{
|
||||||
|
Type: project_model.WorkflowFilterType(key),
|
||||||
|
Value: strValue,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return filters
|
||||||
|
}
|
||||||
|
|
||||||
|
// convertFormToActions converts form actions to WorkflowAction objects
|
||||||
|
func convertFormToActions(formActions map[string]any) []project_model.WorkflowAction {
|
||||||
|
actions := make([]project_model.WorkflowAction, 0)
|
||||||
|
|
||||||
|
for key, value := range formActions {
|
||||||
|
switch key {
|
||||||
|
case "column":
|
||||||
|
if floatValue, ok := value.(string); ok {
|
||||||
|
floatValueInt, _ := strconv.ParseInt(floatValue, 10, 64)
|
||||||
|
if floatValueInt > 0 {
|
||||||
|
actions = append(actions, project_model.WorkflowAction{
|
||||||
|
Type: project_model.WorkflowActionTypeColumn,
|
||||||
|
Value: strconv.FormatInt(floatValueInt, 10),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
case "add_labels":
|
||||||
|
// Handle both []string and []any from JSON unmarshaling
|
||||||
|
if labels, ok := value.([]string); ok && len(labels) > 0 {
|
||||||
|
for _, label := range labels {
|
||||||
|
if label != "" {
|
||||||
|
actions = append(actions, project_model.WorkflowAction{
|
||||||
|
Type: project_model.WorkflowActionTypeAddLabels,
|
||||||
|
Value: label,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} else if labelInterfaces, ok := value.([]any); ok && len(labelInterfaces) > 0 {
|
||||||
|
for _, labelInterface := range labelInterfaces {
|
||||||
|
if label, ok := labelInterface.(string); ok && label != "" {
|
||||||
|
actions = append(actions, project_model.WorkflowAction{
|
||||||
|
Type: project_model.WorkflowActionTypeAddLabels,
|
||||||
|
Value: label,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
case "remove_labels":
|
||||||
|
// Handle both []string and []any from JSON unmarshaling
|
||||||
|
if labels, ok := value.([]string); ok && len(labels) > 0 {
|
||||||
|
for _, label := range labels {
|
||||||
|
if label != "" {
|
||||||
|
actions = append(actions, project_model.WorkflowAction{
|
||||||
|
Type: project_model.WorkflowActionTypeRemoveLabels,
|
||||||
|
Value: label,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} else if labelInterfaces, ok := value.([]any); ok && len(labelInterfaces) > 0 {
|
||||||
|
for _, labelInterface := range labelInterfaces {
|
||||||
|
if label, ok := labelInterface.(string); ok && label != "" {
|
||||||
|
actions = append(actions, project_model.WorkflowAction{
|
||||||
|
Type: project_model.WorkflowActionTypeRemoveLabels,
|
||||||
|
Value: label,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
case "issue_state":
|
||||||
|
if strValue, ok := value.(string); ok {
|
||||||
|
v := strings.ToLower(strValue)
|
||||||
|
if v == "close" || v == "reopen" {
|
||||||
|
actions = append(actions, project_model.WorkflowAction{
|
||||||
|
Type: project_model.WorkflowActionTypeIssueState,
|
||||||
|
Value: v,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return actions
|
||||||
|
}
|
||||||
|
|
||||||
|
func WorkflowsEvents(ctx *context.Context) {
|
||||||
|
projectID := ctx.PathParamInt64("id")
|
||||||
|
p, err := project_model.GetProjectByID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
if project_model.IsErrProjectNotExist(err) {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
} else {
|
||||||
|
ctx.ServerError("GetProjectByID", err)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if p.Type == project_model.TypeRepository && p.RepoID != ctx.Repo.Repository.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if (p.Type == project_model.TypeOrganization || p.Type == project_model.TypeIndividual) && p.OwnerID != ctx.ContextUser.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
ctx.ServerError("FindWorkflowsByProjectID", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
type WorkflowConfig struct {
|
||||||
|
ID int64 `json:"id"`
|
||||||
|
EventID string `json:"event_id"`
|
||||||
|
DisplayName string `json:"display_name"`
|
||||||
|
WorkflowEvent string `json:"workflow_event"` // The workflow event
|
||||||
|
Capabilities project_model.WorkflowEventCapabilities `json:"capabilities"`
|
||||||
|
Filters []project_model.WorkflowFilter `json:"filters"`
|
||||||
|
Actions []project_model.WorkflowAction `json:"actions"`
|
||||||
|
Summary string `json:"summary"` // Human readable filter description
|
||||||
|
Enabled bool `json:"enabled"`
|
||||||
|
IsConfigured bool `json:"isConfigured"` // Whether this workflow is configured/saved
|
||||||
|
}
|
||||||
|
|
||||||
|
outputWorkflows := make([]*WorkflowConfig, 0)
|
||||||
|
events := project_model.GetWorkflowEvents()
|
||||||
|
capabilities := project_model.GetWorkflowEventCapabilities()
|
||||||
|
|
||||||
|
// Create a map for quick lookup of existing workflows
|
||||||
|
workflowMap := make(map[project_model.WorkflowEvent][]*project_model.Workflow)
|
||||||
|
for _, wf := range workflows {
|
||||||
|
workflowMap[wf.WorkflowEvent] = append(workflowMap[wf.WorkflowEvent], wf)
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, event := range events {
|
||||||
|
existingWorkflows := workflowMap[event]
|
||||||
|
if len(existingWorkflows) > 0 {
|
||||||
|
// Add all existing workflows for this event
|
||||||
|
for _, wf := range existingWorkflows {
|
||||||
|
workflowSummary := project_service.GetWorkflowSummary(ctx, wf)
|
||||||
|
outputWorkflows = append(outputWorkflows, &WorkflowConfig{
|
||||||
|
ID: wf.ID,
|
||||||
|
EventID: strconv.FormatInt(wf.ID, 10),
|
||||||
|
DisplayName: string(ctx.Tr(wf.WorkflowEvent.LangKey())),
|
||||||
|
WorkflowEvent: string(wf.WorkflowEvent),
|
||||||
|
Capabilities: capabilities[event],
|
||||||
|
Filters: wf.WorkflowFilters,
|
||||||
|
Actions: wf.WorkflowActions,
|
||||||
|
Summary: workflowSummary,
|
||||||
|
Enabled: wf.Enabled,
|
||||||
|
IsConfigured: true,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
// Add placeholder for creating new workflow
|
||||||
|
outputWorkflows = append(outputWorkflows, &WorkflowConfig{
|
||||||
|
ID: 0,
|
||||||
|
EventID: event.EventID(),
|
||||||
|
DisplayName: string(ctx.Tr(event.LangKey())),
|
||||||
|
WorkflowEvent: string(event),
|
||||||
|
Capabilities: capabilities[event],
|
||||||
|
Summary: "",
|
||||||
|
Enabled: true, // Default to enabled for new workflows
|
||||||
|
IsConfigured: false,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
ctx.JSON(http.StatusOK, outputWorkflows)
|
||||||
|
}
|
||||||
|
|
||||||
|
func WorkflowsColumns(ctx *context.Context) {
|
||||||
|
projectID := ctx.PathParamInt64("id")
|
||||||
|
p, err := project_model.GetProjectByID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
if project_model.IsErrProjectNotExist(err) {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
} else {
|
||||||
|
ctx.ServerError("GetProjectByID", err)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if p.Type == project_model.TypeRepository && p.RepoID != ctx.Repo.Repository.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if (p.Type == project_model.TypeOrganization || p.Type == project_model.TypeIndividual) && p.OwnerID != ctx.ContextUser.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
columns, err := p.GetColumns(ctx)
|
||||||
|
if err != nil {
|
||||||
|
ctx.ServerError("GetProjectColumns", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
type Column struct {
|
||||||
|
ID int64 `json:"id"`
|
||||||
|
Title string `json:"title"`
|
||||||
|
Color string `json:"color"`
|
||||||
|
}
|
||||||
|
outputColumns := make([]*Column, 0, len(columns))
|
||||||
|
for _, col := range columns {
|
||||||
|
outputColumns = append(outputColumns, &Column{
|
||||||
|
ID: col.ID,
|
||||||
|
Title: col.Title,
|
||||||
|
Color: col.Color,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
ctx.JSON(http.StatusOK, outputColumns)
|
||||||
|
}
|
||||||
|
|
||||||
|
func WorkflowsLabels(ctx *context.Context) {
|
||||||
|
projectID := ctx.PathParamInt64("id")
|
||||||
|
p, err := project_model.GetProjectByID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
if project_model.IsErrProjectNotExist(err) {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
} else {
|
||||||
|
ctx.ServerError("GetProjectByID", err)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Only repository projects have access to labels
|
||||||
|
if p.Type != project_model.TypeRepository {
|
||||||
|
ctx.JSON(http.StatusOK, []any{})
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if p.RepoID != ctx.Repo.Repository.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Get repository labels
|
||||||
|
labels, err := issues_model.GetLabelsByRepoID(ctx, p.RepoID, "", db.ListOptions{})
|
||||||
|
if err != nil {
|
||||||
|
ctx.ServerError("GetLabelsByRepoID", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
type Label struct {
|
||||||
|
ID int64 `json:"id"`
|
||||||
|
Name string `json:"name"`
|
||||||
|
Color string `json:"color"`
|
||||||
|
Description string `json:"description"`
|
||||||
|
Exclusive bool `json:"exclusive"`
|
||||||
|
ExclusiveOrder int `json:"exclusiveOrder"`
|
||||||
|
}
|
||||||
|
outputLabels := make([]*Label, 0, len(labels))
|
||||||
|
for _, label := range labels {
|
||||||
|
outputLabels = append(outputLabels, &Label{
|
||||||
|
ID: label.ID,
|
||||||
|
Name: label.Name,
|
||||||
|
Color: label.Color,
|
||||||
|
Description: label.Description,
|
||||||
|
Exclusive: label.Exclusive,
|
||||||
|
ExclusiveOrder: label.ExclusiveOrder,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
ctx.JSON(http.StatusOK, outputLabels)
|
||||||
|
}
|
||||||
|
|
||||||
|
func Workflows(ctx *context.Context) {
|
||||||
|
workflowIDStr := ctx.PathParam("workflow_id")
|
||||||
|
if workflowIDStr == "events" {
|
||||||
|
WorkflowsEvents(ctx)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if workflowIDStr == "columns" {
|
||||||
|
WorkflowsColumns(ctx)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if workflowIDStr == "labels" {
|
||||||
|
WorkflowsLabels(ctx)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
ctx.Data["WorkflowEvents"] = project_model.GetWorkflowEvents()
|
||||||
|
|
||||||
|
projectID := ctx.PathParamInt64("id")
|
||||||
|
p, err := project_model.GetProjectByID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
if project_model.IsErrProjectNotExist(err) {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
} else {
|
||||||
|
ctx.ServerError("GetProjectByID", err)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if p.Type == project_model.TypeRepository && p.RepoID != ctx.Repo.Repository.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if (p.Type == project_model.TypeOrganization || p.Type == project_model.TypeIndividual) && p.OwnerID != ctx.ContextUser.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
ctx.Data["Title"] = ctx.Tr("projects.workflows")
|
||||||
|
ctx.Data["IsProjectsPage"] = true
|
||||||
|
ctx.Data["Project"] = p
|
||||||
|
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
ctx.ServerError("FindWorkflowsByProjectID", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
for _, wf := range workflows {
|
||||||
|
wf.Project = p
|
||||||
|
}
|
||||||
|
ctx.Data["Workflows"] = workflows
|
||||||
|
|
||||||
|
ctx.Data["workflowIDStr"] = workflowIDStr
|
||||||
|
var curWorkflow *project_model.Workflow
|
||||||
|
if workflowIDStr == "" { // get first value workflow or the first workflow
|
||||||
|
for _, wf := range workflows {
|
||||||
|
if wf.ID > 0 {
|
||||||
|
curWorkflow = wf
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
workflowID, _ := strconv.ParseInt(workflowIDStr, 10, 64)
|
||||||
|
if workflowID > 0 {
|
||||||
|
for _, wf := range workflows {
|
||||||
|
if wf.ID == workflowID {
|
||||||
|
curWorkflow = wf
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
ctx.Data["CurWorkflow"] = curWorkflow
|
||||||
|
ctx.Data["ProjectLink"] = project_model.ProjectLinkForRepo(ctx.Repo.Repository, projectID)
|
||||||
|
|
||||||
|
if p.Type == project_model.TypeRepository {
|
||||||
|
ctx.HTML(200, tmplRepoWorkflows)
|
||||||
|
} else {
|
||||||
|
ctx.HTML(200, tmplOrgWorkflows)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
type WorkflowsPostForm struct {
|
||||||
|
EventID string `json:"event_id"`
|
||||||
|
Filters map[string]any `json:"filters"`
|
||||||
|
Actions map[string]any `json:"actions"`
|
||||||
|
}
|
||||||
|
|
||||||
|
// WorkflowsPost handles creating or updating a workflow
|
||||||
|
func WorkflowsPost(ctx *context.Context) {
|
||||||
|
projectID := ctx.PathParamInt64("id")
|
||||||
|
p, err := project_model.GetProjectByID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
if project_model.IsErrProjectNotExist(err) {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
} else {
|
||||||
|
ctx.ServerError("GetProjectByID", err)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if p.Type == project_model.TypeRepository && p.RepoID != ctx.Repo.Repository.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if (p.Type == project_model.TypeOrganization || p.Type == project_model.TypeIndividual) && p.OwnerID != ctx.ContextUser.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Handle both form data and JSON data
|
||||||
|
// Handle JSON data
|
||||||
|
form := &WorkflowsPostForm{}
|
||||||
|
content, err := io.ReadAll(ctx.Req.Body)
|
||||||
|
if err != nil {
|
||||||
|
ctx.ServerError("ReadRequestBody", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
defer ctx.Req.Body.Close()
|
||||||
|
log.Trace("get " + string(content))
|
||||||
|
if err := json.Unmarshal(content, &form); err != nil {
|
||||||
|
ctx.ServerError("DecodeWorkflowsPostForm", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if form.EventID == "" {
|
||||||
|
ctx.JSON(http.StatusBadRequest, map[string]any{"error": "InvalidEventID", "message": "EventID is required"})
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Convert form data to filters and actions
|
||||||
|
filters := convertFormToFilters(form.Filters)
|
||||||
|
actions := convertFormToActions(form.Actions)
|
||||||
|
|
||||||
|
// Validate: at least one action must be configured
|
||||||
|
if len(actions) == 0 {
|
||||||
|
ctx.JSON(http.StatusBadRequest, map[string]any{
|
||||||
|
"error": "NoActions",
|
||||||
|
"message": ctx.Tr("projects.workflows.error.at_least_one_action"),
|
||||||
|
})
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
eventID, _ := strconv.ParseInt(form.EventID, 10, 64)
|
||||||
|
if eventID == 0 {
|
||||||
|
// check if workflow event is valid
|
||||||
|
if !project_model.IsValidWorkflowEvent(form.EventID) {
|
||||||
|
ctx.JSON(http.StatusBadRequest, map[string]any{"error": "EventID is invalid"})
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Create a new workflow for the given event
|
||||||
|
wf := &project_model.Workflow{
|
||||||
|
ProjectID: projectID,
|
||||||
|
WorkflowEvent: project_model.WorkflowEvent(form.EventID),
|
||||||
|
WorkflowFilters: filters,
|
||||||
|
WorkflowActions: actions,
|
||||||
|
Enabled: true, // New workflows are enabled by default
|
||||||
|
}
|
||||||
|
if err := project_model.CreateWorkflow(ctx, wf); err != nil {
|
||||||
|
ctx.ServerError("CreateWorkflow", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Return the newly created workflow with filter summary
|
||||||
|
workflowSummary := project_service.GetWorkflowSummary(ctx, wf)
|
||||||
|
ctx.JSON(http.StatusOK, map[string]any{
|
||||||
|
"success": true,
|
||||||
|
"workflow": map[string]any{
|
||||||
|
"id": wf.ID,
|
||||||
|
"event_id": strconv.FormatInt(wf.ID, 10),
|
||||||
|
"display_name": string(ctx.Tr(wf.WorkflowEvent.LangKey())),
|
||||||
|
"filters": wf.WorkflowFilters,
|
||||||
|
"actions": wf.WorkflowActions,
|
||||||
|
"summary": workflowSummary,
|
||||||
|
"enabled": wf.Enabled,
|
||||||
|
},
|
||||||
|
})
|
||||||
|
} else {
|
||||||
|
// Update an existing workflow
|
||||||
|
wf, err := project_model.GetWorkflowByID(ctx, eventID)
|
||||||
|
if err != nil {
|
||||||
|
ctx.ServerError("GetWorkflowByID", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if wf.ProjectID != projectID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
wf.WorkflowFilters = filters
|
||||||
|
wf.WorkflowActions = actions
|
||||||
|
if err := project_model.UpdateWorkflow(ctx, wf); err != nil {
|
||||||
|
ctx.ServerError("UpdateWorkflow", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Return the updated workflow with filter summary
|
||||||
|
workflowSummary := project_service.GetWorkflowSummary(ctx, wf)
|
||||||
|
ctx.JSON(http.StatusOK, map[string]any{
|
||||||
|
"success": true,
|
||||||
|
"workflow": map[string]any{
|
||||||
|
"id": wf.ID,
|
||||||
|
"event_id": strconv.FormatInt(wf.ID, 10),
|
||||||
|
"display_name": string(ctx.Tr(wf.WorkflowEvent.LangKey())),
|
||||||
|
"filters": wf.WorkflowFilters,
|
||||||
|
"actions": wf.WorkflowActions,
|
||||||
|
"summary": workflowSummary,
|
||||||
|
"enabled": wf.Enabled,
|
||||||
|
},
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func WorkflowsStatus(ctx *context.Context) {
|
||||||
|
projectID := ctx.PathParamInt64("id")
|
||||||
|
workflowID, _ := strconv.ParseInt(ctx.PathParam("workflow_id"), 10, 64)
|
||||||
|
|
||||||
|
p, err := project_model.GetProjectByID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
if project_model.IsErrProjectNotExist(err) {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
} else {
|
||||||
|
ctx.ServerError("GetProjectByID", err)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if p.Type == project_model.TypeRepository && p.RepoID != ctx.Repo.Repository.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if (p.Type == project_model.TypeOrganization || p.Type == project_model.TypeIndividual) && p.OwnerID != ctx.ContextUser.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
wf, err := project_model.GetWorkflowByID(ctx, workflowID)
|
||||||
|
if err != nil {
|
||||||
|
ctx.ServerError("GetWorkflowByID", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if wf.ProjectID != projectID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Get enabled status from form
|
||||||
|
_ = ctx.Req.ParseForm()
|
||||||
|
enabledStr := ctx.Req.FormValue("enabled")
|
||||||
|
enabled, _ := strconv.ParseBool(enabledStr)
|
||||||
|
|
||||||
|
if enabled {
|
||||||
|
if err := project_model.EnableWorkflow(ctx, workflowID); err != nil {
|
||||||
|
ctx.ServerError("EnableWorkflow", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
if err := project_model.DisableWorkflow(ctx, workflowID); err != nil {
|
||||||
|
ctx.ServerError("DisableWorkflow", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
ctx.JSON(http.StatusOK, map[string]any{
|
||||||
|
"success": true,
|
||||||
|
"enabled": wf.Enabled,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
func WorkflowsDelete(ctx *context.Context) {
|
||||||
|
projectID := ctx.PathParamInt64("id")
|
||||||
|
workflowID, _ := strconv.ParseInt(ctx.PathParam("workflow_id"), 10, 64)
|
||||||
|
|
||||||
|
p, err := project_model.GetProjectByID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
if project_model.IsErrProjectNotExist(err) {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
} else {
|
||||||
|
ctx.ServerError("GetProjectByID", err)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if p.Type == project_model.TypeRepository && p.RepoID != ctx.Repo.Repository.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if (p.Type == project_model.TypeOrganization || p.Type == project_model.TypeIndividual) && p.OwnerID != ctx.ContextUser.ID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
wf, err := project_model.GetWorkflowByID(ctx, workflowID)
|
||||||
|
if err != nil {
|
||||||
|
if db.IsErrNotExist(err) {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
} else {
|
||||||
|
ctx.ServerError("GetWorkflowByID", err)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if wf.ProjectID != projectID {
|
||||||
|
ctx.NotFound(nil)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if err := project_model.DeleteWorkflow(ctx, workflowID); err != nil {
|
||||||
|
ctx.ServerError("DeleteWorkflow", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
ctx.JSON(http.StatusOK, map[string]any{
|
||||||
|
"success": true,
|
||||||
|
})
|
||||||
|
}
|
||||||
@ -0,0 +1,31 @@
|
|||||||
|
// Copyright 2025 The Gitea Authors. All rights reserved.
|
||||||
|
// SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
package issue
|
||||||
|
|
||||||
|
import (
|
||||||
|
"context"
|
||||||
|
|
||||||
|
issues_model "code.gitea.io/gitea/models/issues"
|
||||||
|
project_model "code.gitea.io/gitea/models/project"
|
||||||
|
user_model "code.gitea.io/gitea/models/user"
|
||||||
|
"code.gitea.io/gitea/services/notify"
|
||||||
|
)
|
||||||
|
|
||||||
|
func AssignOrRemoveProject(ctx context.Context, issue *issues_model.Issue, doer *user_model.User, projectID int64, position int) error {
|
||||||
|
if err := issues_model.IssueAssignOrRemoveProject(ctx, issue, doer, projectID, 0); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
|
||||||
|
var newProject *project_model.Project
|
||||||
|
var err error
|
||||||
|
if projectID > 0 {
|
||||||
|
newProject, err = project_model.GetProjectByID(ctx, projectID)
|
||||||
|
if err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
notify.IssueChangeProjects(ctx, doer, issue, newProject)
|
||||||
|
return nil
|
||||||
|
}
|
||||||
@ -0,0 +1,94 @@
|
|||||||
|
// Copyright 2025 The Gitea Authors. All rights reserved.
|
||||||
|
// SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
package projects
|
||||||
|
|
||||||
|
import (
|
||||||
|
"context"
|
||||||
|
"strconv"
|
||||||
|
"strings"
|
||||||
|
|
||||||
|
issues_model "code.gitea.io/gitea/models/issues"
|
||||||
|
project_model "code.gitea.io/gitea/models/project"
|
||||||
|
"code.gitea.io/gitea/modules/log"
|
||||||
|
)
|
||||||
|
|
||||||
|
// GetWorkflowSummary returns a human-readable summary of the workflow
|
||||||
|
func GetWorkflowSummary(ctx context.Context, wf *project_model.Workflow) string {
|
||||||
|
filters := wf.WorkflowFilters
|
||||||
|
if len(filters) == 0 {
|
||||||
|
return ""
|
||||||
|
}
|
||||||
|
|
||||||
|
var summary strings.Builder
|
||||||
|
labelIDs := make([]int64, 0)
|
||||||
|
for _, filter := range filters {
|
||||||
|
switch filter.Type {
|
||||||
|
case project_model.WorkflowFilterTypeIssueType:
|
||||||
|
switch filter.Value {
|
||||||
|
case "issue":
|
||||||
|
if summary.Len() > 0 {
|
||||||
|
summary.WriteString(" ")
|
||||||
|
}
|
||||||
|
summary.WriteString("(Issues only)")
|
||||||
|
case "pull_request":
|
||||||
|
if summary.Len() > 0 {
|
||||||
|
summary.WriteString(" ")
|
||||||
|
}
|
||||||
|
summary.WriteString("(Pull requests only)")
|
||||||
|
}
|
||||||
|
case project_model.WorkflowFilterTypeSourceColumn:
|
||||||
|
columnID, _ := strconv.ParseInt(filter.Value, 10, 64)
|
||||||
|
if columnID <= 0 {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
col, err := project_model.GetColumn(ctx, columnID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("GetColumn: %v", err)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if summary.Len() > 0 {
|
||||||
|
summary.WriteString(" ")
|
||||||
|
}
|
||||||
|
summary.WriteString("(Source: " + col.Title + ")")
|
||||||
|
case project_model.WorkflowFilterTypeTargetColumn:
|
||||||
|
columnID, _ := strconv.ParseInt(filter.Value, 10, 64)
|
||||||
|
if columnID <= 0 {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
col, err := project_model.GetColumn(ctx, columnID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("GetColumn: %v", err)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if summary.Len() > 0 {
|
||||||
|
summary.WriteString(" ")
|
||||||
|
}
|
||||||
|
summary.WriteString("(Target: " + col.Title + ")")
|
||||||
|
case project_model.WorkflowFilterTypeLabels:
|
||||||
|
labelID, _ := strconv.ParseInt(filter.Value, 10, 64)
|
||||||
|
if labelID > 0 {
|
||||||
|
labelIDs = append(labelIDs, labelID)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if len(labelIDs) > 0 {
|
||||||
|
labels, err := issues_model.GetLabelsByIDs(ctx, labelIDs)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("GetLabelsByIDs: %v", err)
|
||||||
|
} else {
|
||||||
|
if summary.Len() > 0 {
|
||||||
|
summary.WriteString(" ")
|
||||||
|
}
|
||||||
|
summary.WriteString("(Labels: ")
|
||||||
|
for i, label := range labels {
|
||||||
|
summary.WriteString(label.Name)
|
||||||
|
if i < len(labels)-1 {
|
||||||
|
summary.WriteString(", ")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
summary.WriteString(")")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return summary.String()
|
||||||
|
}
|
||||||
@ -0,0 +1,417 @@
|
|||||||
|
// Copyright 2024 The Gitea Authors. All rights reserved.
|
||||||
|
// SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
package projects
|
||||||
|
|
||||||
|
import (
|
||||||
|
"context"
|
||||||
|
"strconv"
|
||||||
|
"strings"
|
||||||
|
|
||||||
|
issues_model "code.gitea.io/gitea/models/issues"
|
||||||
|
project_model "code.gitea.io/gitea/models/project"
|
||||||
|
user_model "code.gitea.io/gitea/models/user"
|
||||||
|
"code.gitea.io/gitea/modules/log"
|
||||||
|
"code.gitea.io/gitea/modules/util"
|
||||||
|
issue_service "code.gitea.io/gitea/services/issue"
|
||||||
|
notify_service "code.gitea.io/gitea/services/notify"
|
||||||
|
)
|
||||||
|
|
||||||
|
func init() {
|
||||||
|
notify_service.RegisterNotifier(&workflowNotifier{})
|
||||||
|
}
|
||||||
|
|
||||||
|
type workflowNotifier struct {
|
||||||
|
notify_service.NullNotifier
|
||||||
|
}
|
||||||
|
|
||||||
|
var _ notify_service.Notifier = &workflowNotifier{}
|
||||||
|
|
||||||
|
// NewNotifier create a new workflowNotifier notifier
|
||||||
|
func NewNotifier() notify_service.Notifier {
|
||||||
|
return &workflowNotifier{}
|
||||||
|
}
|
||||||
|
|
||||||
|
func (m *workflowNotifier) NewIssue(ctx context.Context, issue *issues_model.Issue, mentions []*user_model.User) {
|
||||||
|
if err := issue.LoadRepo(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadRepo: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if err := issue.LoadProject(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadProject: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if issue.Project == nil {
|
||||||
|
// TODO: handle item opened
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(ctx, issue.Project.ID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("NewIssue: FindWorkflowsByProjectID: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Find workflows for the ItemOpened event
|
||||||
|
for _, workflow := range workflows {
|
||||||
|
if workflow.WorkflowEvent == project_model.WorkflowEventItemOpened {
|
||||||
|
fireIssueWorkflow(ctx, workflow, issue, 0, 0)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func (m *workflowNotifier) NewPullRequest(ctx context.Context, pr *issues_model.PullRequest, mentions []*user_model.User) {
|
||||||
|
if err := pr.LoadIssue(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadIssue: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
issue := pr.Issue
|
||||||
|
m.NewIssue(ctx, issue, mentions)
|
||||||
|
}
|
||||||
|
|
||||||
|
func (m *workflowNotifier) IssueChangeStatus(ctx context.Context, doer *user_model.User, commitID string, issue *issues_model.Issue, actionComment *issues_model.Comment, isClosed bool) {
|
||||||
|
if err := issue.LoadRepo(ctx); err != nil {
|
||||||
|
log.Error("IssueChangeStatus: LoadRepo: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if err := issue.LoadProject(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadProject: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if issue.Project == nil {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(ctx, issue.Project.ID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("IssueChangeStatus: FindWorkflowsByProjectID: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
workflowEvent := util.Iif(isClosed, project_model.WorkflowEventItemClosed, project_model.WorkflowEventItemReopened)
|
||||||
|
// Find workflows for the specific event
|
||||||
|
for _, workflow := range workflows {
|
||||||
|
if workflow.WorkflowEvent == workflowEvent {
|
||||||
|
fireIssueWorkflow(ctx, workflow, issue, 0, 0)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func (*workflowNotifier) IssueChangeProjects(ctx context.Context, doer *user_model.User, issue *issues_model.Issue, newProject *project_model.Project) {
|
||||||
|
if newProject == nil { // removed from project
|
||||||
|
if err := issue.LoadProject(ctx); err != nil {
|
||||||
|
log.Error("LoadProject: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if issue.Project == nil {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(ctx, issue.Project.ID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("IssueChangeStatus: FindWorkflowsByProjectID: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Find workflows for the ItemOpened event
|
||||||
|
for _, workflow := range workflows {
|
||||||
|
if workflow.WorkflowEvent == project_model.WorkflowEventItemRemovedFromProject {
|
||||||
|
fireIssueWorkflow(ctx, workflow, issue, 0, 0)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if err := issue.LoadRepo(ctx); err != nil {
|
||||||
|
log.Error("IssueChangeStatus: LoadRepo: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if err := issue.LoadProject(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadProject: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if issue.Project == nil || issue.Project.ID != newProject.ID {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(ctx, issue.Project.ID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("IssueChangeStatus: FindWorkflowsByProjectID: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Find workflows for the ItemOpened event
|
||||||
|
for _, workflow := range workflows {
|
||||||
|
if workflow.WorkflowEvent == project_model.WorkflowEventItemAddedToProject {
|
||||||
|
fireIssueWorkflow(ctx, workflow, issue, 0, 0)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func (*workflowNotifier) IssueChangeProjectColumn(ctx context.Context, doer *user_model.User, issue *issues_model.Issue, oldColumnID, newColumnID int64) {
|
||||||
|
if err := issue.LoadRepo(ctx); err != nil {
|
||||||
|
log.Error("IssueChangeStatus: LoadRepo: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if err := issue.LoadProject(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadProject: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
newColumn, err := project_model.GetColumn(ctx, newColumnID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("IssueChangeProjectColumn: GetColumn: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if issue.Project == nil || issue.Project.ID != newColumn.ProjectID {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(ctx, issue.Project.ID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("IssueChangeStatus: FindWorkflowsByProjectID: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Find workflows for the ItemColumnChanged event
|
||||||
|
for _, workflow := range workflows {
|
||||||
|
if workflow.WorkflowEvent == project_model.WorkflowEventItemColumnChanged {
|
||||||
|
fireIssueWorkflow(ctx, workflow, issue, oldColumnID, newColumnID)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func (*workflowNotifier) MergePullRequest(ctx context.Context, doer *user_model.User, pr *issues_model.PullRequest) {
|
||||||
|
if err := pr.LoadIssue(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadIssue: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
issue := pr.Issue
|
||||||
|
|
||||||
|
if err := issue.LoadRepo(ctx); err != nil {
|
||||||
|
log.Error("IssueChangeStatus: LoadRepo: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if err := issue.LoadProject(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadProject: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if issue.Project == nil {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(ctx, issue.Project.ID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("IssueChangeStatus: FindWorkflowsByProjectID: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Find workflows for the PullRequestMerged event
|
||||||
|
for _, workflow := range workflows {
|
||||||
|
if workflow.WorkflowEvent == project_model.WorkflowEventPullRequestMerged {
|
||||||
|
fireIssueWorkflow(ctx, workflow, issue, 0, 0)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func (m *workflowNotifier) AutoMergePullRequest(ctx context.Context, doer *user_model.User, pr *issues_model.PullRequest) {
|
||||||
|
m.MergePullRequest(ctx, doer, pr)
|
||||||
|
}
|
||||||
|
|
||||||
|
func (*workflowNotifier) PullRequestReview(ctx context.Context, pr *issues_model.PullRequest, review *issues_model.Review, comment *issues_model.Comment, mentions []*user_model.User) {
|
||||||
|
if err := pr.LoadIssue(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadIssue: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
issue := pr.Issue
|
||||||
|
|
||||||
|
if err := issue.LoadRepo(ctx); err != nil {
|
||||||
|
log.Error("IssueChangeStatus: LoadRepo: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if err := issue.LoadProject(ctx); err != nil {
|
||||||
|
log.Error("NewIssue: LoadProject: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if issue.Project == nil {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(ctx, issue.Project.ID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("IssueChangeStatus: FindWorkflowsByProjectID: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Find workflows for the PullRequestMerged event
|
||||||
|
for _, workflow := range workflows {
|
||||||
|
if (workflow.WorkflowEvent == project_model.WorkflowEventCodeChangesRequested && review.Type == issues_model.ReviewTypeReject) ||
|
||||||
|
(workflow.WorkflowEvent == project_model.WorkflowEventCodeReviewApproved && review.Type == issues_model.ReviewTypeApprove) {
|
||||||
|
fireIssueWorkflow(ctx, workflow, issue, 0, 0)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func fireIssueWorkflow(ctx context.Context, workflow *project_model.Workflow, issue *issues_model.Issue, sourceColumnID, targetColumnID int64) {
|
||||||
|
if !workflow.Enabled {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
// Load issue labels for labels filter
|
||||||
|
if err := issue.LoadLabels(ctx); err != nil {
|
||||||
|
log.Error("LoadLabels: %v", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if !matchWorkflowsFilters(workflow, issue, sourceColumnID, targetColumnID) {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
executeWorkflowActions(ctx, workflow, issue)
|
||||||
|
}
|
||||||
|
|
||||||
|
// matchWorkflowsFilters checks if the issue matches all filters of the workflow
|
||||||
|
func matchWorkflowsFilters(workflow *project_model.Workflow, issue *issues_model.Issue, sourceColumnID, targetColumnID int64) bool {
|
||||||
|
for _, filter := range workflow.WorkflowFilters {
|
||||||
|
switch filter.Type {
|
||||||
|
case project_model.WorkflowFilterTypeIssueType:
|
||||||
|
// If filter value is empty, match all types
|
||||||
|
if filter.Value == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
// Filter value can be "issue" or "pull_request"
|
||||||
|
if filter.Value == "issue" && issue.IsPull {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
if filter.Value == "pull_request" && !issue.IsPull {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
case project_model.WorkflowFilterTypeTargetColumn:
|
||||||
|
// If filter value is empty, match all columns
|
||||||
|
if filter.Value == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
filterColumnID, _ := strconv.ParseInt(filter.Value, 10, 64)
|
||||||
|
if filterColumnID == 0 {
|
||||||
|
log.Error("Invalid column ID: %s", filter.Value)
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
// For column changed event, check against the new column ID
|
||||||
|
if targetColumnID > 0 && targetColumnID != filterColumnID {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
case project_model.WorkflowFilterTypeSourceColumn:
|
||||||
|
// If filter value is empty, match all columns
|
||||||
|
if filter.Value == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
filterColumnID, _ := strconv.ParseInt(filter.Value, 10, 64)
|
||||||
|
if filterColumnID == 0 {
|
||||||
|
log.Error("Invalid column ID: %s", filter.Value)
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
// For column changed event, check against the new column ID
|
||||||
|
if sourceColumnID > 0 && sourceColumnID != filterColumnID {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
case project_model.WorkflowFilterTypeLabels:
|
||||||
|
// Check if issue has the specified label
|
||||||
|
labelID, _ := strconv.ParseInt(filter.Value, 10, 64)
|
||||||
|
if labelID == 0 {
|
||||||
|
log.Error("Invalid label ID: %s", filter.Value)
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
// Check if issue has this label
|
||||||
|
hasLabel := false
|
||||||
|
for _, label := range issue.Labels {
|
||||||
|
if label.ID == labelID {
|
||||||
|
hasLabel = true
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if !hasLabel {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
default:
|
||||||
|
log.Error("Unsupported filter type: %s", filter.Type)
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
|
||||||
|
func executeWorkflowActions(ctx context.Context, workflow *project_model.Workflow, issue *issues_model.Issue) {
|
||||||
|
var toAddedLables, toRemovedLables []*issues_model.Label
|
||||||
|
|
||||||
|
for _, action := range workflow.WorkflowActions {
|
||||||
|
switch action.Type {
|
||||||
|
case project_model.WorkflowActionTypeColumn:
|
||||||
|
columnID, _ := strconv.ParseInt(action.Value, 10, 64)
|
||||||
|
if columnID == 0 {
|
||||||
|
log.Error("Invalid column ID: %s", action.Value)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
column, err := project_model.GetColumnByProjectIDAndColumnID(ctx, issue.Project.ID, columnID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("GetColumnByProjectIDAndColumnID: %v", err)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if err := MoveIssueToAnotherColumn(ctx, user_model.NewProjectWorkflowsUser(), issue, column); err != nil {
|
||||||
|
log.Error("MoveIssueToAnotherColumn: %v", err)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
case project_model.WorkflowActionTypeAddLabels:
|
||||||
|
labelID, _ := strconv.ParseInt(action.Value, 10, 64)
|
||||||
|
if labelID == 0 {
|
||||||
|
log.Error("Invalid label ID: %s", action.Value)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
label, err := issues_model.GetLabelByID(ctx, labelID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("GetLabelByID: %v", err)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
toAddedLables = append(toAddedLables, label)
|
||||||
|
case project_model.WorkflowActionTypeRemoveLabels:
|
||||||
|
labelID, _ := strconv.ParseInt(action.Value, 10, 64)
|
||||||
|
if labelID == 0 {
|
||||||
|
log.Error("Invalid label ID: %s", action.Value)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
label, err := issues_model.GetLabelByID(ctx, labelID)
|
||||||
|
if err != nil {
|
||||||
|
log.Error("GetLabelByID: %v", err)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
toRemovedLables = append(toRemovedLables, label)
|
||||||
|
case project_model.WorkflowActionTypeIssueState:
|
||||||
|
if strings.EqualFold(action.Value, "reopen") {
|
||||||
|
if issue.IsClosed {
|
||||||
|
if err := issue_service.ReopenIssue(ctx, issue, user_model.NewProjectWorkflowsUser(), ""); err != nil {
|
||||||
|
log.Error("ReopenIssue: %v", err)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} else if strings.EqualFold(action.Value, "close") {
|
||||||
|
if !issue.IsClosed {
|
||||||
|
if err := issue_service.CloseIssue(ctx, issue, user_model.NewProjectWorkflowsUser(), ""); err != nil {
|
||||||
|
log.Error("CloseIssue: %v", err)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
default:
|
||||||
|
log.Error("Unsupported action type: %s", action.Type)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if len(toAddedLables)+len(toRemovedLables) > 0 {
|
||||||
|
if err := issue_service.AddRemoveLabels(ctx, issue, user_model.NewProjectWorkflowsUser(), toAddedLables, toRemovedLables); err != nil {
|
||||||
|
log.Error("AddRemoveLabels: %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@ -0,0 +1,13 @@
|
|||||||
|
{{template "base/head" .}}
|
||||||
|
<div role="main" aria-label="{{.Title}}" class="page-content organization repository projects view-project">
|
||||||
|
{{if .ContextUser.IsOrganization}}
|
||||||
|
{{template "org/header" .}}
|
||||||
|
{{else}}
|
||||||
|
{{template "shared/user/org_profile_avatar" .}}
|
||||||
|
<div class="ui container tw-mb-4">
|
||||||
|
{{template "user/overview/header" .}}
|
||||||
|
</div>
|
||||||
|
{{end}}
|
||||||
|
{{template "projects/workflows" .}}
|
||||||
|
</div>
|
||||||
|
{{template "base/footer" .}}
|
||||||
@ -0,0 +1,45 @@
|
|||||||
|
<div class="ui container padded projects-view">
|
||||||
|
<div id="project-workflows"
|
||||||
|
data-project-link="{{.ProjectLink}}"
|
||||||
|
data-event-id="{{.workflowIDStr}}"
|
||||||
|
data-locale-default-workflows="{{ctx.Locale.Tr "projects.workflows.default_workflows"}}"
|
||||||
|
data-locale-view-workflow-configuration="{{ctx.Locale.Tr "projects.workflows.view_workflow_configuration"}}"
|
||||||
|
data-locale-configure-workflow="{{ctx.Locale.Tr "projects.workflows.configure_workflow"}}"
|
||||||
|
data-locale-when="{{ctx.Locale.Tr "projects.workflows.when"}}"
|
||||||
|
data-locale-run-when="{{ctx.Locale.Tr "projects.workflows.run_when"}}"
|
||||||
|
data-locale-filters="{{ctx.Locale.Tr "projects.workflows.filters"}}"
|
||||||
|
data-locale-apply-to="{{ctx.Locale.Tr "projects.workflows.apply_to"}}"
|
||||||
|
data-locale-when-moved-from-column="{{ctx.Locale.Tr "projects.workflows.when_moved_from_column"}}"
|
||||||
|
data-locale-when-moved-to-column="{{ctx.Locale.Tr "projects.workflows.when_moved_to_column"}}"
|
||||||
|
data-locale-only-if-has-labels="{{ctx.Locale.Tr "projects.workflows.only_if_has_labels"}}"
|
||||||
|
data-locale-actions="{{ctx.Locale.Tr "projects.workflows.actions"}}"
|
||||||
|
data-locale-move-to-column="{{ctx.Locale.Tr "projects.workflows.move_to_column"}}"
|
||||||
|
data-locale-add-labels="{{ctx.Locale.Tr "projects.workflows.add_labels"}}"
|
||||||
|
data-locale-remove-labels="{{ctx.Locale.Tr "projects.workflows.remove_labels"}}"
|
||||||
|
data-locale-any-label="{{ctx.Locale.Tr "projects.workflows.any_label"}}"
|
||||||
|
data-locale-any-column="{{ctx.Locale.Tr "projects.workflows.any_column"}}"
|
||||||
|
data-locale-issue-state="{{ctx.Locale.Tr "projects.workflows.issue_state"}}"
|
||||||
|
data-locale-none="{{ctx.Locale.Tr "projects.workflows.none"}}"
|
||||||
|
data-locale-no-change="{{ctx.Locale.Tr "projects.workflows.no_change"}}"
|
||||||
|
data-locale-edit="{{ctx.Locale.Tr "projects.workflows.edit"}}"
|
||||||
|
data-locale-delete="{{ctx.Locale.Tr "projects.workflows.delete"}}"
|
||||||
|
data-locale-save="{{ctx.Locale.Tr "projects.workflows.save"}}"
|
||||||
|
data-locale-clone="{{ctx.Locale.Tr "projects.workflows.clone"}}"
|
||||||
|
data-locale-cancel="{{ctx.Locale.Tr "projects.workflows.cancel"}}"
|
||||||
|
data-locale-disable="{{ctx.Locale.Tr "projects.workflows.disable"}}"
|
||||||
|
data-locale-disabled="{{ctx.Locale.Tr "projects.workflows.disabled"}}"
|
||||||
|
data-locale-enable="{{ctx.Locale.Tr "projects.workflows.enable"}}"
|
||||||
|
data-locale-enabled="{{ctx.Locale.Tr "projects.workflows.enabled"}}"
|
||||||
|
data-locale-issues-and-pull-requests="{{ctx.Locale.Tr "projects.workflows.issues_and_pull_requests"}}"
|
||||||
|
data-locale-issues-only="{{ctx.Locale.Tr "projects.workflows.issues_only"}}"
|
||||||
|
data-locale-pull-requests-only="{{ctx.Locale.Tr "projects.workflows.pull_requests_only"}}"
|
||||||
|
data-locale-select-column="{{ctx.Locale.Tr "projects.workflows.select_column"}}"
|
||||||
|
data-locale-close-issue="{{ctx.Locale.Tr "projects.workflows.close_issue"}}"
|
||||||
|
data-locale-reopen-issue="{{ctx.Locale.Tr "projects.workflows.reopen_issue"}}"
|
||||||
|
data-locale-save-workflow-failed="{{ctx.Locale.Tr "projects.workflows.save_workflow_failed"}}"
|
||||||
|
data-locale-update-workflow-failed="{{ctx.Locale.Tr "projects.workflows.update_workflow_failed"}}"
|
||||||
|
data-locale-delete-workflow-failed="{{ctx.Locale.Tr "projects.workflows.delete_workflow_failed"}}"
|
||||||
|
data-locale-at-least-one-action-required="{{ctx.Locale.Tr "projects.workflows.at_least_one_action_required"}}"
|
||||||
|
>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
@ -0,0 +1,12 @@
|
|||||||
|
{{template "base/head" .}}
|
||||||
|
<div role="main" aria-label="{{.Title}}" class="page-content repository projects view-project">
|
||||||
|
{{template "repo/header" .}}
|
||||||
|
<div class="ui container padded">
|
||||||
|
<div class="tw-flex tw-justify-between tw-items-center tw-mb-4">
|
||||||
|
<a class="ui" href="{{.ProjectLink}}">{{svg "octicon-arrow-left"}} {{ctx.Locale.Tr "projects.workflows"}} {{.Project.Title}}</a>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
{{template "projects/workflows" .}}
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{{template "base/footer" .}}
|
||||||
@ -0,0 +1,774 @@
|
|||||||
|
// Copyright 2025 The Gitea Authors. All rights reserved.
|
||||||
|
// SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
package integration
|
||||||
|
|
||||||
|
import (
|
||||||
|
"fmt"
|
||||||
|
"net/http"
|
||||||
|
"net/url"
|
||||||
|
"path"
|
||||||
|
"strconv"
|
||||||
|
"strings"
|
||||||
|
"testing"
|
||||||
|
|
||||||
|
"code.gitea.io/gitea/models/db"
|
||||||
|
issues_model "code.gitea.io/gitea/models/issues"
|
||||||
|
project_model "code.gitea.io/gitea/models/project"
|
||||||
|
repo_model "code.gitea.io/gitea/models/repo"
|
||||||
|
"code.gitea.io/gitea/models/unittest"
|
||||||
|
user_model "code.gitea.io/gitea/models/user"
|
||||||
|
"code.gitea.io/gitea/modules/gitrepo"
|
||||||
|
"code.gitea.io/gitea/modules/json"
|
||||||
|
"code.gitea.io/gitea/modules/test"
|
||||||
|
"code.gitea.io/gitea/tests"
|
||||||
|
|
||||||
|
"github.com/stretchr/testify/assert"
|
||||||
|
)
|
||||||
|
|
||||||
|
func testCreateProjectWorkflow(t *testing.T, session *TestSession, userName, repoName string, projectID int64, event string, workflowData map[string]any) {
|
||||||
|
req := NewRequestWithJSON(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/%s?_csrf=%s",
|
||||||
|
userName, repoName, projectID, event, GetUserCSRFToken(t, session)),
|
||||||
|
workflowData)
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
var result map[string]any
|
||||||
|
err := json.Unmarshal(resp.Body.Bytes(), &result)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, result["success"].(bool))
|
||||||
|
}
|
||||||
|
|
||||||
|
func testNewIssueReturnIssue(t *testing.T, session *TestSession, repo *repo_model.Repository, opts newIssueOptions) int64 {
|
||||||
|
testNewIssue(t, session, repo.OwnerName, repo.Name, opts)
|
||||||
|
|
||||||
|
// Get the created issue from database to verify
|
||||||
|
issues, err := issues_model.Issues(t.Context(), &issues_model.IssuesOptions{
|
||||||
|
RepoIDs: []int64{repo.ID},
|
||||||
|
SortType: "newest",
|
||||||
|
Paginator: &db.ListOptions{
|
||||||
|
PageSize: 1,
|
||||||
|
},
|
||||||
|
})
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.NotEmpty(t, issues)
|
||||||
|
return issues[0].ID
|
||||||
|
}
|
||||||
|
|
||||||
|
// testAddIssueToProject adds the issue to the project via web form if projectID == 0, it removes the issue from the project
|
||||||
|
func testAddIssueToProject(t *testing.T, session *TestSession, userName, repoName string, projectID, issueID int64) {
|
||||||
|
addToProjectReq := NewRequestWithValues(t, "POST", fmt.Sprintf("/%s/%s/issues/projects?_csrf=%s",
|
||||||
|
userName, repoName, GetUserCSRFToken(t, session)),
|
||||||
|
map[string]string{
|
||||||
|
"_csrf": GetUserCSRFToken(t, session),
|
||||||
|
"id": strconv.FormatInt(projectID, 10),
|
||||||
|
"issue_ids": strconv.FormatInt(issueID, 10),
|
||||||
|
})
|
||||||
|
session.MakeRequest(t, addToProjectReq, http.StatusOK)
|
||||||
|
}
|
||||||
|
|
||||||
|
// TestProjectWorkflowExecutionItemOpened tests workflow execution when an issue is added to project
|
||||||
|
func TestProjectWorkflowExecutionItemOpened(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create project and columns
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Workflow Execution",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
assert.NoError(t, project_model.NewProject(t.Context(), project))
|
||||||
|
|
||||||
|
columnToDo := &project_model.Column{
|
||||||
|
Title: "To Do",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
assert.NoError(t, project_model.NewColumn(t.Context(), columnToDo))
|
||||||
|
|
||||||
|
// Create label
|
||||||
|
label := &issues_model.Label{
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Name: "bug",
|
||||||
|
Color: "ee0701",
|
||||||
|
}
|
||||||
|
assert.NoError(t, issues_model.NewLabel(t.Context(), label))
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Create workflow via HTTP: when item is opened, move to "To Do" and add "bug" label
|
||||||
|
testCreateProjectWorkflow(t, session, user.Name, repo.Name, project.ID, "item_opened", map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventItemOpened),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeIssueType): "issue",
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeColumn): strconv.FormatInt(columnToDo.ID, 10),
|
||||||
|
string(project_model.WorkflowActionTypeAddLabels): []string{strconv.FormatInt(label.ID, 10)},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
issueID := testNewIssueReturnIssue(t, session, repo, newIssueOptions{
|
||||||
|
Title: "Test Issue for Workflow",
|
||||||
|
Content: "This should trigger item_opened workflow",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
})
|
||||||
|
|
||||||
|
// Verify workflow executed: issue moved to "To Do" and has "bug" label
|
||||||
|
issue, err := issues_model.GetIssueByID(t.Context(), issueID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
projectIssue := &project_model.ProjectIssue{}
|
||||||
|
has, err := db.GetEngine(t.Context()).Where("issue_id=?", issue.ID).Get(projectIssue)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, has)
|
||||||
|
assert.Equal(t, columnToDo.ID, projectIssue.ProjectColumnID)
|
||||||
|
|
||||||
|
err = issue.LoadLabels(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Len(t, issue.Labels, 1)
|
||||||
|
assert.Equal(t, label.ID, issue.Labels[0].ID)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowExecutionItemAddedToProject(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create project and columns
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Workflow Execution",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
assert.NoError(t, project_model.NewProject(t.Context(), project))
|
||||||
|
|
||||||
|
columnToDo := &project_model.Column{
|
||||||
|
Title: "To Do",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
assert.NoError(t, project_model.NewColumn(t.Context(), columnToDo))
|
||||||
|
|
||||||
|
// Create label
|
||||||
|
label := &issues_model.Label{
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Name: "bug",
|
||||||
|
Color: "ee0701",
|
||||||
|
}
|
||||||
|
assert.NoError(t, issues_model.NewLabel(t.Context(), label))
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Create workflow via HTTP: when item added to project, move to "To Do" and add "bug" label
|
||||||
|
testCreateProjectWorkflow(t, session, user.Name, repo.Name, project.ID, "item_added_to_project", map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventItemAddedToProject),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeIssueType): "issue",
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeColumn): strconv.FormatInt(columnToDo.ID, 10),
|
||||||
|
string(project_model.WorkflowActionTypeAddLabels): []string{strconv.FormatInt(label.ID, 10)},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
issueID := testNewIssueReturnIssue(t, session, repo, newIssueOptions{
|
||||||
|
Title: "Test Issue for Workflow",
|
||||||
|
Content: "This should trigger workflow when added to project",
|
||||||
|
})
|
||||||
|
|
||||||
|
// Add issue to project via Web form - this triggers the workflow
|
||||||
|
testAddIssueToProject(t, session, user.Name, repo.Name, project.ID, issueID)
|
||||||
|
|
||||||
|
// Verify workflow executed: issue moved to "To Do" and has "bug" label
|
||||||
|
issue, err := issues_model.GetIssueByID(t.Context(), issueID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
projectIssue := &project_model.ProjectIssue{}
|
||||||
|
has, err := db.GetEngine(t.Context()).Where("issue_id=?", issue.ID).Get(projectIssue)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, has)
|
||||||
|
assert.Equal(t, columnToDo.ID, projectIssue.ProjectColumnID)
|
||||||
|
|
||||||
|
err = issue.LoadLabels(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Len(t, issue.Labels, 1)
|
||||||
|
assert.Equal(t, label.ID, issue.Labels[0].ID)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowExecutionItemRemovedFromProject(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create project and columns
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Workflow Execution",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
assert.NoError(t, project_model.NewProject(t.Context(), project))
|
||||||
|
|
||||||
|
columnToDo := &project_model.Column{
|
||||||
|
Title: "To Do",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
assert.NoError(t, project_model.NewColumn(t.Context(), columnToDo))
|
||||||
|
|
||||||
|
// Create label
|
||||||
|
label := &issues_model.Label{
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Name: "no-project",
|
||||||
|
Color: "ee0701",
|
||||||
|
}
|
||||||
|
assert.NoError(t, issues_model.NewLabel(t.Context(), label))
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Create workflow via HTTP: when item added to project, move to "To Do" and add "bug" label
|
||||||
|
testCreateProjectWorkflow(t, session, user.Name, repo.Name, project.ID, "item_removed_from_project", map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventItemRemovedFromProject),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeIssueType): "issue",
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeAddLabels): []string{strconv.FormatInt(label.ID, 10)},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
issueID := testNewIssueReturnIssue(t, session, repo, newIssueOptions{
|
||||||
|
Title: "Test Issue for Workflow",
|
||||||
|
Content: "This should trigger workflow when removed from project",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
})
|
||||||
|
|
||||||
|
// remove issue from the project to trigger the workflow
|
||||||
|
testAddIssueToProject(t, session, user.Name, repo.Name, 0, issueID)
|
||||||
|
|
||||||
|
issue, err := issues_model.GetIssueByID(t.Context(), issueID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.NoError(t, issue.LoadProject(t.Context()))
|
||||||
|
assert.Nil(t, issue.Project)
|
||||||
|
|
||||||
|
err = issue.LoadLabels(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Len(t, issue.Labels, 1)
|
||||||
|
assert.Equal(t, label.ID, issue.Labels[0].ID)
|
||||||
|
}
|
||||||
|
|
||||||
|
// TestProjectWorkflowExecutionItemClosed tests workflow when issue is closed
|
||||||
|
func TestProjectWorkflowExecutionItemClosed(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Close Workflow",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
columnDone := &project_model.Column{
|
||||||
|
Title: "Done",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
err = project_model.NewColumn(t.Context(), columnDone)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
labelCompleted := &issues_model.Label{
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Name: "completed",
|
||||||
|
Color: "00ff00",
|
||||||
|
}
|
||||||
|
err = issues_model.NewLabel(t.Context(), labelCompleted)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Create workflow: when closed, move to "Done" and add "completed" label
|
||||||
|
testCreateProjectWorkflow(t, session, user.Name, repo.Name, project.ID, "item_closed", map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventItemClosed),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeIssueType): "issue",
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeColumn): strconv.FormatInt(columnDone.ID, 10),
|
||||||
|
string(project_model.WorkflowActionTypeAddLabels): []string{strconv.FormatInt(labelCompleted.ID, 10)},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
issueID := testNewIssueReturnIssue(t, session, repo, newIssueOptions{
|
||||||
|
Title: "Test Issue for Workflow",
|
||||||
|
Content: "This should trigger workflow when item is closed",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
})
|
||||||
|
|
||||||
|
issue, err := issues_model.GetIssueByID(t.Context(), issueID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.False(t, issue.IsClosed)
|
||||||
|
assert.NoError(t, issue.LoadRepo(t.Context()))
|
||||||
|
|
||||||
|
// Close issue via API
|
||||||
|
testIssueAddComment(t, session, issue.Link(), "Test comment 3", "close")
|
||||||
|
|
||||||
|
// Verify workflow executed
|
||||||
|
issue, err = issues_model.GetIssueByID(t.Context(), issueID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, issue.IsClosed)
|
||||||
|
|
||||||
|
projectIssue := &project_model.ProjectIssue{}
|
||||||
|
has, err := db.GetEngine(t.Context()).Where("issue_id=?", issue.ID).Get(projectIssue)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, has)
|
||||||
|
assert.Equal(t, columnDone.ID, projectIssue.ProjectColumnID)
|
||||||
|
|
||||||
|
err = issue.LoadLabels(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
hasLabel := false
|
||||||
|
for _, l := range issue.Labels {
|
||||||
|
if l.ID == labelCompleted.ID {
|
||||||
|
hasLabel = true
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
assert.True(t, hasLabel)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowExecutionItemReopened(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Close Workflow",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
columnDone := &project_model.Column{
|
||||||
|
Title: "Done",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
err = project_model.NewColumn(t.Context(), columnDone)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
labelCompleted := &issues_model.Label{
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Name: "completed",
|
||||||
|
Color: "00ff00",
|
||||||
|
}
|
||||||
|
err = issues_model.NewLabel(t.Context(), labelCompleted)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
testCreateProjectWorkflow(t, session, user.Name, repo.Name, project.ID, "item_reopened",
|
||||||
|
map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventItemReopened),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeIssueType): "issue",
|
||||||
|
string(project_model.WorkflowFilterTypeLabels): strconv.FormatInt(labelCompleted.ID, 10),
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeColumn): strconv.FormatInt(columnDone.ID, 10),
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
issueID := testNewIssueReturnIssue(t, session, repo, newIssueOptions{
|
||||||
|
Title: "Test Issue for Workflow",
|
||||||
|
Content: "This should trigger workflow when item is reopened",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
LabelIDs: []int64{labelCompleted.ID},
|
||||||
|
})
|
||||||
|
|
||||||
|
issue, err := issues_model.GetIssueByID(t.Context(), issueID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.False(t, issue.IsClosed)
|
||||||
|
assert.NoError(t, issue.LoadRepo(t.Context()))
|
||||||
|
|
||||||
|
// Reopen issue
|
||||||
|
testIssueAddComment(t, session, issue.Link(), "Test comment 3", "close")
|
||||||
|
testIssueAddComment(t, session, issue.Link(), "Test comment 3", "reopen")
|
||||||
|
|
||||||
|
// Reload and Verify workflow executed
|
||||||
|
issue, err = issues_model.GetIssueByID(t.Context(), issueID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.False(t, issue.IsClosed)
|
||||||
|
|
||||||
|
projectIssue := &project_model.ProjectIssue{}
|
||||||
|
has, err := db.GetEngine(t.Context()).Where("issue_id=?", issue.ID).Get(projectIssue)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, has)
|
||||||
|
assert.Equal(t, columnDone.ID, projectIssue.ProjectColumnID)
|
||||||
|
|
||||||
|
err = issue.LoadLabels(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
hasLabel := false
|
||||||
|
for _, l := range issue.Labels {
|
||||||
|
if l.ID == labelCompleted.ID {
|
||||||
|
hasLabel = true
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
assert.True(t, hasLabel)
|
||||||
|
}
|
||||||
|
|
||||||
|
// TestProjectWorkflowExecutionColumnChanged tests workflow when moving between columns
|
||||||
|
func TestProjectWorkflowExecutionColumnChanged(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Column Change",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
columnToDo := &project_model.Column{Title: "To Do", ProjectID: project.ID}
|
||||||
|
err = project_model.NewColumn(t.Context(), columnToDo)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
columnDone := &project_model.Column{Title: "Done", ProjectID: project.ID}
|
||||||
|
err = project_model.NewColumn(t.Context(), columnDone)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
labelWIP := &issues_model.Label{RepoID: repo.ID, Name: "wip", Color: "fbca04"}
|
||||||
|
err = issues_model.NewLabel(t.Context(), labelWIP)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Create workflow: when moved to "Done", remove "wip" and close
|
||||||
|
testCreateProjectWorkflow(t, session, user.Name, repo.Name, project.ID, "item_column_changed", map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventItemColumnChanged),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeTargetColumn): strconv.FormatInt(columnDone.ID, 10),
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeRemoveLabels): []string{strconv.FormatInt(labelWIP.ID, 10)},
|
||||||
|
string(project_model.WorkflowActionTypeIssueState): "close",
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
// Create issue with "wip" label
|
||||||
|
issueID := testNewIssueReturnIssue(t, session, repo, newIssueOptions{
|
||||||
|
Title: "Test Column Change",
|
||||||
|
Content: "Will move columns",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
LabelIDs: []int64{labelWIP.ID},
|
||||||
|
})
|
||||||
|
|
||||||
|
// Move to "To Do" first
|
||||||
|
moveReq := NewRequestWithJSON(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/%d/move?_csrf=%s", user.Name, repo.Name, project.ID, columnToDo.ID, GetUserCSRFToken(t, session)),
|
||||||
|
map[string]any{
|
||||||
|
"issues": []map[string]any{
|
||||||
|
{
|
||||||
|
"issueID": issueID,
|
||||||
|
"sorting": 0,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
session.MakeRequest(t, moveReq, http.StatusOK)
|
||||||
|
|
||||||
|
// Move to "Done" - triggers workflow
|
||||||
|
moveReq = NewRequestWithJSON(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/%d/move?_csrf=%s", user.Name, repo.Name, project.ID, columnDone.ID, GetUserCSRFToken(t, session)),
|
||||||
|
map[string]any{
|
||||||
|
"issues": []map[string]any{
|
||||||
|
{
|
||||||
|
"issueID": issueID,
|
||||||
|
"sorting": 0,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
session.MakeRequest(t, moveReq, http.StatusOK)
|
||||||
|
|
||||||
|
// Verify workflow executed
|
||||||
|
issue, err := issues_model.GetIssueByID(t.Context(), issueID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, issue.IsClosed, "Issue should be closed")
|
||||||
|
|
||||||
|
err = issue.LoadLabels(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
hasWIP := false
|
||||||
|
for _, l := range issue.Labels {
|
||||||
|
if l.ID == labelWIP.ID {
|
||||||
|
hasWIP = true
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
assert.False(t, hasWIP, "WIP label should be removed")
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowExecutionCodeChangesRequested(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
// Use existing PR #3 from fixtures (issue_id: 3, pull_request id: 2)
|
||||||
|
pr := unittest.AssertExistsAndLoadBean(t, &issues_model.PullRequest{ID: 2})
|
||||||
|
assert.NoError(t, pr.LoadIssue(t.Context()))
|
||||||
|
assert.NoError(t, pr.LoadBaseRepo(t.Context()))
|
||||||
|
|
||||||
|
repo := pr.BaseRepo
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Code Changes Requested",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
columnInProgress := &project_model.Column{Title: "In Progress", ProjectID: project.ID}
|
||||||
|
err = project_model.NewColumn(t.Context(), columnInProgress)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
labelNeedChange := &issues_model.Label{RepoID: repo.ID, Name: "needs-changes", Color: "fbca04"}
|
||||||
|
err = issues_model.NewLabel(t.Context(), labelNeedChange)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Create workflow: when code changes requested, add "needs-changes" label
|
||||||
|
testCreateProjectWorkflow(t, session, user.Name, repo.Name, project.ID, "code_changes_requested", map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventCodeChangesRequested),
|
||||||
|
"filters": map[string]any{},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeAddLabels): []string{strconv.FormatInt(labelNeedChange.ID, 10)},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
// Add PR to project
|
||||||
|
testAddIssueToProject(t, session, user.Name, repo.Name, project.ID, pr.Issue.ID)
|
||||||
|
|
||||||
|
// User 2 submits a "REQUEST_CHANGES" review
|
||||||
|
user2Session := loginUser(t, "user2")
|
||||||
|
prURL := fmt.Sprintf("/%s/%s/pulls/%d", user.Name, repo.Name, pr.Issue.Index)
|
||||||
|
req := NewRequest(t, "GET", prURL+"/files")
|
||||||
|
resp := user2Session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
htmlDoc := NewHTMLParser(t, resp.Body)
|
||||||
|
gitRepo, err := gitrepo.OpenRepository(t.Context(), pr.BaseRepo)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
defer gitRepo.Close()
|
||||||
|
|
||||||
|
commitID, err := gitRepo.GetRefCommitID(pr.GetGitHeadRefName())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
testSubmitReview(t, user2Session, htmlDoc.GetCSRF(), user.Name, repo.Name, strconv.FormatInt(pr.Issue.Index, 10), commitID, "reject", http.StatusOK)
|
||||||
|
|
||||||
|
// Verify workflow executed: PR should have "needs-changes" label
|
||||||
|
issue, err := issues_model.GetIssueByID(t.Context(), pr.Issue.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
err = issue.LoadLabels(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
hasNeedChangeLabel := false
|
||||||
|
for _, l := range issue.Labels {
|
||||||
|
if l.ID == labelNeedChange.ID {
|
||||||
|
hasNeedChangeLabel = true
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
assert.True(t, hasNeedChangeLabel, "needs-changes label should be added")
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowExecutionCodeReviewApproved(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
// Use existing PR #3 from fixtures (issue_id: 3, pull_request id: 2)
|
||||||
|
pr := unittest.AssertExistsAndLoadBean(t, &issues_model.PullRequest{ID: 2})
|
||||||
|
assert.NoError(t, pr.LoadIssue(t.Context()))
|
||||||
|
assert.NoError(t, pr.LoadBaseRepo(t.Context()))
|
||||||
|
|
||||||
|
repo := pr.BaseRepo
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Code Review Approved",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
columnReadyToMerge := &project_model.Column{Title: "Ready to Merge", ProjectID: project.ID}
|
||||||
|
err = project_model.NewColumn(t.Context(), columnReadyToMerge)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
labelApproved := &issues_model.Label{RepoID: repo.ID, Name: "approved", Color: "00ff00"}
|
||||||
|
err = issues_model.NewLabel(t.Context(), labelApproved)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Create workflow: when code review approved, move to "Ready to Merge" and add "approved" label
|
||||||
|
testCreateProjectWorkflow(t, session, user.Name, repo.Name, project.ID, "code_review_approved", map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventCodeReviewApproved),
|
||||||
|
"filters": map[string]any{},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeColumn): strconv.FormatInt(columnReadyToMerge.ID, 10),
|
||||||
|
string(project_model.WorkflowActionTypeAddLabels): []string{strconv.FormatInt(labelApproved.ID, 10)},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
// Add PR to project
|
||||||
|
testAddIssueToProject(t, session, user.Name, repo.Name, project.ID, pr.Issue.ID)
|
||||||
|
|
||||||
|
// User 2 submits an "APPROVE" review
|
||||||
|
user2Session := loginUser(t, "user2")
|
||||||
|
prURL := fmt.Sprintf("/%s/%s/pulls/%d", user.Name, repo.Name, pr.Issue.Index)
|
||||||
|
req := NewRequest(t, "GET", prURL+"/files")
|
||||||
|
resp := user2Session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
htmlDoc := NewHTMLParser(t, resp.Body)
|
||||||
|
gitRepo, err := gitrepo.OpenRepository(t.Context(), pr.BaseRepo)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
defer gitRepo.Close()
|
||||||
|
|
||||||
|
commitID, err := gitRepo.GetRefCommitID(pr.GetGitHeadRefName())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
testSubmitReview(t, user2Session, htmlDoc.GetCSRF(), user.Name, repo.Name, strconv.FormatInt(pr.Issue.Index, 10), commitID, "approve", http.StatusOK)
|
||||||
|
|
||||||
|
// Verify workflow executed: PR should be in "Ready to Merge" column and have "approved" label
|
||||||
|
issue, err := issues_model.GetIssueByID(t.Context(), pr.Issue.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
err = issue.LoadLabels(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
hasApprovedLabel := false
|
||||||
|
for _, l := range issue.Labels {
|
||||||
|
if l.ID == labelApproved.ID {
|
||||||
|
hasApprovedLabel = true
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
assert.True(t, hasApprovedLabel, "approved label should be added")
|
||||||
|
|
||||||
|
// Check column
|
||||||
|
projectIssue := &project_model.ProjectIssue{}
|
||||||
|
has, err := db.GetEngine(t.Context()).Where("issue_id=?", issue.ID).Get(projectIssue)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, has)
|
||||||
|
assert.Equal(t, columnReadyToMerge.ID, projectIssue.ProjectColumnID)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowExecutionPullRequestMerged(t *testing.T) {
|
||||||
|
onGiteaRun(t, func(t *testing.T, u *url.URL) {
|
||||||
|
// Fork repo1 and create a PR that can be merged
|
||||||
|
session := loginUser(t, "user1")
|
||||||
|
testRepoFork(t, session, "user2", "repo1", "user1", "repo1", "")
|
||||||
|
testEditFile(t, session, "user1", "repo1", "master", "README.md", "Hello, World (Edited for merge test)\n")
|
||||||
|
|
||||||
|
// Get the base repo
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{OwnerName: "user2", Name: "repo1"})
|
||||||
|
|
||||||
|
// Create project in base repo
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test PR Merged",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
columnDone := &project_model.Column{Title: "Done", ProjectID: project.ID}
|
||||||
|
err = project_model.NewColumn(t.Context(), columnDone)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
labelMerged := &issues_model.Label{RepoID: repo.ID, Name: "merged", Color: "6f42c1"}
|
||||||
|
err = issues_model.NewLabel(t.Context(), labelMerged)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Login as user2 (repo owner) to create workflow
|
||||||
|
user2Session := loginUser(t, "user2")
|
||||||
|
|
||||||
|
// Create workflow: when PR merged, move to "Done" and add "merged" label
|
||||||
|
testCreateProjectWorkflow(t, user2Session, "user2", "repo1", project.ID, "pull_request_merged", map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventPullRequestMerged),
|
||||||
|
"filters": map[string]any{},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeColumn): strconv.FormatInt(columnDone.ID, 10),
|
||||||
|
string(project_model.WorkflowActionTypeAddLabels): []string{strconv.FormatInt(labelMerged.ID, 10)},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
// Create PR from user1's fork to user2's repo
|
||||||
|
resp := testPullCreate(t, session, "user1", "repo1", false, "master", "master", "Test PR for Merge Workflow")
|
||||||
|
|
||||||
|
// Get PR details from redirect URL
|
||||||
|
elem := strings.Split(test.RedirectURL(resp), "/")
|
||||||
|
assert.Equal(t, "pulls", elem[3])
|
||||||
|
prNum := elem[4]
|
||||||
|
|
||||||
|
// Load the PR
|
||||||
|
prNumInt, err := strconv.ParseInt(prNum, 10, 64)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
pr := unittest.AssertExistsAndLoadBean(t, &issues_model.PullRequest{BaseRepoID: repo.ID, Index: prNumInt})
|
||||||
|
assert.NoError(t, pr.LoadIssue(t.Context()))
|
||||||
|
|
||||||
|
// Add PR to project (as user2, the repo owner)
|
||||||
|
testAddIssueToProject(t, user2Session, "user2", "repo1", project.ID, pr.Issue.ID)
|
||||||
|
|
||||||
|
// Merge the PR (as user2, who has permission)
|
||||||
|
prURL := "/user2/repo1/pulls/" + prNum
|
||||||
|
req := NewRequest(t, "GET", prURL)
|
||||||
|
resp = user2Session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
htmlDoc := NewHTMLParser(t, resp.Body)
|
||||||
|
|
||||||
|
req = NewRequestWithValues(t, "POST", path.Join(prURL, "merge"), map[string]string{
|
||||||
|
"_csrf": htmlDoc.GetCSRF(),
|
||||||
|
"do": string(repo_model.MergeStyleMerge),
|
||||||
|
})
|
||||||
|
user2Session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
// Verify workflow executed: PR should be in "Done" column and have "merged" label
|
||||||
|
issue, err := issues_model.GetIssueByID(t.Context(), pr.Issue.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
err = issue.LoadLabels(t.Context())
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
hasMergedLabel := false
|
||||||
|
for _, l := range issue.Labels {
|
||||||
|
if l.ID == labelMerged.ID {
|
||||||
|
hasMergedLabel = true
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
assert.True(t, hasMergedLabel, "merged label should be added")
|
||||||
|
|
||||||
|
// Check column
|
||||||
|
projectIssue := &project_model.ProjectIssue{}
|
||||||
|
has, err := db.GetEngine(t.Context()).Where("issue_id=?", issue.ID).Get(projectIssue)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, has)
|
||||||
|
assert.Equal(t, columnDone.ID, projectIssue.ProjectColumnID)
|
||||||
|
|
||||||
|
// Verify PR is merged
|
||||||
|
pr, err = issues_model.GetPullRequestByID(t.Context(), pr.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, pr.HasMerged, "PR should be merged")
|
||||||
|
})
|
||||||
|
}
|
||||||
@ -0,0 +1,557 @@
|
|||||||
|
// Copyright 2025 The Gitea Authors. All rights reserved.
|
||||||
|
// SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
package integration
|
||||||
|
|
||||||
|
import (
|
||||||
|
"fmt"
|
||||||
|
"net/http"
|
||||||
|
"strconv"
|
||||||
|
"strings"
|
||||||
|
"testing"
|
||||||
|
|
||||||
|
"code.gitea.io/gitea/models/db"
|
||||||
|
project_model "code.gitea.io/gitea/models/project"
|
||||||
|
repo_model "code.gitea.io/gitea/models/repo"
|
||||||
|
"code.gitea.io/gitea/models/unittest"
|
||||||
|
user_model "code.gitea.io/gitea/models/user"
|
||||||
|
"code.gitea.io/gitea/modules/json"
|
||||||
|
"code.gitea.io/gitea/tests"
|
||||||
|
|
||||||
|
"github.com/stretchr/testify/assert"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestProjectWorkflowsPage(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create a project
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Project for Workflows",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create columns for the project
|
||||||
|
column1 := &project_model.Column{
|
||||||
|
Title: "To Do",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
err = project_model.NewColumn(t.Context(), column1)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
column2 := &project_model.Column{
|
||||||
|
Title: "Done",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
err = project_model.NewColumn(t.Context(), column2)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create some workflows
|
||||||
|
workflow1 := &project_model.Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: project_model.WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []project_model.WorkflowFilter{
|
||||||
|
{
|
||||||
|
Type: project_model.WorkflowFilterTypeIssueType,
|
||||||
|
Value: "issue",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
WorkflowActions: []project_model.WorkflowAction{
|
||||||
|
{
|
||||||
|
Type: project_model.WorkflowActionTypeColumn,
|
||||||
|
Value: strconv.FormatInt(column1.ID, 10),
|
||||||
|
},
|
||||||
|
},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err = project_model.CreateWorkflow(t.Context(), workflow1)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
workflow2 := &project_model.Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: project_model.WorkflowEventItemClosed,
|
||||||
|
WorkflowFilters: []project_model.WorkflowFilter{
|
||||||
|
{
|
||||||
|
Type: project_model.WorkflowFilterTypeIssueType,
|
||||||
|
Value: "pull_request",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
WorkflowActions: []project_model.WorkflowAction{
|
||||||
|
{
|
||||||
|
Type: project_model.WorkflowActionTypeColumn,
|
||||||
|
Value: strconv.FormatInt(column2.ID, 10),
|
||||||
|
},
|
||||||
|
},
|
||||||
|
Enabled: false, // Disabled workflow
|
||||||
|
}
|
||||||
|
err = project_model.CreateWorkflow(t.Context(), workflow2)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Test accessing workflows page
|
||||||
|
req := NewRequest(t, "GET", fmt.Sprintf("/%s/%s/projects/%d/workflows", user.Name, repo.Name, project.ID))
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
htmlDoc := NewHTMLParser(t, resp.Body)
|
||||||
|
|
||||||
|
// Verify the main workflow container exists
|
||||||
|
assert.Positive(t, htmlDoc.Find("#project-workflows").Length(), "Main workflow container should exist")
|
||||||
|
|
||||||
|
// Verify data attributes are set correctly
|
||||||
|
workflowDiv := htmlDoc.Find("#project-workflows")
|
||||||
|
assert.Positive(t, workflowDiv.Length(), "Workflow div should exist")
|
||||||
|
|
||||||
|
// Check that locale data attributes exist
|
||||||
|
assert.NotEmpty(t, workflowDiv.AttrOr("data-locale-default-workflows", ""), "data-locale-default-workflows should be set")
|
||||||
|
assert.NotEmpty(t, workflowDiv.AttrOr("data-locale-when", ""), "data-locale-when should be set")
|
||||||
|
assert.NotEmpty(t, workflowDiv.AttrOr("data-locale-actions", ""), "data-locale-actions should be set")
|
||||||
|
assert.NotEmpty(t, workflowDiv.AttrOr("data-locale-filters", ""), "data-locale-filters should be set")
|
||||||
|
assert.NotEmpty(t, workflowDiv.AttrOr("data-locale-close-issue", ""), "data-locale-close-issue should be set")
|
||||||
|
assert.NotEmpty(t, workflowDiv.AttrOr("data-locale-reopen-issue", ""), "data-locale-reopen-issue should be set")
|
||||||
|
assert.NotEmpty(t, workflowDiv.AttrOr("data-locale-issues-and-pull-requests", ""), "data-locale-issues-and-pull-requests should be set")
|
||||||
|
|
||||||
|
// Verify project link is set
|
||||||
|
projectLink := workflowDiv.AttrOr("data-project-link", "")
|
||||||
|
assert.Equal(t, fmt.Sprintf("/%s/%s/projects/%d", user.Name, repo.Name, project.ID), projectLink, "Project link should be correct")
|
||||||
|
|
||||||
|
// Test that unauthenticated users cannot access
|
||||||
|
req = NewRequest(t, "GET", fmt.Sprintf("/%s/%s/projects/%d/workflows", user.Name, repo.Name, project.ID))
|
||||||
|
MakeRequest(t, req, http.StatusNotFound)
|
||||||
|
|
||||||
|
// Test accessing non-existent project workflows page
|
||||||
|
req = NewRequest(t, "GET", fmt.Sprintf("/%s/%s/projects/999999/workflows", user.Name, repo.Name))
|
||||||
|
session.MakeRequest(t, req, http.StatusNotFound)
|
||||||
|
|
||||||
|
// Verify workflows were created
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(t.Context(), project.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Len(t, workflows, 2, "Should have 2 workflows")
|
||||||
|
|
||||||
|
// Verify workflow details
|
||||||
|
assert.Equal(t, project_model.WorkflowEventItemOpened, workflows[0].WorkflowEvent)
|
||||||
|
assert.True(t, workflows[0].Enabled, "First workflow should be enabled")
|
||||||
|
assert.Equal(t, project_model.WorkflowEventItemClosed, workflows[1].WorkflowEvent)
|
||||||
|
assert.False(t, workflows[1].Enabled, "Second workflow should be disabled")
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowCreate(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create a project
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Project for Workflow Create",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create a column
|
||||||
|
column := &project_model.Column{
|
||||||
|
Title: "Test Column",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
err = project_model.NewColumn(t.Context(), column)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Create a workflow via API
|
||||||
|
workflowData := map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventItemOpened),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeIssueType): "issue",
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeColumn): strconv.FormatInt(column.ID, 10),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
body, err := json.Marshal(workflowData)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
req := NewRequestWithBody(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/item_opened?_csrf=%s", user.Name, repo.Name, project.ID, GetUserCSRFToken(t, session)),
|
||||||
|
strings.NewReader(string(body)))
|
||||||
|
req.Header.Set("Content-Type", "application/json")
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
// Parse response
|
||||||
|
var result map[string]any
|
||||||
|
err = json.Unmarshal(resp.Body.Bytes(), &result)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, result["success"].(bool))
|
||||||
|
|
||||||
|
// Verify workflow was created
|
||||||
|
workflows, err := project_model.FindWorkflowsByProjectID(t.Context(), project.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Len(t, workflows, 1)
|
||||||
|
assert.Equal(t, project_model.WorkflowEventItemOpened, workflows[0].WorkflowEvent)
|
||||||
|
assert.True(t, workflows[0].Enabled)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowUpdate(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create a project
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Project for Workflow Update",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create a column
|
||||||
|
column := &project_model.Column{
|
||||||
|
Title: "Test Column",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
err = project_model.NewColumn(t.Context(), column)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create a workflow
|
||||||
|
workflow := &project_model.Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: project_model.WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []project_model.WorkflowFilter{
|
||||||
|
{
|
||||||
|
Type: project_model.WorkflowFilterTypeIssueType,
|
||||||
|
Value: "issue",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
WorkflowActions: []project_model.WorkflowAction{
|
||||||
|
{
|
||||||
|
Type: project_model.WorkflowActionTypeColumn,
|
||||||
|
Value: strconv.FormatInt(column.ID, 10),
|
||||||
|
},
|
||||||
|
},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err = project_model.CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Update the workflow
|
||||||
|
updateData := map[string]any{
|
||||||
|
"event_id": strconv.FormatInt(workflow.ID, 10),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeIssueType): "pull_request", // Change to PR
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
string(project_model.WorkflowActionTypeColumn): strconv.FormatInt(column.ID, 10),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
body, err := json.Marshal(updateData)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
req := NewRequestWithBody(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/%d?_csrf=%s", user.Name, repo.Name, project.ID, workflow.ID, GetUserCSRFToken(t, session)),
|
||||||
|
strings.NewReader(string(body)))
|
||||||
|
req.Header.Set("Content-Type", "application/json")
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
// Parse response
|
||||||
|
var result map[string]any
|
||||||
|
err = json.Unmarshal(resp.Body.Bytes(), &result)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, result["success"].(bool))
|
||||||
|
|
||||||
|
// Verify workflow was updated
|
||||||
|
updatedWorkflow, err := project_model.GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, updatedWorkflow.Enabled)
|
||||||
|
assert.Len(t, updatedWorkflow.WorkflowFilters, 1)
|
||||||
|
assert.Equal(t, "pull_request", updatedWorkflow.WorkflowFilters[0].Value)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowToggleStatus(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create a project
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Project for Workflow Status",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create a workflow that is initially enabled
|
||||||
|
workflow := &project_model.Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: project_model.WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []project_model.WorkflowFilter{},
|
||||||
|
WorkflowActions: []project_model.WorkflowAction{},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err = project_model.CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Test 1: Toggle status from enabled to disabled
|
||||||
|
t.Run("Disable workflow", func(t *testing.T) {
|
||||||
|
req := NewRequestWithValues(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/%d/status?_csrf=%s", user.Name, repo.Name, project.ID, workflow.ID, GetUserCSRFToken(t, session)),
|
||||||
|
map[string]string{
|
||||||
|
"enabled": "false",
|
||||||
|
})
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
// Parse response
|
||||||
|
var result map[string]any
|
||||||
|
err = json.Unmarshal(resp.Body.Bytes(), &result)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, result["success"].(bool), "Response should indicate success")
|
||||||
|
|
||||||
|
// Verify status was changed to disabled
|
||||||
|
updatedWorkflow, err := project_model.GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.False(t, updatedWorkflow.Enabled, "Workflow should be disabled")
|
||||||
|
})
|
||||||
|
|
||||||
|
// Test 2: Toggle status from disabled to enabled
|
||||||
|
t.Run("Enable workflow", func(t *testing.T) {
|
||||||
|
req := NewRequestWithValues(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/%d/status?_csrf=%s", user.Name, repo.Name, project.ID, workflow.ID, GetUserCSRFToken(t, session)),
|
||||||
|
map[string]string{
|
||||||
|
"enabled": "true",
|
||||||
|
})
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
// Parse response
|
||||||
|
var result map[string]any
|
||||||
|
err = json.Unmarshal(resp.Body.Bytes(), &result)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, result["success"].(bool), "Response should indicate success")
|
||||||
|
|
||||||
|
// Verify status was changed back to enabled
|
||||||
|
updatedWorkflow, err := project_model.GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, updatedWorkflow.Enabled, "Workflow should be enabled")
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowDelete(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create a project
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Project for Workflow Delete",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create a workflow
|
||||||
|
workflow := &project_model.Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: project_model.WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []project_model.WorkflowFilter{},
|
||||||
|
WorkflowActions: []project_model.WorkflowAction{},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err = project_model.CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Delete the workflow
|
||||||
|
req := NewRequest(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/%d/delete?_csrf=%s", user.Name, repo.Name, project.ID, workflow.ID, GetUserCSRFToken(t, session)))
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
// Parse response
|
||||||
|
var result map[string]any
|
||||||
|
err = json.Unmarshal(resp.Body.Bytes(), &result)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.True(t, result["success"].(bool), "Delete response should indicate success")
|
||||||
|
|
||||||
|
// Verify workflow was deleted - should return ErrNotExist
|
||||||
|
_, err = project_model.GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.Error(t, err, "Should return an error when workflow doesn't exist")
|
||||||
|
assert.True(t, db.IsErrNotExist(err), "Error should be ErrNotExist type")
|
||||||
|
|
||||||
|
// Verify we cannot delete it again (should fail gracefully)
|
||||||
|
req = NewRequest(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/%d/delete?_csrf=%s", user.Name, repo.Name, project.ID, workflow.ID, GetUserCSRFToken(t, session)))
|
||||||
|
session.MakeRequest(t, req, http.StatusNotFound)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowPermissions(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 4})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create a project
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Project for Workflow Permissions",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create a workflow
|
||||||
|
workflow := &project_model.Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: project_model.WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []project_model.WorkflowFilter{},
|
||||||
|
WorkflowActions: []project_model.WorkflowAction{},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err = project_model.CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// User with write permission should be able to access workflows
|
||||||
|
session1 := loginUser(t, user.Name)
|
||||||
|
req := NewRequest(t, "GET", fmt.Sprintf("/%s/%s/projects/%d/workflows", user.Name, repo.Name, project.ID))
|
||||||
|
session1.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
// User without write permission should not be able to modify workflows
|
||||||
|
session2 := loginUser(t, user2.Name)
|
||||||
|
req = NewRequest(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/%d/delete?_csrf=%s", user.Name, repo.Name, project.ID, workflow.ID, GetUserCSRFToken(t, session2)))
|
||||||
|
session2.MakeRequest(t, req, http.StatusNotFound) // we use 404 to avoid leaking existence
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestProjectWorkflowValidation(t *testing.T) {
|
||||||
|
defer tests.PrepareTestEnv(t)()
|
||||||
|
|
||||||
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
||||||
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
|
||||||
|
|
||||||
|
// Create a project
|
||||||
|
project := &project_model.Project{
|
||||||
|
Title: "Test Project for Workflow Validation",
|
||||||
|
RepoID: repo.ID,
|
||||||
|
Type: project_model.TypeRepository,
|
||||||
|
TemplateType: project_model.TemplateTypeNone,
|
||||||
|
}
|
||||||
|
err := project_model.NewProject(t.Context(), project)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
session := loginUser(t, user.Name)
|
||||||
|
|
||||||
|
// Test 1: Try to create a workflow without any actions (should fail)
|
||||||
|
t.Run("Create workflow without actions should fail", func(t *testing.T) {
|
||||||
|
workflowData := map[string]any{
|
||||||
|
"event_id": string(project_model.WorkflowEventItemOpened),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeIssueType): "issue",
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
// No actions provided - this should trigger validation error
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
body, err := json.Marshal(workflowData)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
req := NewRequestWithBody(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/item_opened?_csrf=%s", user.Name, repo.Name, project.ID, GetUserCSRFToken(t, session)),
|
||||||
|
strings.NewReader(string(body)))
|
||||||
|
req.Header.Set("Content-Type", "application/json")
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusBadRequest)
|
||||||
|
|
||||||
|
// Parse response
|
||||||
|
var result map[string]any
|
||||||
|
err = json.Unmarshal(resp.Body.Bytes(), &result)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Equal(t, "NoActions", result["error"], "Error should be NoActions")
|
||||||
|
assert.NotEmpty(t, result["message"], "Error message should be provided")
|
||||||
|
})
|
||||||
|
|
||||||
|
// Test 2: Try to update a workflow to have no actions (should fail)
|
||||||
|
t.Run("Update workflow to remove all actions should fail", func(t *testing.T) {
|
||||||
|
// First create a valid workflow
|
||||||
|
column := &project_model.Column{
|
||||||
|
Title: "Test Column",
|
||||||
|
ProjectID: project.ID,
|
||||||
|
}
|
||||||
|
err := project_model.NewColumn(t.Context(), column)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
workflow := &project_model.Workflow{
|
||||||
|
ProjectID: project.ID,
|
||||||
|
WorkflowEvent: project_model.WorkflowEventItemOpened,
|
||||||
|
WorkflowFilters: []project_model.WorkflowFilter{
|
||||||
|
{
|
||||||
|
Type: project_model.WorkflowFilterTypeIssueType,
|
||||||
|
Value: "issue",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
WorkflowActions: []project_model.WorkflowAction{
|
||||||
|
{
|
||||||
|
Type: project_model.WorkflowActionTypeColumn,
|
||||||
|
Value: strconv.FormatInt(column.ID, 10),
|
||||||
|
},
|
||||||
|
},
|
||||||
|
Enabled: true,
|
||||||
|
}
|
||||||
|
err = project_model.CreateWorkflow(t.Context(), workflow)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Try to update it to have no actions
|
||||||
|
updateData := map[string]any{
|
||||||
|
"event_id": strconv.FormatInt(workflow.ID, 10),
|
||||||
|
"filters": map[string]any{
|
||||||
|
string(project_model.WorkflowFilterTypeIssueType): "issue",
|
||||||
|
},
|
||||||
|
"actions": map[string]any{
|
||||||
|
// No actions - should fail
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
body, err := json.Marshal(updateData)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
req := NewRequestWithBody(t, "POST",
|
||||||
|
fmt.Sprintf("/%s/%s/projects/%d/workflows/%d?_csrf=%s", user.Name, repo.Name, project.ID, workflow.ID, GetUserCSRFToken(t, session)),
|
||||||
|
strings.NewReader(string(body)))
|
||||||
|
req.Header.Set("Content-Type", "application/json")
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusBadRequest)
|
||||||
|
|
||||||
|
// Parse response
|
||||||
|
var result map[string]any
|
||||||
|
err = json.Unmarshal(resp.Body.Bytes(), &result)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Equal(t, "NoActions", result["error"], "Error should be NoActions")
|
||||||
|
assert.NotEmpty(t, result["message"], "Error message should be provided")
|
||||||
|
|
||||||
|
// Verify the workflow was not changed
|
||||||
|
unchangedWorkflow, err := project_model.GetWorkflowByID(t.Context(), workflow.ID)
|
||||||
|
assert.NoError(t, err)
|
||||||
|
assert.Len(t, unchangedWorkflow.WorkflowActions, 1, "Workflow should still have the original action")
|
||||||
|
})
|
||||||
|
}
|
||||||
File diff suppressed because it is too large
Load Diff
@ -0,0 +1,358 @@
|
|||||||
|
import {reactive} from 'vue';
|
||||||
|
import {GET, POST} from '../../modules/fetch.ts';
|
||||||
|
import {showErrorToast} from '../../modules/toast.ts';
|
||||||
|
|
||||||
|
type WorkflowFilters = {
|
||||||
|
issue_type: string;
|
||||||
|
source_column: string;
|
||||||
|
target_column: string;
|
||||||
|
labels: string[];
|
||||||
|
};
|
||||||
|
|
||||||
|
type WorkflowIssueStateAction = '' | 'close' | 'reopen';
|
||||||
|
|
||||||
|
type WorkflowActions = {
|
||||||
|
column: string;
|
||||||
|
add_labels: string[];
|
||||||
|
remove_labels: string[];
|
||||||
|
issue_state: WorkflowIssueStateAction;
|
||||||
|
};
|
||||||
|
|
||||||
|
type WorkflowDraftState = {
|
||||||
|
filters: WorkflowFilters;
|
||||||
|
actions: WorkflowActions;
|
||||||
|
};
|
||||||
|
|
||||||
|
const createDefaultFilters = (): WorkflowFilters => ({issue_type: '', source_column: '', target_column: '', labels: []});
|
||||||
|
const createDefaultActions = (): WorkflowActions => ({column: '', add_labels: [], remove_labels: [], issue_state: ''});
|
||||||
|
|
||||||
|
function convertFilters(workflow: any): WorkflowFilters {
|
||||||
|
const filters = createDefaultFilters();
|
||||||
|
if (workflow?.filters && Array.isArray(workflow.filters)) {
|
||||||
|
for (const filter of workflow.filters) {
|
||||||
|
if (filter.type === 'issue_type') {
|
||||||
|
filters.issue_type = filter.value;
|
||||||
|
} else if (filter.type === 'source_column') {
|
||||||
|
filters.source_column = filter.value;
|
||||||
|
} else if (filter.type === 'target_column') {
|
||||||
|
filters.target_column = filter.value;
|
||||||
|
} else if (filter.type === 'labels') {
|
||||||
|
filters.labels.push(filter.value);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return filters;
|
||||||
|
}
|
||||||
|
|
||||||
|
function convertActions(workflow: any): WorkflowActions {
|
||||||
|
const actions = createDefaultActions();
|
||||||
|
|
||||||
|
if (workflow?.actions && Array.isArray(workflow.actions)) {
|
||||||
|
for (const action of workflow.actions) {
|
||||||
|
if (action.type === 'column') {
|
||||||
|
// Backend returns string, keep as string to match column.id type
|
||||||
|
actions.column = action.value;
|
||||||
|
} else if (action.type === 'add_labels') {
|
||||||
|
// Backend returns string, keep as string to match label.id type
|
||||||
|
actions.add_labels.push(action.value);
|
||||||
|
} else if (action.type === 'remove_labels') {
|
||||||
|
// Backend returns string, keep as string to match label.id type
|
||||||
|
actions.remove_labels.push(action.value);
|
||||||
|
} else if (action.type === 'issue_state') {
|
||||||
|
actions.issue_state = action.value as WorkflowIssueStateAction;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return actions;
|
||||||
|
}
|
||||||
|
|
||||||
|
const cloneFilters = (filters: WorkflowFilters): WorkflowFilters => ({
|
||||||
|
issue_type: filters.issue_type,
|
||||||
|
source_column: filters.source_column,
|
||||||
|
target_column: filters.target_column,
|
||||||
|
labels: Array.from(filters.labels),
|
||||||
|
});
|
||||||
|
|
||||||
|
const cloneActions = (actions: WorkflowActions): WorkflowActions => ({
|
||||||
|
column: actions.column,
|
||||||
|
add_labels: Array.from(actions.add_labels),
|
||||||
|
remove_labels: Array.from(actions.remove_labels),
|
||||||
|
issue_state: actions.issue_state,
|
||||||
|
});
|
||||||
|
|
||||||
|
export function createWorkflowStore(props: any) {
|
||||||
|
const store = reactive({
|
||||||
|
workflowEvents: [],
|
||||||
|
selectedItem: props.eventID,
|
||||||
|
selectedWorkflow: null,
|
||||||
|
projectColumns: [],
|
||||||
|
projectLabels: [], // Add labels data
|
||||||
|
saving: false,
|
||||||
|
loading: false, // Add loading state to prevent rapid clicks
|
||||||
|
showCreateDialog: false, // For create workflow dialog
|
||||||
|
selectedEventType: null, // For workflow creation
|
||||||
|
|
||||||
|
workflowFilters: createDefaultFilters(),
|
||||||
|
workflowActions: createDefaultActions(),
|
||||||
|
|
||||||
|
workflowDrafts: {} as Record<string, WorkflowDraftState>,
|
||||||
|
|
||||||
|
getDraft(eventId: string): WorkflowDraftState | undefined {
|
||||||
|
return store.workflowDrafts[eventId];
|
||||||
|
},
|
||||||
|
|
||||||
|
updateDraft(eventId: string, filters: WorkflowFilters, actions: WorkflowActions) {
|
||||||
|
store.workflowDrafts[eventId] = {
|
||||||
|
filters: cloneFilters(filters),
|
||||||
|
actions: cloneActions(actions),
|
||||||
|
};
|
||||||
|
},
|
||||||
|
|
||||||
|
clearDraft(eventId: string) {
|
||||||
|
delete store.workflowDrafts[eventId];
|
||||||
|
},
|
||||||
|
|
||||||
|
async loadEvents() {
|
||||||
|
const response = await GET(`${props.projectLink}/workflows/events`);
|
||||||
|
store.workflowEvents = await response.json();
|
||||||
|
return store.workflowEvents;
|
||||||
|
},
|
||||||
|
|
||||||
|
async loadProjectColumns() {
|
||||||
|
try {
|
||||||
|
const response = await GET(`${props.projectLink}/workflows/columns`);
|
||||||
|
store.projectColumns = await response.json();
|
||||||
|
} catch (error) {
|
||||||
|
console.error('Failed to load project columns:', error);
|
||||||
|
store.projectColumns = [];
|
||||||
|
}
|
||||||
|
},
|
||||||
|
|
||||||
|
async loadWorkflowData(eventId: string) {
|
||||||
|
store.loading = true;
|
||||||
|
try {
|
||||||
|
// Load project columns and labels for the dropdowns
|
||||||
|
await store.loadProjectColumns();
|
||||||
|
await store.loadProjectLabels();
|
||||||
|
|
||||||
|
const draft = store.getDraft(eventId);
|
||||||
|
if (draft) {
|
||||||
|
store.workflowFilters = cloneFilters(draft.filters);
|
||||||
|
store.workflowActions = cloneActions(draft.actions);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
// Find the workflow from existing workflowEvents
|
||||||
|
const workflow = store.workflowEvents.find((e) => e.event_id === eventId);
|
||||||
|
|
||||||
|
store.workflowFilters = convertFilters(workflow);
|
||||||
|
store.workflowActions = convertActions(workflow);
|
||||||
|
store.updateDraft(eventId, store.workflowFilters, store.workflowActions);
|
||||||
|
} finally {
|
||||||
|
store.loading = false;
|
||||||
|
}
|
||||||
|
},
|
||||||
|
|
||||||
|
async loadProjectLabels() {
|
||||||
|
try {
|
||||||
|
const response = await GET(`${props.projectLink}/workflows/labels`);
|
||||||
|
store.projectLabels = await response.json();
|
||||||
|
} catch (error) {
|
||||||
|
console.error('Failed to load project labels:', error);
|
||||||
|
store.projectLabels = [];
|
||||||
|
}
|
||||||
|
},
|
||||||
|
|
||||||
|
resetWorkflowData() {
|
||||||
|
store.workflowFilters = createDefaultFilters();
|
||||||
|
store.workflowActions = createDefaultActions();
|
||||||
|
|
||||||
|
const currentEventId = store.selectedWorkflow?.event_id;
|
||||||
|
if (currentEventId) {
|
||||||
|
store.updateDraft(currentEventId, store.workflowFilters, store.workflowActions);
|
||||||
|
}
|
||||||
|
},
|
||||||
|
|
||||||
|
async saveWorkflow() {
|
||||||
|
if (!store.selectedWorkflow) return;
|
||||||
|
|
||||||
|
// Validate: at least one action must be configured
|
||||||
|
const hasAtLeastOneAction = Boolean(
|
||||||
|
store.workflowActions.column ||
|
||||||
|
store.workflowActions.add_labels.length > 0 ||
|
||||||
|
store.workflowActions.remove_labels.length > 0 ||
|
||||||
|
store.workflowActions.issue_state,
|
||||||
|
);
|
||||||
|
|
||||||
|
if (!hasAtLeastOneAction) {
|
||||||
|
showErrorToast(props.locale.atLeastOneActionRequired || 'At least one action must be configured');
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
store.saving = true;
|
||||||
|
try {
|
||||||
|
// For new workflows, use the base event type
|
||||||
|
const eventId = store.selectedWorkflow.event_id;
|
||||||
|
|
||||||
|
// Convert frontend data format to backend JSON format
|
||||||
|
const postData = {
|
||||||
|
event_id: eventId,
|
||||||
|
filters: store.workflowFilters,
|
||||||
|
actions: store.workflowActions,
|
||||||
|
};
|
||||||
|
|
||||||
|
const response = await POST(`${props.projectLink}/workflows/${eventId}`, {
|
||||||
|
data: postData,
|
||||||
|
headers: {
|
||||||
|
'Content-Type': 'application/json',
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
if (!response.ok) {
|
||||||
|
let errorMessage = `${props.locale.failedToSaveWorkflow}: ${response.status} ${response.statusText}`;
|
||||||
|
try {
|
||||||
|
const errorData = await response.json();
|
||||||
|
if (errorData.message) {
|
||||||
|
errorMessage = errorData.message;
|
||||||
|
} else if (errorData.error === 'NoActions') {
|
||||||
|
errorMessage = props.locale.atLeastOneActionRequired || 'At least one action must be configured';
|
||||||
|
}
|
||||||
|
} catch {
|
||||||
|
const errorText = await response.text();
|
||||||
|
console.error('Response error:', errorText);
|
||||||
|
errorMessage += `\n${errorText}`;
|
||||||
|
}
|
||||||
|
showErrorToast(errorMessage);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const result = await response.json();
|
||||||
|
if (result.success && result.workflow) {
|
||||||
|
// Always reload the events list to get the updated structure
|
||||||
|
// This ensures we have both the base event and the new filtered event
|
||||||
|
const eventKey = typeof store.selectedWorkflow.event_id === 'string' ? store.selectedWorkflow.event_id : '';
|
||||||
|
const wasNewWorkflow = store.selectedWorkflow.id === 0 ||
|
||||||
|
eventKey.startsWith('new-') ||
|
||||||
|
eventKey.startsWith('clone-');
|
||||||
|
|
||||||
|
if (wasNewWorkflow) {
|
||||||
|
store.clearDraft(store.selectedWorkflow.workflow_event);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Reload events from server to get the correct event structure
|
||||||
|
await store.loadEvents();
|
||||||
|
|
||||||
|
// Find the reloaded workflow which has complete data including capabilities
|
||||||
|
const reloadedWorkflow = store.workflowEvents.find((w) => w.event_id === result.workflow.event_id);
|
||||||
|
|
||||||
|
if (reloadedWorkflow) {
|
||||||
|
// Use the reloaded workflow as it has all the necessary fields
|
||||||
|
store.selectedWorkflow = reloadedWorkflow;
|
||||||
|
store.selectedItem = reloadedWorkflow.event_id;
|
||||||
|
} else {
|
||||||
|
// Fallback: use the result from backend (shouldn't normally happen)
|
||||||
|
store.selectedWorkflow = result.workflow;
|
||||||
|
store.selectedItem = result.workflow.event_id;
|
||||||
|
}
|
||||||
|
|
||||||
|
store.workflowFilters = convertFilters(store.selectedWorkflow);
|
||||||
|
store.workflowActions = convertActions(store.selectedWorkflow);
|
||||||
|
if (store.selectedWorkflow?.event_id) {
|
||||||
|
store.updateDraft(store.selectedWorkflow.event_id, store.workflowFilters, store.workflowActions);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Update URL to use the new workflow ID
|
||||||
|
if (wasNewWorkflow) {
|
||||||
|
const newUrl = `${props.projectLink}/workflows/${store.selectedWorkflow.event_id}`;
|
||||||
|
window.history.replaceState({eventId: store.selectedWorkflow.event_id}, '', newUrl);
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
console.error('Unexpected response format:', result);
|
||||||
|
showErrorToast(`${props.locale.failedToSaveWorkflow}: Unexpected response format`);
|
||||||
|
}
|
||||||
|
} catch (error) {
|
||||||
|
console.error('Failed to save workflow:', error);
|
||||||
|
showErrorToast(`${props.locale.failedToSaveWorkflow}: ${error.message}`);
|
||||||
|
} finally {
|
||||||
|
store.saving = false;
|
||||||
|
}
|
||||||
|
},
|
||||||
|
|
||||||
|
async saveWorkflowStatus() {
|
||||||
|
if (!store.selectedWorkflow || store.selectedWorkflow.id === 0) return;
|
||||||
|
|
||||||
|
try {
|
||||||
|
const formData = new FormData();
|
||||||
|
formData.append('enabled', store.selectedWorkflow.enabled.toString());
|
||||||
|
|
||||||
|
// Use workflow ID for status update
|
||||||
|
const workflowId = store.selectedWorkflow.id;
|
||||||
|
const response = await POST(`${props.projectLink}/workflows/${workflowId}/status`, {
|
||||||
|
data: formData,
|
||||||
|
});
|
||||||
|
|
||||||
|
if (!response.ok) {
|
||||||
|
const errorText = await response.text();
|
||||||
|
console.error('Failed to update workflow status:', errorText);
|
||||||
|
showErrorToast(`${props.locale.failedToUpdateWorkflowStatus}: ${response.status} ${response.statusText}`);
|
||||||
|
// Revert the status change on error
|
||||||
|
store.selectedWorkflow.enabled = !store.selectedWorkflow.enabled;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const result = await response.json();
|
||||||
|
if (result.success) {
|
||||||
|
// Update workflow in the list
|
||||||
|
const existingIndex = store.workflowEvents.findIndex((e) => e.event_id === store.selectedWorkflow.event_id);
|
||||||
|
if (existingIndex >= 0) {
|
||||||
|
store.workflowEvents[existingIndex].enabled = store.selectedWorkflow.enabled;
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
// Revert the status change on failure
|
||||||
|
store.selectedWorkflow.enabled = !store.selectedWorkflow.enabled;
|
||||||
|
showErrorToast(`${props.locale.failedToUpdateWorkflowStatus}: Unexpected error`);
|
||||||
|
}
|
||||||
|
} catch (error) {
|
||||||
|
console.error('Failed to update workflow status:', error);
|
||||||
|
// Revert the status change on error
|
||||||
|
store.selectedWorkflow.enabled = !store.selectedWorkflow.enabled;
|
||||||
|
showErrorToast(`${props.locale.failedToUpdateWorkflowStatus}: ${error.message}`);
|
||||||
|
}
|
||||||
|
},
|
||||||
|
|
||||||
|
async deleteWorkflow() {
|
||||||
|
if (!store.selectedWorkflow || store.selectedWorkflow.id === 0) return;
|
||||||
|
|
||||||
|
try {
|
||||||
|
// Use workflow ID for deletion
|
||||||
|
const workflowId = store.selectedWorkflow.id;
|
||||||
|
const response = await POST(`${props.projectLink}/workflows/${workflowId}/delete`, {
|
||||||
|
data: new FormData(),
|
||||||
|
});
|
||||||
|
|
||||||
|
if (!response.ok) {
|
||||||
|
const errorText = await response.text();
|
||||||
|
console.error('Failed to delete workflow:', errorText);
|
||||||
|
showErrorToast(`${props.locale.failedToDeleteWorkflow}: ${response.status} ${response.statusText}`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const result = await response.json();
|
||||||
|
if (result.success) {
|
||||||
|
// Remove workflow from the list
|
||||||
|
const existingIndex = store.workflowEvents.findIndex((e) => e.event_id === store.selectedWorkflow.event_id);
|
||||||
|
if (existingIndex >= 0) {
|
||||||
|
store.workflowEvents.splice(existingIndex, 1);
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
showErrorToast(`${props.locale.failedToDeleteWorkflow}: Unexpected error`);
|
||||||
|
}
|
||||||
|
} catch (error) {
|
||||||
|
console.error('Error deleting workflow:', error);
|
||||||
|
showErrorToast(`${props.locale.failedToDeleteWorkflow}: ${error.message}`);
|
||||||
|
}
|
||||||
|
},
|
||||||
|
|
||||||
|
});
|
||||||
|
return store;
|
||||||
|
}
|
||||||
@ -0,0 +1,60 @@
|
|||||||
|
import {createApp} from 'vue';
|
||||||
|
import ProjectWorkflow from '../../components/projects/ProjectWorkflow.vue';
|
||||||
|
|
||||||
|
export async function initProjectWorkflow() {
|
||||||
|
const workflowDiv = document.querySelector('#project-workflows');
|
||||||
|
if (!workflowDiv) return;
|
||||||
|
|
||||||
|
try {
|
||||||
|
const locale = {
|
||||||
|
defaultWorkflows: workflowDiv.getAttribute('data-locale-default-workflows'),
|
||||||
|
moveToColumn: workflowDiv.getAttribute('data-locale-move-to-column'),
|
||||||
|
viewWorkflowConfiguration: workflowDiv.getAttribute('data-locale-view-workflow-configuration'),
|
||||||
|
configureWorkflow: workflowDiv.getAttribute('data-locale-configure-workflow'),
|
||||||
|
when: workflowDiv.getAttribute('data-locale-when'),
|
||||||
|
runWhen: workflowDiv.getAttribute('data-locale-run-when'),
|
||||||
|
filters: workflowDiv.getAttribute('data-locale-filters'),
|
||||||
|
applyTo: workflowDiv.getAttribute('data-locale-apply-to'),
|
||||||
|
whenMovedFromColumn: workflowDiv.getAttribute('data-locale-when-moved-from-column'),
|
||||||
|
whenMovedToColumn: workflowDiv.getAttribute('data-locale-when-moved-to-column'),
|
||||||
|
onlyIfHasLabels: workflowDiv.getAttribute('data-locale-only-if-has-labels'),
|
||||||
|
actions: workflowDiv.getAttribute('data-locale-actions'),
|
||||||
|
addLabels: workflowDiv.getAttribute('data-locale-add-labels'),
|
||||||
|
removeLabels: workflowDiv.getAttribute('data-locale-remove-labels'),
|
||||||
|
anyLabel: workflowDiv.getAttribute('data-locale-any-label'),
|
||||||
|
anyColumn: workflowDiv.getAttribute('data-locale-any-column'),
|
||||||
|
issueState: workflowDiv.getAttribute('data-locale-issue-state'),
|
||||||
|
none: workflowDiv.getAttribute('data-locale-none'),
|
||||||
|
noChange: workflowDiv.getAttribute('data-locale-no-change'),
|
||||||
|
edit: workflowDiv.getAttribute('data-locale-edit'),
|
||||||
|
delete: workflowDiv.getAttribute('data-locale-delete'),
|
||||||
|
save: workflowDiv.getAttribute('data-locale-save'),
|
||||||
|
clone: workflowDiv.getAttribute('data-locale-clone'),
|
||||||
|
cancel: workflowDiv.getAttribute('data-locale-cancel'),
|
||||||
|
disable: workflowDiv.getAttribute('data-locale-disable'),
|
||||||
|
disabled: workflowDiv.getAttribute('data-locale-disabled'),
|
||||||
|
enabled: workflowDiv.getAttribute('data-locale-enabled'),
|
||||||
|
enable: workflowDiv.getAttribute('data-locale-enable'),
|
||||||
|
issuesAndPullRequests: workflowDiv.getAttribute('data-locale-issues-and-pull-requests'),
|
||||||
|
issuesOnly: workflowDiv.getAttribute('data-locale-issues-only'),
|
||||||
|
pullRequestsOnly: workflowDiv.getAttribute('data-locale-pull-requests-only'),
|
||||||
|
selectColumn: workflowDiv.getAttribute('data-locale-select-column'),
|
||||||
|
closeIssue: workflowDiv.getAttribute('data-locale-close-issue'),
|
||||||
|
reopenIssue: workflowDiv.getAttribute('data-locale-reopen-issue'),
|
||||||
|
saveWorkflowFailed: workflowDiv.getAttribute('data-locale-save-workflow-failed'),
|
||||||
|
updateWorkflowFailed: workflowDiv.getAttribute('data-locale-update-workflow-failed'),
|
||||||
|
deleteWorkflowFailed: workflowDiv.getAttribute('data-locale-delete-workflow-failed'),
|
||||||
|
atLeastOneActionRequired: workflowDiv.getAttribute('data-locale-at-least-one-action-required'),
|
||||||
|
};
|
||||||
|
|
||||||
|
const View = createApp(ProjectWorkflow, {
|
||||||
|
projectLink: workflowDiv.getAttribute('data-project-link'),
|
||||||
|
eventID: workflowDiv.getAttribute('data-event-id'),
|
||||||
|
locale,
|
||||||
|
});
|
||||||
|
View.mount(workflowDiv);
|
||||||
|
} catch (err) {
|
||||||
|
console.error('Project Workflow failed to load', err);
|
||||||
|
workflowDiv.textContent = 'Project Workflow failed to load';
|
||||||
|
}
|
||||||
|
}
|
||||||
Loading…
Reference in New Issue