refactor: change to GetRunsByIDsAndTriggerUserID

Author: zsbahtiar

models/actions/run.go

@@ -480,9 +480,11 @@ func DeleteRunByIDs(ctx context.Context, runIDs, jobIDs []int64) error {
 	return committer.Commit()
 }
 
-func GetRunsByIDs(ctx context.Context, ids []int64) ([]*ActionRun, error) {
+// GetRunsByIDsAndTriggerUserID -- get all action run by trigger user with selected ids
+func GetRunsByIDsAndTriggerUserID(ctx context.Context, ids []int64, triggerUserID int64) ([]*ActionRun, error) {
 	var runs []*ActionRun
-	err := db.GetEngine(ctx).In("id", ids).Find(&runs)
+	err := db.GetEngine(ctx).Where("trigger_user_id=?", triggerUserID).
+		In("id", ids).Find(&runs)
 	if err != nil {
 		return nil, err
 	}

models/actions/run_test.go

@@ -0,0 +1,14 @@
+package actions
+
+import (
+	db_model "code.gitea.io/gitea/models/db"
+	"testing"
+)
+
+func TestDeleteRunByIDs(t *testing.T) {
+	tests := []struct{
+		name string
+		mock func(m *db_model.Engine)
+
+	}
+}

routers/web/repo/actions/actions.go

@@ -441,7 +441,7 @@ func DeleteRuns(ctx *context.Context) {
 		return
 	}
 
-	actionRun, err := actions_model.GetRunsByIDs(ctx, req.ActionIDs)
+	actionRun, err := actions_model.GetRunsByIDsAndTriggerUserID(ctx, req.ActionIDs, ctx.Doer.ID)
 	if err != nil {
 		ctx.ServerError("failed to get action_run", err)
 		return