aboutsummaryrefslogtreecommitdiff
path: root/internal/db/org.go
diff options
context:
space:
mode:
authorᴜɴᴋɴᴡᴏɴ <u@gogs.io>2020-04-18 12:07:30 +0800
committerGitHub <noreply@github.com>2020-04-18 12:07:30 +0800
commit9d64d222a8c30db5076a591981ad7cfe672ba6f1 (patch)
treec753eb898a1ead0afbe5b6a7a28b466ab0d2b8ac /internal/db/org.go
parentfa497b16332c24bc4d9e788c64bda94e3c1499a7 (diff)
db: add tests for users (#6116)
* Add new methods * Use Users.Create to replace previous hack * Reduce side effect * Do not clear tables when test failed * test_users_Authenticate * Rename constant * test_users_Create * test_users_GetByEmail * test_users_GetByID * test_users_GetByUsername
Diffstat (limited to 'internal/db/org.go')
-rw-r--r--internal/db/org.go8
1 files changed, 5 insertions, 3 deletions
diff --git a/internal/db/org.go b/internal/db/org.go
index b28d4333..f235928b 100644
--- a/internal/db/org.go
+++ b/internal/db/org.go
@@ -12,6 +12,8 @@ import (
"xorm.io/builder"
"xorm.io/xorm"
+
+ "gogs.io/gogs/internal/errutil"
)
var (
@@ -99,7 +101,7 @@ func (org *User) RemoveOrgRepo(repoID int64) error {
// CreateOrganization creates record of a new organization.
func CreateOrganization(org, owner *User) (err error) {
- if err = IsUsableUsername(org.Name); err != nil {
+ if err = isUsernameAllowed(org.Name); err != nil {
return err
}
@@ -107,7 +109,7 @@ func CreateOrganization(org, owner *User) (err error) {
if err != nil {
return err
} else if isExist {
- return ErrUserAlreadyExist{org.Name}
+ return ErrUserAlreadyExist{args: errutil.Args{"name": org.Name}}
}
org.LowerName = strings.ToLower(org.Name)
@@ -177,7 +179,7 @@ func GetOrgByName(name string) (*User, error) {
}
u := &User{
LowerName: strings.ToLower(name),
- Type: USER_TYPE_ORGANIZATION,
+ Type: UserOrganization,
}
has, err := x.Get(u)
if err != nil {