Nulo/forgejo
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Prevent "Race" detected in TestAdmin*User (#16830)

Browse source 
These tests are missing the defer prefix.

Related #1441

Signed-off-by: Andrew Thornton <art27@cantab.net>
This commit is contained in:
zeripath 2021-08-27 14:00:37 +01:00 committed by GitHub
parent e37342db0c
commit cd8db3a83d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
integrations/admin_user_test.go
View file

@ -15,7 +15,7 @@ import (
)
func TestAdminViewUsers(t *testing.T) {
prepareTestEnv(t)
defer prepareTestEnv(t)()
session := loginUser(t, "user1")
req := NewRequest(t, "GET", "/admin/users")
@ -27,7 +27,7 @@ func TestAdminViewUsers(t *testing.T) {
}
func TestAdminViewUser(t *testing.T) {
prepareTestEnv(t)
defer prepareTestEnv(t)()
session := loginUser(t, "user1")
req := NewRequest(t, "GET", "/admin/users/1")
@ -39,7 +39,7 @@ func TestAdminViewUser(t *testing.T) {
}
func TestAdminEditUser(t *testing.T) {
prepareTestEnv(t)
defer prepareTestEnv(t)()
testSuccessfullEdit(t, models.User{ID: 2, Name: "newusername", LoginName: "otherlogin", Email: "new@e-mail.gitea"})
}