123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206120712081209121012111212121312141215121612171218121912201221122212231224122512261227122812291230123112321233123412351236123712381239124012411242124312441245124612471248124912501251125212531254125512561257125812591260126112621263126412651266126712681269127012711272127312741275127612771278127912801281128212831284128512861287128812891290129112921293129412951296129712981299130013011302130313041305130613071308130913101311131213131314131513161317131813191320132113221323132413251326132713281329133013311332133313341335133613371338133913401341134213431344134513461347134813491350135113521353135413551356135713581359136013611362136313641365136613671368136913701371137213731374137513761377137813791380138113821383138413851386138713881389139013911392139313941395 |
- // Copyright 2020 The Gogs Authors. All rights reserved.
- // Use of this source code is governed by a MIT-style
- // license that can be found in the LICENSE file.
- package database
- import (
- "context"
- "fmt"
- "os"
- "path/filepath"
- "strings"
- "testing"
- "time"
- "github.com/stretchr/testify/assert"
- "github.com/stretchr/testify/require"
- "gorm.io/gorm"
- "gogs.io/gogs/internal/auth"
- "gogs.io/gogs/internal/conf"
- "gogs.io/gogs/internal/dbutil"
- "gogs.io/gogs/internal/errutil"
- "gogs.io/gogs/internal/osutil"
- "gogs.io/gogs/internal/repoutil"
- "gogs.io/gogs/internal/userutil"
- "gogs.io/gogs/public"
- )
- func TestUser_BeforeCreate(t *testing.T) {
- now := time.Now()
- db := &gorm.DB{
- Config: &gorm.Config{
- SkipDefaultTransaction: true,
- NowFunc: func() time.Time {
- return now
- },
- },
- }
- t.Run("CreatedUnix has been set", func(t *testing.T) {
- user := &User{
- CreatedUnix: 1,
- }
- _ = user.BeforeCreate(db)
- assert.Equal(t, int64(1), user.CreatedUnix)
- assert.Equal(t, int64(0), user.UpdatedUnix)
- })
- t.Run("CreatedUnix has not been set", func(t *testing.T) {
- user := &User{}
- _ = user.BeforeCreate(db)
- assert.Equal(t, db.NowFunc().Unix(), user.CreatedUnix)
- assert.Equal(t, db.NowFunc().Unix(), user.UpdatedUnix)
- })
- }
- func TestUser_AfterFind(t *testing.T) {
- now := time.Now()
- db := &gorm.DB{
- Config: &gorm.Config{
- SkipDefaultTransaction: true,
- NowFunc: func() time.Time {
- return now
- },
- },
- }
- user := &User{
- FullName: "user1<script src=http://localhost:8181/xss.js>",
- CreatedUnix: now.Unix(),
- UpdatedUnix: now.Unix(),
- }
- _ = user.AfterFind(db)
- assert.Equal(t, "user1", user.FullName)
- assert.Equal(t, user.CreatedUnix, user.Created.Unix())
- assert.Equal(t, user.UpdatedUnix, user.Updated.Unix())
- }
- func TestUsers(t *testing.T) {
- if testing.Short() {
- t.Skip()
- }
- t.Parallel()
- ctx := context.Background()
- db := &users{
- DB: newTestDB(t, "users"),
- }
- for _, tc := range []struct {
- name string
- test func(t *testing.T, ctx context.Context, db *users)
- }{
- {"Authenticate", usersAuthenticate},
- {"ChangeUsername", usersChangeUsername},
- {"Count", usersCount},
- {"Create", usersCreate},
- {"DeleteCustomAvatar", usersDeleteCustomAvatar},
- {"DeleteByID", usersDeleteByID},
- {"DeleteInactivated", usersDeleteInactivated},
- {"GetByEmail", usersGetByEmail},
- {"GetByID", usersGetByID},
- {"GetByUsername", usersGetByUsername},
- {"GetByKeyID", usersGetByKeyID},
- {"GetMailableEmailsByUsernames", usersGetMailableEmailsByUsernames},
- {"IsUsernameUsed", usersIsUsernameUsed},
- {"List", usersList},
- {"ListFollowers", usersListFollowers},
- {"ListFollowings", usersListFollowings},
- {"SearchByName", usersSearchByName},
- {"Update", usersUpdate},
- {"UseCustomAvatar", usersUseCustomAvatar},
- {"AddEmail", usersAddEmail},
- {"GetEmail", usersGetEmail},
- {"ListEmails", usersListEmails},
- {"MarkEmailActivated", usersMarkEmailActivated},
- {"MarkEmailPrimary", usersMarkEmailPrimary},
- {"DeleteEmail", usersDeleteEmail},
- {"Follow", usersFollow},
- {"IsFollowing", usersIsFollowing},
- {"Unfollow", usersUnfollow},
- } {
- t.Run(tc.name, func(t *testing.T) {
- t.Cleanup(func() {
- err := clearTables(t, db.DB)
- require.NoError(t, err)
- })
- tc.test(t, ctx, db)
- })
- if t.Failed() {
- break
- }
- }
- }
- func usersAuthenticate(t *testing.T, ctx context.Context, db *users) {
- password := "pa$$word"
- alice, err := db.Create(ctx, "alice", "[email protected]",
- CreateUserOptions{
- Password: password,
- },
- )
- require.NoError(t, err)
- t.Run("user not found", func(t *testing.T) {
- _, err := db.Authenticate(ctx, "bob", password, -1)
- wantErr := auth.ErrBadCredentials{Args: map[string]any{"login": "bob"}}
- assert.Equal(t, wantErr, err)
- })
- t.Run("invalid password", func(t *testing.T) {
- _, err := db.Authenticate(ctx, alice.Name, "bad_password", -1)
- wantErr := auth.ErrBadCredentials{Args: map[string]any{"login": alice.Name, "userID": alice.ID}}
- assert.Equal(t, wantErr, err)
- })
- t.Run("via email and password", func(t *testing.T) {
- user, err := db.Authenticate(ctx, alice.Email, password, -1)
- require.NoError(t, err)
- assert.Equal(t, alice.Name, user.Name)
- })
- t.Run("via username and password", func(t *testing.T) {
- user, err := db.Authenticate(ctx, alice.Name, password, -1)
- require.NoError(t, err)
- assert.Equal(t, alice.Name, user.Name)
- })
- t.Run("login source mismatch", func(t *testing.T) {
- _, err := db.Authenticate(ctx, alice.Email, password, 1)
- gotErr := fmt.Sprintf("%v", err)
- wantErr := ErrLoginSourceMismatch{args: map[string]any{"actual": 0, "expect": 1}}.Error()
- assert.Equal(t, wantErr, gotErr)
- })
- t.Run("via login source", func(t *testing.T) {
- mockLoginSources := NewMockLoginSourcesStore()
- mockLoginSources.GetByIDFunc.SetDefaultHook(func(ctx context.Context, id int64) (*LoginSource, error) {
- mockProvider := NewMockProvider()
- mockProvider.AuthenticateFunc.SetDefaultReturn(&auth.ExternalAccount{}, nil)
- s := &LoginSource{
- IsActived: true,
- Provider: mockProvider,
- }
- return s, nil
- })
- setMockLoginSourcesStore(t, mockLoginSources)
- bob, err := db.Create(ctx, "bob", "[email protected]",
- CreateUserOptions{
- Password: password,
- LoginSource: 1,
- },
- )
- require.NoError(t, err)
- user, err := db.Authenticate(ctx, bob.Email, password, 1)
- require.NoError(t, err)
- assert.Equal(t, bob.Name, user.Name)
- })
- t.Run("new user via login source", func(t *testing.T) {
- mockLoginSources := NewMockLoginSourcesStore()
- mockLoginSources.GetByIDFunc.SetDefaultHook(func(ctx context.Context, id int64) (*LoginSource, error) {
- mockProvider := NewMockProvider()
- mockProvider.AuthenticateFunc.SetDefaultReturn(
- &auth.ExternalAccount{
- Name: "cindy",
- Email: "[email protected]",
- },
- nil,
- )
- s := &LoginSource{
- IsActived: true,
- Provider: mockProvider,
- }
- return s, nil
- })
- setMockLoginSourcesStore(t, mockLoginSources)
- user, err := db.Authenticate(ctx, "cindy", password, 1)
- require.NoError(t, err)
- assert.Equal(t, "cindy", user.Name)
- user, err = db.GetByUsername(ctx, "cindy")
- require.NoError(t, err)
- assert.Equal(t, "[email protected]", user.Email)
- })
- }
- func usersChangeUsername(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(
- ctx,
- "alice",
- "[email protected]",
- CreateUserOptions{
- Activated: true,
- },
- )
- require.NoError(t, err)
- t.Run("name not allowed", func(t *testing.T) {
- err := db.ChangeUsername(ctx, alice.ID, "-")
- wantErr := ErrNameNotAllowed{
- args: errutil.Args{
- "reason": "reserved",
- "name": "-",
- },
- }
- assert.Equal(t, wantErr, err)
- })
- t.Run("name already exists", func(t *testing.T) {
- bob, err := db.Create(
- ctx,
- "bob",
- "[email protected]",
- CreateUserOptions{
- Activated: true,
- },
- )
- require.NoError(t, err)
- err = db.ChangeUsername(ctx, alice.ID, bob.Name)
- wantErr := ErrUserAlreadyExist{
- args: errutil.Args{
- "name": bob.Name,
- },
- }
- assert.Equal(t, wantErr, err)
- })
- tempRepositoryRoot := filepath.Join(os.TempDir(), "usersChangeUsername-tempRepositoryRoot")
- conf.SetMockRepository(
- t,
- conf.RepositoryOpts{
- Root: tempRepositoryRoot,
- },
- )
- err = os.RemoveAll(tempRepositoryRoot)
- require.NoError(t, err)
- defer func() { _ = os.RemoveAll(tempRepositoryRoot) }()
- tempServerAppDataPath := filepath.Join(os.TempDir(), "usersChangeUsername-tempServerAppDataPath")
- conf.SetMockServer(
- t,
- conf.ServerOpts{
- AppDataPath: tempServerAppDataPath,
- },
- )
- err = os.RemoveAll(tempServerAppDataPath)
- require.NoError(t, err)
- defer func() { _ = os.RemoveAll(tempServerAppDataPath) }()
- repo, err := NewReposStore(db.DB).Create(
- ctx,
- alice.ID,
- CreateRepoOptions{
- Name: "test-repo-1",
- },
- )
- require.NoError(t, err)
- // TODO: Use PullRequests.Create to replace SQL hack when the method is available.
- err = db.Exec(`INSERT INTO pull_request (head_user_name) VALUES (?)`, alice.Name).Error
- require.NoError(t, err)
- err = db.Model(&User{}).Where("id = ?", alice.ID).Update("updated_unix", 0).Error
- require.NoError(t, err)
- err = os.MkdirAll(repoutil.UserPath(alice.Name), os.ModePerm)
- require.NoError(t, err)
- err = os.MkdirAll(repoutil.RepositoryLocalPath(repo.ID), os.ModePerm)
- require.NoError(t, err)
- err = os.MkdirAll(repoutil.RepositoryLocalWikiPath(repo.ID), os.ModePerm)
- require.NoError(t, err)
- // Make sure mock data is set up correctly
- // TODO: Use PullRequests.GetByID to replace SQL hack when the method is available.
- var headUserName string
- err = db.Model(&PullRequest{}).Select("head_user_name").Row().Scan(&headUserName)
- require.NoError(t, err)
- assert.Equal(t, headUserName, alice.Name)
- var updatedUnix int64
- err = db.Model(&User{}).Select("updated_unix").Where("id = ?", alice.ID).Row().Scan(&updatedUnix)
- require.NoError(t, err)
- assert.Equal(t, int64(0), updatedUnix)
- assert.True(t, osutil.IsExist(repoutil.UserPath(alice.Name)))
- assert.True(t, osutil.IsExist(repoutil.RepositoryLocalPath(repo.ID)))
- assert.True(t, osutil.IsExist(repoutil.RepositoryLocalWikiPath(repo.ID)))
- const newUsername = "alice-new"
- err = db.ChangeUsername(ctx, alice.ID, newUsername)
- require.NoError(t, err)
- // TODO: Use PullRequests.GetByID to replace SQL hack when the method is available.
- err = db.Model(&PullRequest{}).Select("head_user_name").Row().Scan(&headUserName)
- require.NoError(t, err)
- assert.Equal(t, headUserName, newUsername)
- assert.True(t, osutil.IsExist(repoutil.UserPath(newUsername)))
- assert.False(t, osutil.IsExist(repoutil.UserPath(alice.Name)))
- assert.False(t, osutil.IsExist(repoutil.RepositoryLocalPath(repo.ID)))
- assert.False(t, osutil.IsExist(repoutil.RepositoryLocalWikiPath(repo.ID)))
- alice, err = db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.Equal(t, newUsername, alice.Name)
- assert.Equal(t, db.NowFunc().Unix(), alice.UpdatedUnix)
- // Change the cases of the username should just be fine
- err = db.ChangeUsername(ctx, alice.ID, strings.ToUpper(newUsername))
- require.NoError(t, err)
- alice, err = db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.Equal(t, strings.ToUpper(newUsername), alice.Name)
- }
- func usersCount(t *testing.T, ctx context.Context, db *users) {
- // Has no user initially
- got := db.Count(ctx)
- assert.Equal(t, int64(0), got)
- _, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- got = db.Count(ctx)
- assert.Equal(t, int64(1), got)
- // Create an organization shouldn't count
- // TODO: Use Orgs.Create to replace SQL hack when the method is available.
- org1, err := db.Create(ctx, "org1", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.Exec(
- dbutil.Quote("UPDATE %s SET type = ? WHERE id = ?", "user"),
- UserTypeOrganization, org1.ID,
- ).Error
- require.NoError(t, err)
- got = db.Count(ctx)
- assert.Equal(t, int64(1), got)
- }
- func usersCreate(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(
- ctx,
- "alice",
- "[email protected]",
- CreateUserOptions{
- Activated: true,
- },
- )
- require.NoError(t, err)
- t.Run("name not allowed", func(t *testing.T) {
- _, err := db.Create(ctx, "-", "", CreateUserOptions{})
- wantErr := ErrNameNotAllowed{
- args: errutil.Args{
- "reason": "reserved",
- "name": "-",
- },
- }
- assert.Equal(t, wantErr, err)
- })
- t.Run("name already exists", func(t *testing.T) {
- _, err := db.Create(ctx, alice.Name, "", CreateUserOptions{})
- wantErr := ErrUserAlreadyExist{
- args: errutil.Args{
- "name": alice.Name,
- },
- }
- assert.Equal(t, wantErr, err)
- })
- t.Run("email already exists", func(t *testing.T) {
- _, err := db.Create(ctx, "bob", alice.Email, CreateUserOptions{})
- wantErr := ErrEmailAlreadyUsed{
- args: errutil.Args{
- "email": alice.Email,
- },
- }
- assert.Equal(t, wantErr, err)
- })
- user, err := db.GetByUsername(ctx, alice.Name)
- require.NoError(t, err)
- assert.Equal(t, db.NowFunc().Format(time.RFC3339), user.Created.UTC().Format(time.RFC3339))
- assert.Equal(t, db.NowFunc().Format(time.RFC3339), user.Updated.UTC().Format(time.RFC3339))
- }
- func usersDeleteCustomAvatar(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- avatar, err := public.Files.ReadFile("img/avatar_default.png")
- require.NoError(t, err)
- avatarPath := userutil.CustomAvatarPath(alice.ID)
- _ = os.Remove(avatarPath)
- defer func() { _ = os.Remove(avatarPath) }()
- err = db.UseCustomAvatar(ctx, alice.ID, avatar)
- require.NoError(t, err)
- // Make sure avatar is saved and the user flag is updated.
- got := osutil.IsFile(avatarPath)
- assert.True(t, got)
- alice, err = db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.True(t, alice.UseCustomAvatar)
- // Delete avatar should remove the file and revert the user flag.
- err = db.DeleteCustomAvatar(ctx, alice.ID)
- require.NoError(t, err)
- got = osutil.IsFile(avatarPath)
- assert.False(t, got)
- alice, err = db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.False(t, alice.UseCustomAvatar)
- }
- func usersDeleteByID(t *testing.T, ctx context.Context, db *users) {
- reposStore := NewReposStore(db.DB)
- t.Run("user still has repository ownership", func(t *testing.T) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- _, err = reposStore.Create(ctx, alice.ID, CreateRepoOptions{Name: "repo1"})
- require.NoError(t, err)
- err = db.DeleteByID(ctx, alice.ID, false)
- wantErr := ErrUserOwnRepos{errutil.Args{"userID": alice.ID}}
- assert.Equal(t, wantErr, err)
- })
- t.Run("user still has organization membership", func(t *testing.T) {
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- // TODO: Use Orgs.Create to replace SQL hack when the method is available.
- org1, err := db.Create(ctx, "org1", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.Exec(
- dbutil.Quote("UPDATE %s SET type = ? WHERE id IN (?)", "user"),
- UserTypeOrganization, org1.ID,
- ).Error
- require.NoError(t, err)
- // TODO: Use Orgs.Join to replace SQL hack when the method is available.
- err = db.Exec(`INSERT INTO org_user (uid, org_id) VALUES (?, ?)`, bob.ID, org1.ID).Error
- require.NoError(t, err)
- err = db.DeleteByID(ctx, bob.ID, false)
- wantErr := ErrUserHasOrgs{errutil.Args{"userID": bob.ID}}
- assert.Equal(t, wantErr, err)
- })
- cindy, err := db.Create(ctx, "cindy", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- frank, err := db.Create(ctx, "frank", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- repo2, err := reposStore.Create(ctx, cindy.ID, CreateRepoOptions{Name: "repo2"})
- require.NoError(t, err)
- testUser, err := db.Create(ctx, "testUser", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- // Mock watches, stars and follows
- err = reposStore.Watch(ctx, testUser.ID, repo2.ID)
- require.NoError(t, err)
- err = reposStore.Star(ctx, testUser.ID, repo2.ID)
- require.NoError(t, err)
- err = db.Follow(ctx, testUser.ID, cindy.ID)
- require.NoError(t, err)
- err = db.Follow(ctx, frank.ID, testUser.ID)
- require.NoError(t, err)
- // Mock "authorized_keys" file
- // TODO: Use PublicKeys.Add to replace SQL hack when the method is available.
- publicKey := &PublicKey{
- OwnerID: testUser.ID,
- Name: "test-key",
- Fingerprint: "12:f8:7e:78:61:b4:bf:e2:de:24:15:96:4e:d4:72:53",
- Content: "test-key-content",
- }
- err = db.DB.Create(publicKey).Error
- require.NoError(t, err)
- tempSSHRootPath := filepath.Join(os.TempDir(), "usersDeleteByID-tempSSHRootPath")
- conf.SetMockSSH(t, conf.SSHOpts{RootPath: tempSSHRootPath})
- err = NewPublicKeysStore(db.DB).RewriteAuthorizedKeys()
- require.NoError(t, err)
- // Mock issue assignee
- // TODO: Use Issues.Assign to replace SQL hack when the method is available.
- issue := &Issue{
- RepoID: repo2.ID,
- Index: 1,
- PosterID: cindy.ID,
- Title: "test-issue",
- AssigneeID: testUser.ID,
- }
- err = db.DB.Create(issue).Error
- require.NoError(t, err)
- // Mock random entries in related tables
- for _, table := range []any{
- &AccessToken{UserID: testUser.ID},
- &Collaboration{UserID: testUser.ID},
- &Access{UserID: testUser.ID},
- &Action{UserID: testUser.ID},
- &IssueUser{UserID: testUser.ID},
- &EmailAddress{UserID: testUser.ID},
- } {
- err = db.DB.Create(table).Error
- require.NoError(t, err, "table for %T", table)
- }
- // Mock user directory
- tempRepositoryRoot := filepath.Join(os.TempDir(), "usersDeleteByID-tempRepositoryRoot")
- conf.SetMockRepository(t, conf.RepositoryOpts{Root: tempRepositoryRoot})
- tempUserPath := repoutil.UserPath(testUser.Name)
- err = os.MkdirAll(tempUserPath, os.ModePerm)
- require.NoError(t, err)
- // Mock user custom avatar
- tempPictureAvatarUploadPath := filepath.Join(os.TempDir(), "usersDeleteByID-tempPictureAvatarUploadPath")
- conf.SetMockPicture(t, conf.PictureOpts{AvatarUploadPath: tempPictureAvatarUploadPath})
- err = os.MkdirAll(tempPictureAvatarUploadPath, os.ModePerm)
- require.NoError(t, err)
- tempCustomAvatarPath := userutil.CustomAvatarPath(testUser.ID)
- err = os.WriteFile(tempCustomAvatarPath, []byte("test"), 0600)
- require.NoError(t, err)
- // Verify mock data
- repo2, err = reposStore.GetByID(ctx, repo2.ID)
- require.NoError(t, err)
- assert.Equal(t, 2, repo2.NumWatches) // The owner is watching the repo by default.
- assert.Equal(t, 1, repo2.NumStars)
- cindy, err = db.GetByID(ctx, cindy.ID)
- require.NoError(t, err)
- assert.Equal(t, 1, cindy.NumFollowers)
- frank, err = db.GetByID(ctx, frank.ID)
- require.NoError(t, err)
- assert.Equal(t, 1, frank.NumFollowing)
- authorizedKeys, err := os.ReadFile(authorizedKeysPath())
- require.NoError(t, err)
- assert.Contains(t, string(authorizedKeys), fmt.Sprintf("key-%d", publicKey.ID))
- assert.Contains(t, string(authorizedKeys), publicKey.Content)
- // TODO: Use Issues.GetByID to replace SQL hack when the method is available.
- err = db.DB.First(issue, issue.ID).Error
- require.NoError(t, err)
- assert.Equal(t, testUser.ID, issue.AssigneeID)
- relatedTables := []any{
- &Watch{UserID: testUser.ID},
- &Star{UserID: testUser.ID},
- &Follow{UserID: testUser.ID},
- &PublicKey{OwnerID: testUser.ID},
- &AccessToken{UserID: testUser.ID},
- &Collaboration{UserID: testUser.ID},
- &Access{UserID: testUser.ID},
- &Action{UserID: testUser.ID},
- &IssueUser{UserID: testUser.ID},
- &EmailAddress{UserID: testUser.ID},
- }
- for _, table := range relatedTables {
- var count int64
- err = db.DB.Model(table).Where(table).Count(&count).Error
- require.NoError(t, err, "table for %T", table)
- assert.NotZero(t, count, "table for %T", table)
- }
- assert.True(t, osutil.IsExist(tempUserPath))
- assert.True(t, osutil.IsExist(tempCustomAvatarPath))
- // Pull the trigger
- err = db.DeleteByID(ctx, testUser.ID, false)
- require.NoError(t, err)
- // Verify after-the-fact data
- repo2, err = reposStore.GetByID(ctx, repo2.ID)
- require.NoError(t, err)
- assert.Equal(t, 1, repo2.NumWatches) // The owner is watching the repo by default.
- assert.Equal(t, 0, repo2.NumStars)
- cindy, err = db.GetByID(ctx, cindy.ID)
- require.NoError(t, err)
- assert.Equal(t, 0, cindy.NumFollowers)
- frank, err = db.GetByID(ctx, frank.ID)
- require.NoError(t, err)
- assert.Equal(t, 0, frank.NumFollowing)
- authorizedKeys, err = os.ReadFile(authorizedKeysPath())
- require.NoError(t, err)
- assert.Empty(t, authorizedKeys)
- // TODO: Use Issues.GetByID to replace SQL hack when the method is available.
- err = db.DB.First(issue, issue.ID).Error
- require.NoError(t, err)
- assert.Equal(t, int64(0), issue.AssigneeID)
- for _, table := range []any{
- &Watch{UserID: testUser.ID},
- &Star{UserID: testUser.ID},
- &Follow{UserID: testUser.ID},
- &PublicKey{OwnerID: testUser.ID},
- &AccessToken{UserID: testUser.ID},
- &Collaboration{UserID: testUser.ID},
- &Access{UserID: testUser.ID},
- &Action{UserID: testUser.ID},
- &IssueUser{UserID: testUser.ID},
- &EmailAddress{UserID: testUser.ID},
- } {
- var count int64
- err = db.DB.Model(table).Where(table).Count(&count).Error
- require.NoError(t, err, "table for %T", table)
- assert.Equal(t, int64(0), count, "table for %T", table)
- }
- assert.False(t, osutil.IsExist(tempUserPath))
- assert.False(t, osutil.IsExist(tempCustomAvatarPath))
- _, err = db.GetByID(ctx, testUser.ID)
- wantErr := ErrUserNotExist{errutil.Args{"userID": testUser.ID}}
- assert.Equal(t, wantErr, err)
- }
- func usersDeleteInactivated(t *testing.T, ctx context.Context, db *users) {
- // User with repository ownership should be skipped
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- reposStore := NewReposStore(db.DB)
- _, err = reposStore.Create(ctx, alice.ID, CreateRepoOptions{Name: "repo1"})
- require.NoError(t, err)
- // User with organization membership should be skipped
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- // TODO: Use Orgs.Create to replace SQL hack when the method is available.
- org1, err := db.Create(ctx, "org1", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.Exec(
- dbutil.Quote("UPDATE %s SET type = ? WHERE id IN (?)", "user"),
- UserTypeOrganization, org1.ID,
- ).Error
- require.NoError(t, err)
- // TODO: Use Orgs.Join to replace SQL hack when the method is available.
- err = db.Exec(`INSERT INTO org_user (uid, org_id) VALUES (?, ?)`, bob.ID, org1.ID).Error
- require.NoError(t, err)
- // User activated state should be skipped
- _, err = db.Create(ctx, "cindy", "[email protected]", CreateUserOptions{Activated: true})
- require.NoError(t, err)
- // User meant to be deleted
- david, err := db.Create(ctx, "david", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- tempSSHRootPath := filepath.Join(os.TempDir(), "usersDeleteInactivated-tempSSHRootPath")
- conf.SetMockSSH(t, conf.SSHOpts{RootPath: tempSSHRootPath})
- err = db.DeleteInactivated()
- require.NoError(t, err)
- _, err = db.GetByID(ctx, david.ID)
- wantErr := ErrUserNotExist{errutil.Args{"userID": david.ID}}
- assert.Equal(t, wantErr, err)
- users, err := db.List(ctx, 1, 10)
- require.NoError(t, err)
- require.Len(t, users, 3)
- }
- func usersGetByEmail(t *testing.T, ctx context.Context, db *users) {
- t.Run("empty email", func(t *testing.T) {
- _, err := db.GetByEmail(ctx, "")
- wantErr := ErrUserNotExist{args: errutil.Args{"email": ""}}
- assert.Equal(t, wantErr, err)
- })
- t.Run("ignore organization", func(t *testing.T) {
- // TODO: Use Orgs.Create to replace SQL hack when the method is available.
- org, err := db.Create(ctx, "gogs", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.Model(&User{}).Where("id", org.ID).UpdateColumn("type", UserTypeOrganization).Error
- require.NoError(t, err)
- _, err = db.GetByEmail(ctx, org.Email)
- wantErr := ErrUserNotExist{args: errutil.Args{"email": org.Email}}
- assert.Equal(t, wantErr, err)
- })
- t.Run("by primary email", func(t *testing.T) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- _, err = db.GetByEmail(ctx, alice.Email)
- wantErr := ErrUserNotExist{args: errutil.Args{"email": alice.Email}}
- assert.Equal(t, wantErr, err)
- // Mark user as activated
- // TODO: Use UserEmails.Verify to replace SQL hack when the method is available.
- err = db.Model(&User{}).Where("id", alice.ID).UpdateColumn("is_active", true).Error
- require.NoError(t, err)
- user, err := db.GetByEmail(ctx, alice.Email)
- require.NoError(t, err)
- assert.Equal(t, alice.Name, user.Name)
- })
- t.Run("by secondary email", func(t *testing.T) {
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- // TODO: Use UserEmails.Create to replace SQL hack when the method is available.
- email2 := "[email protected]"
- err = db.Exec(`INSERT INTO email_address (uid, email) VALUES (?, ?)`, bob.ID, email2).Error
- require.NoError(t, err)
- _, err = db.GetByEmail(ctx, email2)
- wantErr := ErrUserNotExist{args: errutil.Args{"email": email2}}
- assert.Equal(t, wantErr, err)
- // TODO: Use UserEmails.Verify to replace SQL hack when the method is available.
- err = db.Exec(`UPDATE email_address SET is_activated = ? WHERE email = ?`, true, email2).Error
- require.NoError(t, err)
- user, err := db.GetByEmail(ctx, email2)
- require.NoError(t, err)
- assert.Equal(t, bob.Name, user.Name)
- })
- }
- func usersGetByID(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- user, err := db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.Equal(t, alice.Name, user.Name)
- _, err = db.GetByID(ctx, 404)
- wantErr := ErrUserNotExist{args: errutil.Args{"userID": int64(404)}}
- assert.Equal(t, wantErr, err)
- }
- func usersGetByUsername(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- user, err := db.GetByUsername(ctx, alice.Name)
- require.NoError(t, err)
- assert.Equal(t, alice.Name, user.Name)
- _, err = db.GetByUsername(ctx, "bad_username")
- wantErr := ErrUserNotExist{args: errutil.Args{"name": "bad_username"}}
- assert.Equal(t, wantErr, err)
- }
- func usersGetByKeyID(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- // TODO: Use PublicKeys.Create to replace SQL hack when the method is available.
- publicKey := &PublicKey{
- OwnerID: alice.ID,
- Name: "test-key",
- Fingerprint: "12:f8:7e:78:61:b4:bf:e2:de:24:15:96:4e:d4:72:53",
- Content: "test-key-content",
- CreatedUnix: db.NowFunc().Unix(),
- UpdatedUnix: db.NowFunc().Unix(),
- }
- err = db.WithContext(ctx).Create(publicKey).Error
- require.NoError(t, err)
- user, err := db.GetByKeyID(ctx, publicKey.ID)
- require.NoError(t, err)
- assert.Equal(t, alice.Name, user.Name)
- _, err = db.GetByKeyID(ctx, publicKey.ID+1)
- wantErr := ErrUserNotExist{args: errutil.Args{"keyID": publicKey.ID + 1}}
- assert.Equal(t, wantErr, err)
- }
- func usersGetMailableEmailsByUsernames(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{Activated: true})
- require.NoError(t, err)
- _, err = db.Create(ctx, "cindy", "[email protected]", CreateUserOptions{Activated: true})
- require.NoError(t, err)
- got, err := db.GetMailableEmailsByUsernames(ctx, []string{alice.Name, bob.Name, "ignore-non-exist"})
- require.NoError(t, err)
- want := []string{bob.Email}
- assert.Equal(t, want, got)
- }
- func usersIsUsernameUsed(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- tests := []struct {
- name string
- username string
- excludeUserID int64
- want bool
- }{
- {
- name: "no change",
- username: alice.Name,
- excludeUserID: alice.ID,
- want: false,
- },
- {
- name: "change case",
- username: strings.ToUpper(alice.Name),
- excludeUserID: alice.ID,
- want: false,
- },
- {
- name: "not used",
- username: "bob",
- excludeUserID: alice.ID,
- want: false,
- },
- {
- name: "not used when not excluded",
- username: "bob",
- excludeUserID: 0,
- want: false,
- },
- {
- name: "used when not excluded",
- username: alice.Name,
- excludeUserID: 0,
- want: true,
- },
- }
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- got := db.IsUsernameUsed(ctx, test.username, test.excludeUserID)
- assert.Equal(t, test.want, got)
- })
- }
- }
- func usersList(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- // Create an organization shouldn't count
- // TODO: Use Orgs.Create to replace SQL hack when the method is available.
- org1, err := db.Create(ctx, "org1", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.Exec(
- dbutil.Quote("UPDATE %s SET type = ? WHERE id = ?", "user"),
- UserTypeOrganization, org1.ID,
- ).Error
- require.NoError(t, err)
- got, err := db.List(ctx, 1, 1)
- require.NoError(t, err)
- require.Len(t, got, 1)
- assert.Equal(t, alice.ID, got[0].ID)
- got, err = db.List(ctx, 2, 1)
- require.NoError(t, err)
- require.Len(t, got, 1)
- assert.Equal(t, bob.ID, got[0].ID)
- got, err = db.List(ctx, 1, 3)
- require.NoError(t, err)
- require.Len(t, got, 2)
- assert.Equal(t, alice.ID, got[0].ID)
- assert.Equal(t, bob.ID, got[1].ID)
- }
- func usersListFollowers(t *testing.T, ctx context.Context, db *users) {
- john, err := db.Create(ctx, "john", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- got, err := db.ListFollowers(ctx, john.ID, 1, 1)
- require.NoError(t, err)
- assert.Empty(t, got)
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.Follow(ctx, alice.ID, john.ID)
- require.NoError(t, err)
- err = db.Follow(ctx, bob.ID, john.ID)
- require.NoError(t, err)
- // First page only has bob
- got, err = db.ListFollowers(ctx, john.ID, 1, 1)
- require.NoError(t, err)
- require.Len(t, got, 1)
- assert.Equal(t, bob.ID, got[0].ID)
- // Second page only has alice
- got, err = db.ListFollowers(ctx, john.ID, 2, 1)
- require.NoError(t, err)
- require.Len(t, got, 1)
- assert.Equal(t, alice.ID, got[0].ID)
- }
- func usersListFollowings(t *testing.T, ctx context.Context, db *users) {
- john, err := db.Create(ctx, "john", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- got, err := db.ListFollowers(ctx, john.ID, 1, 1)
- require.NoError(t, err)
- assert.Empty(t, got)
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.Follow(ctx, john.ID, alice.ID)
- require.NoError(t, err)
- err = db.Follow(ctx, john.ID, bob.ID)
- require.NoError(t, err)
- // First page only has bob
- got, err = db.ListFollowings(ctx, john.ID, 1, 1)
- require.NoError(t, err)
- require.Len(t, got, 1)
- assert.Equal(t, bob.ID, got[0].ID)
- // Second page only has alice
- got, err = db.ListFollowings(ctx, john.ID, 2, 1)
- require.NoError(t, err)
- require.Len(t, got, 1)
- assert.Equal(t, alice.ID, got[0].ID)
- }
- func usersSearchByName(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{FullName: "Alice Jordan"})
- require.NoError(t, err)
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{FullName: "Bob Jordan"})
- require.NoError(t, err)
- t.Run("search for username alice", func(t *testing.T) {
- users, count, err := db.SearchByName(ctx, "Li", 1, 1, "")
- require.NoError(t, err)
- require.Len(t, users, int(count))
- assert.Equal(t, int64(1), count)
- assert.Equal(t, alice.ID, users[0].ID)
- })
- t.Run("search for username bob", func(t *testing.T) {
- users, count, err := db.SearchByName(ctx, "oB", 1, 1, "")
- require.NoError(t, err)
- require.Len(t, users, int(count))
- assert.Equal(t, int64(1), count)
- assert.Equal(t, bob.ID, users[0].ID)
- })
- t.Run("search for full name jordan", func(t *testing.T) {
- users, count, err := db.SearchByName(ctx, "Jo", 1, 10, "")
- require.NoError(t, err)
- require.Len(t, users, int(count))
- assert.Equal(t, int64(2), count)
- })
- t.Run("search for full name jordan ORDER BY id DESC LIMIT 1", func(t *testing.T) {
- users, count, err := db.SearchByName(ctx, "Jo", 1, 1, "id DESC")
- require.NoError(t, err)
- require.Len(t, users, 1)
- assert.Equal(t, int64(2), count)
- assert.Equal(t, bob.ID, users[0].ID)
- })
- }
- func usersUpdate(t *testing.T, ctx context.Context, db *users) {
- const oldPassword = "Password"
- alice, err := db.Create(
- ctx,
- "alice",
- "[email protected]",
- CreateUserOptions{
- FullName: "FullName",
- Password: oldPassword,
- LoginSource: 9,
- LoginName: "LoginName",
- Location: "Location",
- Website: "Website",
- Activated: false,
- Admin: false,
- },
- )
- require.NoError(t, err)
- t.Run("update password", func(t *testing.T) {
- got := userutil.ValidatePassword(alice.Password, alice.Salt, oldPassword)
- require.True(t, got)
- newPassword := "NewPassword"
- err = db.Update(ctx, alice.ID, UpdateUserOptions{Password: &newPassword})
- require.NoError(t, err)
- alice, err = db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- got = userutil.ValidatePassword(alice.Password, alice.Salt, oldPassword)
- assert.False(t, got, "Old password should stop working")
- got = userutil.ValidatePassword(alice.Password, alice.Salt, newPassword)
- assert.True(t, got, "New password should work")
- })
- t.Run("update email but already used", func(t *testing.T) {
- bob, err := db.Create(
- ctx,
- "bob",
- "[email protected]",
- CreateUserOptions{
- Activated: true,
- },
- )
- require.NoError(t, err)
- got := db.Update(ctx, alice.ID, UpdateUserOptions{Email: &bob.Email})
- want := ErrEmailAlreadyUsed{args: errutil.Args{"email": bob.Email}}
- assert.Equal(t, want, got)
- })
- loginSource := int64(1)
- maxRepoCreation := 99
- lastRepoVisibility := true
- overLimitStr := strings.Repeat("a", 2050)
- opts := UpdateUserOptions{
- LoginSource: &loginSource,
- LoginName: &alice.Name,
- FullName: &overLimitStr,
- Website: &overLimitStr,
- Location: &overLimitStr,
- Description: &overLimitStr,
- MaxRepoCreation: &maxRepoCreation,
- LastRepoVisibility: &lastRepoVisibility,
- IsActivated: &lastRepoVisibility,
- IsAdmin: &lastRepoVisibility,
- AllowGitHook: &lastRepoVisibility,
- AllowImportLocal: &lastRepoVisibility,
- ProhibitLogin: &lastRepoVisibility,
- Avatar: &overLimitStr,
- AvatarEmail: &overLimitStr,
- }
- err = db.Update(ctx, alice.ID, opts)
- require.NoError(t, err)
- alice, err = db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assertValues := func() {
- assert.Equal(t, loginSource, alice.LoginSource)
- assert.Equal(t, alice.Name, alice.LoginName)
- wantStr255 := strings.Repeat("a", 255)
- assert.Equal(t, wantStr255, alice.FullName)
- assert.Equal(t, wantStr255, alice.Website)
- assert.Equal(t, wantStr255, alice.Location)
- assert.Equal(t, wantStr255, alice.Description)
- assert.Equal(t, maxRepoCreation, alice.MaxRepoCreation)
- assert.Equal(t, lastRepoVisibility, alice.LastRepoVisibility)
- assert.Equal(t, lastRepoVisibility, alice.IsActive)
- assert.Equal(t, lastRepoVisibility, alice.IsAdmin)
- assert.Equal(t, lastRepoVisibility, alice.AllowGitHook)
- assert.Equal(t, lastRepoVisibility, alice.AllowImportLocal)
- assert.Equal(t, lastRepoVisibility, alice.ProhibitLogin)
- wantStr2048 := strings.Repeat("a", 2048)
- assert.Equal(t, wantStr2048, alice.Avatar)
- assert.Equal(t, wantStr255, alice.AvatarEmail)
- }
- assertValues()
- // Test ignored values
- err = db.Update(ctx, alice.ID, UpdateUserOptions{})
- require.NoError(t, err)
- alice, err = db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assertValues()
- }
- func usersUseCustomAvatar(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- avatar, err := public.Files.ReadFile("img/avatar_default.png")
- require.NoError(t, err)
- avatarPath := userutil.CustomAvatarPath(alice.ID)
- _ = os.Remove(avatarPath)
- defer func() { _ = os.Remove(avatarPath) }()
- err = db.UseCustomAvatar(ctx, alice.ID, avatar)
- require.NoError(t, err)
- // Make sure avatar is saved and the user flag is updated.
- got := osutil.IsFile(avatarPath)
- assert.True(t, got)
- alice, err = db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.True(t, alice.UseCustomAvatar)
- }
- func TestIsUsernameAllowed(t *testing.T) {
- for name := range reservedUsernames {
- t.Run(name, func(t *testing.T) {
- assert.True(t, IsErrNameNotAllowed(isUsernameAllowed(name)))
- })
- }
- for _, pattern := range reservedUsernamePatterns {
- t.Run(pattern, func(t *testing.T) {
- username := strings.ReplaceAll(pattern, "*", "alice")
- assert.True(t, IsErrNameNotAllowed(isUsernameAllowed(username)))
- })
- }
- }
- func usersAddEmail(t *testing.T, ctx context.Context, db *users) {
- t.Run("multiple users can add the same unverified email", func(t *testing.T) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.AddEmail(ctx, alice.ID+1, "[email protected]", false)
- require.NoError(t, err)
- })
- t.Run("only one user can add the same verified email", func(t *testing.T) {
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{Activated: true})
- require.NoError(t, err)
- got := db.AddEmail(ctx, bob.ID+1, "[email protected]", true)
- want := ErrEmailAlreadyUsed{args: errutil.Args{"email": "[email protected]"}}
- require.Equal(t, want, got)
- })
- }
- func usersGetEmail(t *testing.T, ctx context.Context, db *users) {
- const testUserID = 1
- const testEmail = "[email protected]"
- _, err := db.GetEmail(ctx, testUserID, testEmail, false)
- wantErr := ErrEmailNotExist{
- args: errutil.Args{
- "email": testEmail,
- },
- }
- assert.Equal(t, wantErr, err)
- err = db.AddEmail(ctx, testUserID, testEmail, false)
- require.NoError(t, err)
- got, err := db.GetEmail(ctx, testUserID, testEmail, false)
- require.NoError(t, err)
- assert.Equal(t, testEmail, got.Email)
- // Should not return if we ask for a different user
- _, err = db.GetEmail(ctx, testUserID+1, testEmail, false)
- assert.Equal(t, wantErr, err)
- // Should not return if we only want activated emails
- _, err = db.GetEmail(ctx, testUserID, testEmail, true)
- assert.Equal(t, wantErr, err)
- err = db.MarkEmailActivated(ctx, testUserID, testEmail)
- require.NoError(t, err)
- got, err = db.GetEmail(ctx, testUserID, testEmail, true)
- require.NoError(t, err)
- assert.Equal(t, testEmail, got.Email)
- }
- func usersListEmails(t *testing.T, ctx context.Context, db *users) {
- t.Run("list emails with primary email", func(t *testing.T) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.AddEmail(ctx, alice.ID, "[email protected]", true)
- require.NoError(t, err)
- err = db.MarkEmailPrimary(ctx, alice.ID, "[email protected]")
- require.NoError(t, err)
- emails, err := db.ListEmails(ctx, alice.ID)
- require.NoError(t, err)
- got := make([]string, 0, len(emails))
- for _, email := range emails {
- got = append(got, email.Email)
- }
- want := []string{"[email protected]", "[email protected]"}
- assert.Equal(t, want, got)
- })
- t.Run("list emails without primary email", func(t *testing.T) {
- bob, err := db.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.AddEmail(ctx, bob.ID, "[email protected]", false)
- require.NoError(t, err)
- emails, err := db.ListEmails(ctx, bob.ID)
- require.NoError(t, err)
- got := make([]string, 0, len(emails))
- for _, email := range emails {
- got = append(got, email.Email)
- }
- want := []string{"[email protected]", "[email protected]"}
- assert.Equal(t, want, got)
- })
- }
- func usersMarkEmailActivated(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.AddEmail(ctx, alice.ID, "[email protected]", false)
- require.NoError(t, err)
- err = db.MarkEmailActivated(ctx, alice.ID, "[email protected]")
- require.NoError(t, err)
- gotEmail, err := db.GetEmail(ctx, alice.ID, "[email protected]", true)
- require.NoError(t, err)
- assert.True(t, gotEmail.IsActivated)
- gotAlice, err := db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.NotEqual(t, alice.Rands, gotAlice.Rands)
- }
- func usersMarkEmailPrimary(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.AddEmail(ctx, alice.ID, "[email protected]", false)
- require.NoError(t, err)
- // Should fail because email not verified
- gotError := db.MarkEmailPrimary(ctx, alice.ID, "[email protected]")
- wantError := ErrEmailNotVerified{args: errutil.Args{"email": "[email protected]"}}
- assert.Equal(t, wantError, gotError)
- // Mark email as verified and should succeed
- err = db.MarkEmailActivated(ctx, alice.ID, "[email protected]")
- require.NoError(t, err)
- err = db.MarkEmailPrimary(ctx, alice.ID, "[email protected]")
- require.NoError(t, err)
- gotAlice, err := db.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.Equal(t, "[email protected]", gotAlice.Email)
- // Former primary email should be preserved
- gotEmail, err := db.GetEmail(ctx, alice.ID, "[email protected]", false)
- require.NoError(t, err)
- assert.False(t, gotEmail.IsActivated)
- }
- func usersDeleteEmail(t *testing.T, ctx context.Context, db *users) {
- alice, err := db.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.AddEmail(ctx, alice.ID, "[email protected]", false)
- require.NoError(t, err)
- _, err = db.GetEmail(ctx, alice.ID, "[email protected]", false)
- require.NoError(t, err)
- err = db.DeleteEmail(ctx, alice.ID, "[email protected]")
- require.NoError(t, err)
- _, got := db.GetEmail(ctx, alice.ID, "[email protected]", false)
- want := ErrEmailNotExist{args: errutil.Args{"email": "[email protected]"}}
- require.Equal(t, want, got)
- }
- func usersFollow(t *testing.T, ctx context.Context, db *users) {
- usersStore := NewUsersStore(db.DB)
- alice, err := usersStore.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- bob, err := usersStore.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.Follow(ctx, alice.ID, bob.ID)
- require.NoError(t, err)
- // It is OK to follow multiple times and just be noop.
- err = db.Follow(ctx, alice.ID, bob.ID)
- require.NoError(t, err)
- alice, err = usersStore.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.Equal(t, 1, alice.NumFollowing)
- bob, err = usersStore.GetByID(ctx, bob.ID)
- require.NoError(t, err)
- assert.Equal(t, 1, bob.NumFollowers)
- }
- func usersIsFollowing(t *testing.T, ctx context.Context, db *users) {
- usersStore := NewUsersStore(db.DB)
- alice, err := usersStore.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- bob, err := usersStore.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- got := db.IsFollowing(ctx, alice.ID, bob.ID)
- assert.False(t, got)
- err = db.Follow(ctx, alice.ID, bob.ID)
- require.NoError(t, err)
- got = db.IsFollowing(ctx, alice.ID, bob.ID)
- assert.True(t, got)
- err = db.Unfollow(ctx, alice.ID, bob.ID)
- require.NoError(t, err)
- got = db.IsFollowing(ctx, alice.ID, bob.ID)
- assert.False(t, got)
- }
- func usersUnfollow(t *testing.T, ctx context.Context, db *users) {
- usersStore := NewUsersStore(db.DB)
- alice, err := usersStore.Create(ctx, "alice", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- bob, err := usersStore.Create(ctx, "bob", "[email protected]", CreateUserOptions{})
- require.NoError(t, err)
- err = db.Follow(ctx, alice.ID, bob.ID)
- require.NoError(t, err)
- // It is OK to unfollow multiple times and just be noop.
- err = db.Unfollow(ctx, alice.ID, bob.ID)
- require.NoError(t, err)
- err = db.Unfollow(ctx, alice.ID, bob.ID)
- require.NoError(t, err)
- alice, err = usersStore.GetByID(ctx, alice.ID)
- require.NoError(t, err)
- assert.Equal(t, 0, alice.NumFollowing)
- bob, err = usersStore.GetByID(ctx, bob.ID)
- require.NoError(t, err)
- assert.Equal(t, 0, bob.NumFollowers)
- }
|