894d9b2836
Since `modules/context` has to depend on `models` and many other packages, it should be moved from `modules/context` to `services/context` according to design principles. There is no logic code change on this PR, only move packages. - Move `code.gitea.io/gitea/modules/context` to `code.gitea.io/gitea/services/context` - Move `code.gitea.io/gitea/modules/contexttest` to `code.gitea.io/gitea/services/contexttest` because of depending on context - Move `code.gitea.io/gitea/modules/upload` to `code.gitea.io/gitea/services/context/upload` because of depending on context (cherry picked from commit 29f149bd9f517225a3c9f1ca3fb0a7b5325af696) Conflicts: routers/api/packages/alpine/alpine.go routers/api/v1/repo/issue_reaction.go routers/install/install.go routers/web/admin/config.go routers/web/passkey.go routers/web/repo/search.go routers/web/repo/setting/default_branch.go routers/web/user/home.go routers/web/user/profile.go tests/integration/editor_test.go tests/integration/integration_test.go tests/integration/mirror_push_test.go trivial context conflicts also modified all other occurrences in Forgejo specific files
200 lines
4.5 KiB
Go
200 lines
4.5 KiB
Go
// Copyright 2017 The Gitea Authors. All rights reserved.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package admin
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"code.gitea.io/gitea/models/unittest"
|
|
user_model "code.gitea.io/gitea/models/user"
|
|
"code.gitea.io/gitea/modules/setting"
|
|
api "code.gitea.io/gitea/modules/structs"
|
|
"code.gitea.io/gitea/modules/web"
|
|
"code.gitea.io/gitea/services/contexttest"
|
|
"code.gitea.io/gitea/services/forms"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
func TestNewUserPost_MustChangePassword(t *testing.T) {
|
|
unittest.PrepareTestEnv(t)
|
|
ctx, _ := contexttest.MockContext(t, "admin/users/new")
|
|
|
|
u := unittest.AssertExistsAndLoadBean(t, &user_model.User{
|
|
IsAdmin: true,
|
|
ID: 2,
|
|
})
|
|
|
|
ctx.Doer = u
|
|
|
|
username := "gitea"
|
|
email := "gitea@gitea.io"
|
|
|
|
form := forms.AdminCreateUserForm{
|
|
LoginType: "local",
|
|
LoginName: "local",
|
|
UserName: username,
|
|
Email: email,
|
|
Password: "abc123ABC!=$",
|
|
SendNotify: false,
|
|
MustChangePassword: true,
|
|
}
|
|
|
|
web.SetForm(ctx, &form)
|
|
NewUserPost(ctx)
|
|
|
|
assert.NotEmpty(t, ctx.Flash.SuccessMsg)
|
|
|
|
u, err := user_model.GetUserByName(ctx, username)
|
|
|
|
assert.NoError(t, err)
|
|
assert.Equal(t, username, u.Name)
|
|
assert.Equal(t, email, u.Email)
|
|
assert.True(t, u.MustChangePassword)
|
|
}
|
|
|
|
func TestNewUserPost_MustChangePasswordFalse(t *testing.T) {
|
|
unittest.PrepareTestEnv(t)
|
|
ctx, _ := contexttest.MockContext(t, "admin/users/new")
|
|
|
|
u := unittest.AssertExistsAndLoadBean(t, &user_model.User{
|
|
IsAdmin: true,
|
|
ID: 2,
|
|
})
|
|
|
|
ctx.Doer = u
|
|
|
|
username := "gitea"
|
|
email := "gitea@gitea.io"
|
|
|
|
form := forms.AdminCreateUserForm{
|
|
LoginType: "local",
|
|
LoginName: "local",
|
|
UserName: username,
|
|
Email: email,
|
|
Password: "abc123ABC!=$",
|
|
SendNotify: false,
|
|
MustChangePassword: false,
|
|
}
|
|
|
|
web.SetForm(ctx, &form)
|
|
NewUserPost(ctx)
|
|
|
|
assert.NotEmpty(t, ctx.Flash.SuccessMsg)
|
|
|
|
u, err := user_model.GetUserByName(ctx, username)
|
|
|
|
assert.NoError(t, err)
|
|
assert.Equal(t, username, u.Name)
|
|
assert.Equal(t, email, u.Email)
|
|
assert.False(t, u.MustChangePassword)
|
|
}
|
|
|
|
func TestNewUserPost_InvalidEmail(t *testing.T) {
|
|
unittest.PrepareTestEnv(t)
|
|
ctx, _ := contexttest.MockContext(t, "admin/users/new")
|
|
|
|
u := unittest.AssertExistsAndLoadBean(t, &user_model.User{
|
|
IsAdmin: true,
|
|
ID: 2,
|
|
})
|
|
|
|
ctx.Doer = u
|
|
|
|
username := "gitea"
|
|
email := "gitea@gitea.io\r\n"
|
|
|
|
form := forms.AdminCreateUserForm{
|
|
LoginType: "local",
|
|
LoginName: "local",
|
|
UserName: username,
|
|
Email: email,
|
|
Password: "abc123ABC!=$",
|
|
SendNotify: false,
|
|
MustChangePassword: false,
|
|
}
|
|
|
|
web.SetForm(ctx, &form)
|
|
NewUserPost(ctx)
|
|
|
|
assert.NotEmpty(t, ctx.Flash.ErrorMsg)
|
|
}
|
|
|
|
func TestNewUserPost_VisibilityDefaultPublic(t *testing.T) {
|
|
unittest.PrepareTestEnv(t)
|
|
ctx, _ := contexttest.MockContext(t, "admin/users/new")
|
|
|
|
u := unittest.AssertExistsAndLoadBean(t, &user_model.User{
|
|
IsAdmin: true,
|
|
ID: 2,
|
|
})
|
|
|
|
ctx.Doer = u
|
|
|
|
username := "gitea"
|
|
email := "gitea@gitea.io"
|
|
|
|
form := forms.AdminCreateUserForm{
|
|
LoginType: "local",
|
|
LoginName: "local",
|
|
UserName: username,
|
|
Email: email,
|
|
Password: "abc123ABC!=$",
|
|
SendNotify: false,
|
|
MustChangePassword: false,
|
|
}
|
|
|
|
web.SetForm(ctx, &form)
|
|
NewUserPost(ctx)
|
|
|
|
assert.NotEmpty(t, ctx.Flash.SuccessMsg)
|
|
|
|
u, err := user_model.GetUserByName(ctx, username)
|
|
|
|
assert.NoError(t, err)
|
|
assert.Equal(t, username, u.Name)
|
|
assert.Equal(t, email, u.Email)
|
|
// As default user visibility
|
|
assert.Equal(t, setting.Service.DefaultUserVisibilityMode, u.Visibility)
|
|
}
|
|
|
|
func TestNewUserPost_VisibilityPrivate(t *testing.T) {
|
|
unittest.PrepareTestEnv(t)
|
|
ctx, _ := contexttest.MockContext(t, "admin/users/new")
|
|
|
|
u := unittest.AssertExistsAndLoadBean(t, &user_model.User{
|
|
IsAdmin: true,
|
|
ID: 2,
|
|
})
|
|
|
|
ctx.Doer = u
|
|
|
|
username := "gitea"
|
|
email := "gitea@gitea.io"
|
|
|
|
form := forms.AdminCreateUserForm{
|
|
LoginType: "local",
|
|
LoginName: "local",
|
|
UserName: username,
|
|
Email: email,
|
|
Password: "abc123ABC!=$",
|
|
SendNotify: false,
|
|
MustChangePassword: false,
|
|
Visibility: api.VisibleTypePrivate,
|
|
}
|
|
|
|
web.SetForm(ctx, &form)
|
|
NewUserPost(ctx)
|
|
|
|
assert.NotEmpty(t, ctx.Flash.SuccessMsg)
|
|
|
|
u, err := user_model.GetUserByName(ctx, username)
|
|
|
|
assert.NoError(t, err)
|
|
assert.Equal(t, username, u.Name)
|
|
assert.Equal(t, email, u.Email)
|
|
// As default user visibility
|
|
assert.True(t, u.Visibility.IsPrivate())
|
|
}
|