Skip to content

Commit c87cbd7

Browse files
committed
message format changes.
1 parent ae4ef11 commit c87cbd7

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

api/user/UserRestHandler.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -229,12 +229,12 @@ func (handler UserRestHandlerImpl) UpdateUser(w http.ResponseWriter, r *http.Req
229229

230230
if rolesChanged || groupsModified {
231231
// warning
232-
message := fmt.Errorf("User permissions updated partially. " + groups + " could not be modified. You do not have manager permission for some or all projects in these groups.")
232+
message := fmt.Errorf("User permissions updated partially. Group(s): " + groups + " could not be added/removed. You do not have manager permission for some or all projects in these groups.")
233233
common.WriteJsonResp(w, message, nil, http.StatusExpectationFailed)
234234

235235
} else {
236236
//error
237-
message := fmt.Errorf("Permission could not be modified: You do not have manager permission for some or all projects in " + groups)
237+
message := fmt.Errorf("Permission could not be added/removed: You do not have manager permission for some or all projects in group(s): " + groups)
238238
common.WriteJsonResp(w, message, nil, http.StatusBadRequest)
239239
}
240240
}

pkg/user/UserService_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ func TestUserUpdateService(t *testing.T) {
189189
nil,
190190
nil)
191191

192-
token := "eyJhbGciOiJSUzI1NiIsImtpZCI6IjZhNmE3Y2Y4ZTkzNDE1Njk3N2U4MmVlYzQ4M2I2OTM3NjRmMTU4NjcifQ.eyJpc3MiOiJodHRwczovL2RlbW8uZGV2dHJvbi5pbmZvOjMyNDQzL29yY2hlc3RyYXRvci9hcGkvZGV4Iiwic3ViIjoiQ2hVeE1ESTFOVGd3TURjNU5qa3pNelk1T0Rrek5USVNCbWR2YjJkc1pRIiwiYXVkIjoiYXJnby1jZCIsImV4cCI6MTY2NjEyMDMzNCwiaWF0IjoxNjY2MDMzOTM0LCJhdF9oYXNoIjoiUGtNdDJmYzlGQ1VXU2NRNEFuY0Y3USIsImNfaGFzaCI6ImN3WVh2bnhWbnVmdkFtWjdWZXV1elEiLCJlbWFpbCI6ImF5dXNoQGRldnRyb24uYWkiLCJlbWFpbF92ZXJpZmllZCI6dHJ1ZSwibmFtZSI6IkF5dXNoIE1haGVzaHdhcmkifQ.vqUl-bJJc8Eg5fiIo-dOTIQfS-VZTHvVXSNFA24XJT1ITqgXYNC7RuQT-IF-ZR2c5R_2mmiQ7gjZsAe9aztHUD0pkeHNThoVU-BFiqp5Wv1IkQpoRNzFen-qlKJccTKogOXFV22whvdi0n-EHzr7d0km8IuOihaYP00W6xG1wEk1WpDFMqusKXj3iQCzA_yV6Y6V-GgkngpKtkSeLtTUzLtXkyMhA3VOY4HMGaLDZ2kI29alN8j7TrGukhBSfKRxYz-WOk746tW2e3NLeT0nR9AHHtthr6fwPpmMsF4uYZzp385cjprsk0jiugMBFJsGljIJvrDwqQx3hYX_wFNmFw"
192+
token := ""
193193
_, isRolesChanged, isGroupsModified, restrictedGroups, _ := userServiceImpl.UpdateUser(&userInfo, token, nil)
194194

195195
assert.Equal(t, isRolesChanged, false)

0 commit comments

Comments
 (0)