Fix test

lunny · lunny · commit 9dd9893fdd64 · 2025-05-11T20:33:43.000-07:00
diff --git a/routers/api/v1/repo/issue.go b/routers/api/v1/repo/issue.go
@@ -898,7 +898,7 @@ func EditIssue(ctx *context.APIContext) {
 		if issue.MilestoneID > 0 {
 			issue.Milestone, err = issues_model.GetMilestoneByRepoID(ctx, ctx.Repo.Repository.ID, *form.Milestone)
 			if err != nil {
-				ctx.APIErrorInternal(err)
+				ctx.Error(http.StatusInternalServerError, "GetMilestoneByRepoID", err)
 				return
 			}
 		} else {
diff --git a/routers/api/v1/repo/pull.go b/routers/api/v1/repo/pull.go
@@ -696,7 +696,7 @@ func EditPullRequest(ctx *context.APIContext) {
 		issue.MilestoneID = form.Milestone
 		issue.Milestone, err = issues_model.GetMilestoneByRepoID(ctx, ctx.Repo.Repository.ID, form.Milestone)
 		if err != nil {
-			ctx.APIErrorInternal(err)
+			ctx.Error(http.StatusInternalServerError, "GetMilestoneByRepoID", err)
 			return
 		}
 		if err = issue_service.ChangeMilestoneAssign(ctx, issue, ctx.Doer, oldMilestoneID); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -898,7 +898,7 @@ func EditIssue(ctx *context.APIContext) {`
`898`	`898`	`if issue.MilestoneID > 0 {`
`899`	`899`	`issue.Milestone, err = issues_model.GetMilestoneByRepoID(ctx, ctx.Repo.Repository.ID, *form.Milestone)`
`900`	`900`	`if err != nil {`
`901`		`- ctx.APIErrorInternal(err)`
	`901`	`+ ctx.Error(http.StatusInternalServerError, "GetMilestoneByRepoID", err)`
`902`	`902`	`return`
`903`	`903`	`}`
`904`	`904`	`} else {`
Original file line number	Diff line number	Diff line change
`@@ -696,7 +696,7 @@ func EditPullRequest(ctx *context.APIContext) {`
`696`	`696`	`issue.MilestoneID = form.Milestone`
`697`	`697`	`issue.Milestone, err = issues_model.GetMilestoneByRepoID(ctx, ctx.Repo.Repository.ID, form.Milestone)`
`698`	`698`	`if err != nil {`
`699`		`- ctx.APIErrorInternal(err)`
	`699`	`+ ctx.Error(http.StatusInternalServerError, "GetMilestoneByRepoID", err)`
`700`	`700`	`return`
`701`	`701`	`}`
`702`	`702`	`if err = issue_service.ChangeMilestoneAssign(ctx, issue, ctx.Doer, oldMilestoneID); err != nil {`