Skip to content

Commit 47f40cc

Browse files
ethantkoeniglunny
authored andcommitted
Add integration test for user deleting account (#2891)
1 parent 5abd1d2 commit 47f40cc

File tree

1 file changed

+45
-2
lines changed

1 file changed

+45
-2
lines changed

integrations/delete_user_test.go

Lines changed: 45 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,26 @@
55
package integrations
66

77
import (
8+
"fmt"
89
"net/http"
910
"testing"
1011

1112
"code.gitea.io/gitea/models"
1213
)
1314

14-
func TestDeleteUser(t *testing.T) {
15+
func assertUserDeleted(t *testing.T, userID int64) {
16+
models.AssertNotExistsBean(t, &models.User{ID: userID})
17+
models.AssertNotExistsBean(t, &models.Follow{UserID: userID})
18+
models.AssertNotExistsBean(t, &models.Follow{FollowID: userID})
19+
models.AssertNotExistsBean(t, &models.Repository{OwnerID: userID})
20+
models.AssertNotExistsBean(t, &models.Access{UserID: userID})
21+
models.AssertNotExistsBean(t, &models.OrgUser{UID: userID})
22+
models.AssertNotExistsBean(t, &models.IssueUser{UID: userID})
23+
models.AssertNotExistsBean(t, &models.TeamUser{UID: userID})
24+
models.AssertNotExistsBean(t, &models.Star{UID: userID})
25+
}
26+
27+
func TestAdminDeleteUser(t *testing.T) {
1528
prepareTestEnv(t)
1629

1730
session := loginUser(t, "user1")
@@ -22,6 +35,36 @@ func TestDeleteUser(t *testing.T) {
2235
})
2336
session.MakeRequest(t, req, http.StatusOK)
2437

25-
models.AssertNotExistsBean(t, &models.User{ID: 8})
38+
assertUserDeleted(t, 8)
2639
models.CheckConsistencyFor(t, &models.User{})
2740
}
41+
42+
func TestUserDeleteAccount(t *testing.T) {
43+
prepareTestEnv(t)
44+
45+
session := loginUser(t, "user8")
46+
csrf := GetCSRF(t, session, "/user/settings/delete")
47+
urlStr := fmt.Sprintf("/user/settings/delete?password=%s", userPassword)
48+
req := NewRequestWithValues(t, "POST", urlStr, map[string]string{
49+
"_csrf": csrf,
50+
})
51+
session.MakeRequest(t, req, http.StatusFound)
52+
53+
assertUserDeleted(t, 8)
54+
models.CheckConsistencyFor(t, &models.User{})
55+
}
56+
57+
func TestUserDeleteAccountStillOwnRepos(t *testing.T) {
58+
prepareTestEnv(t)
59+
60+
session := loginUser(t, "user2")
61+
csrf := GetCSRF(t, session, "/user/settings/delete")
62+
urlStr := fmt.Sprintf("/user/settings/delete?password=%s", userPassword)
63+
req := NewRequestWithValues(t, "POST", urlStr, map[string]string{
64+
"_csrf": csrf,
65+
})
66+
session.MakeRequest(t, req, http.StatusFound)
67+
68+
// user should not have been deleted, because the user still owns repos
69+
models.AssertExistsAndLoadBean(t, &models.User{ID: 2})
70+
}

0 commit comments

Comments
 (0)