Skip to content

Commit fda7476

Browse files
committed
Update error messages for org labels + add missing assertion in test
1 parent e76508a commit fda7476

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

routers/web/org/org_labels.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ func NewLabel(ctx *context.Context) {
5353
}
5454
if err := issues_model.NewLabel(ctx, l); err != nil {
5555
if label.IsErrInvalidLabelColor(err) {
56-
ctx.Flash.Error(err.Error())
56+
ctx.Flash.Error("NewLabel: " + err.Error())
5757
} else {
5858
ctx.ServerError("NewLabel", err)
5959
return
@@ -85,7 +85,7 @@ func UpdateLabel(ctx *context.Context) {
8585
l.SetArchived(form.IsArchived)
8686
if err := issues_model.UpdateLabel(ctx, l); err != nil {
8787
if label.IsErrInvalidLabelColor(err) {
88-
ctx.Flash.Error(err.Error())
88+
ctx.Flash.Error("UpdateLabel: " + err.Error())
8989
} else {
9090
ctx.ServerError("UpdateLabel", err)
9191
return

routers/web/repo/issue_label_test.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,7 @@ func TestNewLabelGivenInvalidLabelCode(t *testing.T) {
103103
})
104104
NewLabel(ctx)
105105
assert.Equal(t, http.StatusSeeOther, ctx.Resp.WrittenStatus())
106+
assert.Equal(t, "/user2/repo1/labels", test.RedirectURL(ctx.Resp))
106107
assert.True(t, ctx.Flash.Has("error"))
107108
unittest.AssertNotExistsBean(t, &issues_model.Label{
108109
Name: "newlabel",

0 commit comments

Comments
 (0)