| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496 |
- package sqlstore
- import (
- "context"
- "fmt"
- "testing"
- "time"
- . "github.com/smartystreets/goconvey/convey"
- "github.com/grafana/grafana/pkg/models"
- )
- func TestUserDataAccess(t *testing.T) {
- Convey("Testing DB", t, func() {
- ss := InitTestDB(t)
- Convey("Creates a user", func() {
- cmd := &models.CreateUserCommand{
- Email: "usertest@test.com",
- Name: "user name",
- Login: "user_test_login",
- }
- err := CreateUser(context.Background(), cmd)
- So(err, ShouldBeNil)
- Convey("Loading a user", func() {
- query := models.GetUserByIdQuery{Id: cmd.Result.Id}
- err := GetUserById(&query)
- So(err, ShouldBeNil)
- So(query.Result.Email, ShouldEqual, "usertest@test.com")
- So(query.Result.Password, ShouldEqual, "")
- So(query.Result.Rands, ShouldHaveLength, 10)
- So(query.Result.Salt, ShouldHaveLength, 10)
- So(query.Result.IsDisabled, ShouldBeFalse)
- })
- })
- Convey("Creates disabled user", func() {
- cmd := &models.CreateUserCommand{
- Email: "usertest@test.com",
- Name: "user name",
- Login: "user_test_login",
- IsDisabled: true,
- }
- err := CreateUser(context.Background(), cmd)
- So(err, ShouldBeNil)
- Convey("Loading a user", func() {
- query := models.GetUserByIdQuery{Id: cmd.Result.Id}
- err := GetUserById(&query)
- So(err, ShouldBeNil)
- So(query.Result.Email, ShouldEqual, "usertest@test.com")
- So(query.Result.Password, ShouldEqual, "")
- So(query.Result.Rands, ShouldHaveLength, 10)
- So(query.Result.Salt, ShouldHaveLength, 10)
- So(query.Result.IsDisabled, ShouldBeTrue)
- })
- })
- Convey("Given 5 users", func() {
- users := createFiveTestUsers(func(i int) *models.CreateUserCommand {
- return &models.CreateUserCommand{
- Email: fmt.Sprint("user", i, "@test.com"),
- Name: fmt.Sprint("user", i),
- Login: fmt.Sprint("loginuser", i),
- IsDisabled: false,
- }
- })
- Convey("Can return the first page of users and a total count", func() {
- query := models.SearchUsersQuery{Query: "", Page: 1, Limit: 3}
- err := SearchUsers(&query)
- So(err, ShouldBeNil)
- So(len(query.Result.Users), ShouldEqual, 3)
- So(query.Result.TotalCount, ShouldEqual, 5)
- })
- Convey("Can return the second page of users and a total count", func() {
- query := models.SearchUsersQuery{Query: "", Page: 2, Limit: 3}
- err := SearchUsers(&query)
- So(err, ShouldBeNil)
- So(len(query.Result.Users), ShouldEqual, 2)
- So(query.Result.TotalCount, ShouldEqual, 5)
- })
- Convey("Can return list of users matching query on user name", func() {
- query := models.SearchUsersQuery{Query: "use", Page: 1, Limit: 3}
- err := SearchUsers(&query)
- So(err, ShouldBeNil)
- So(len(query.Result.Users), ShouldEqual, 3)
- So(query.Result.TotalCount, ShouldEqual, 5)
- query = models.SearchUsersQuery{Query: "ser1", Page: 1, Limit: 3}
- err = SearchUsers(&query)
- So(err, ShouldBeNil)
- So(len(query.Result.Users), ShouldEqual, 1)
- So(query.Result.TotalCount, ShouldEqual, 1)
- query = models.SearchUsersQuery{Query: "USER1", Page: 1, Limit: 3}
- err = SearchUsers(&query)
- So(err, ShouldBeNil)
- So(len(query.Result.Users), ShouldEqual, 1)
- So(query.Result.TotalCount, ShouldEqual, 1)
- query = models.SearchUsersQuery{Query: "idontexist", Page: 1, Limit: 3}
- err = SearchUsers(&query)
- So(err, ShouldBeNil)
- So(len(query.Result.Users), ShouldEqual, 0)
- So(query.Result.TotalCount, ShouldEqual, 0)
- })
- Convey("Can return list of users matching query on email", func() {
- query := models.SearchUsersQuery{Query: "ser1@test.com", Page: 1, Limit: 3}
- err := SearchUsers(&query)
- So(err, ShouldBeNil)
- So(len(query.Result.Users), ShouldEqual, 1)
- So(query.Result.TotalCount, ShouldEqual, 1)
- })
- Convey("Can return list of users matching query on login name", func() {
- query := models.SearchUsersQuery{Query: "loginuser1", Page: 1, Limit: 3}
- err := SearchUsers(&query)
- So(err, ShouldBeNil)
- So(len(query.Result.Users), ShouldEqual, 1)
- So(query.Result.TotalCount, ShouldEqual, 1)
- })
- Convey("Can return list users based on their auth type", func() {
- // add users to auth table
- for index, user := range users {
- authModule := "killa"
- // define every second user as ldap
- if index%2 == 0 {
- authModule = "ldap"
- }
- cmd2 := &models.SetAuthInfoCommand{
- UserId: user.Id,
- AuthModule: authModule,
- AuthId: "gorilla",
- }
- err := SetAuthInfo(cmd2)
- So(err, ShouldBeNil)
- }
- query := models.SearchUsersQuery{AuthModule: "ldap"}
- err := SearchUsers(&query)
- So(err, ShouldBeNil)
- So(query.Result.Users, ShouldHaveLength, 3)
- zero, second, fourth := false, false, false
- for _, user := range query.Result.Users {
- if user.Name == "user0" {
- zero = true
- }
- if user.Name == "user2" {
- second = true
- }
- if user.Name == "user4" {
- fourth = true
- }
- }
- So(zero, ShouldBeTrue)
- So(second, ShouldBeTrue)
- So(fourth, ShouldBeTrue)
- })
- Convey("Can return list users based on their is_disabled flag", func() {
- ss = InitTestDB(t)
- createFiveTestUsers(func(i int) *models.CreateUserCommand {
- return &models.CreateUserCommand{
- Email: fmt.Sprint("user", i, "@test.com"),
- Name: fmt.Sprint("user", i),
- Login: fmt.Sprint("loginuser", i),
- IsDisabled: i%2 == 0,
- }
- })
- query := models.SearchUsersQuery{IsDisabled: "false"}
- err := SearchUsers(&query)
- So(err, ShouldBeNil)
- So(query.Result.Users, ShouldHaveLength, 2)
- first, third := false, false
- for _, user := range query.Result.Users {
- if user.Name == "user1" {
- first = true
- }
- if user.Name == "user3" {
- third = true
- }
- }
- So(first, ShouldBeTrue)
- So(third, ShouldBeTrue)
- ss = InitTestDB(t)
- users = createFiveTestUsers(func(i int) *models.CreateUserCommand {
- return &models.CreateUserCommand{
- Email: fmt.Sprint("user", i, "@test.com"),
- Name: fmt.Sprint("user", i),
- Login: fmt.Sprint("loginuser", i),
- IsDisabled: false,
- }
- })
- })
- Convey("when a user is an org member and has been assigned permissions", func() {
- err := AddOrgUser(&models.AddOrgUserCommand{LoginOrEmail: users[1].Login, Role: models.ROLE_VIEWER, OrgId: users[0].OrgId, UserId: users[1].Id})
- So(err, ShouldBeNil)
- testHelperUpdateDashboardAcl(1, models.DashboardAcl{DashboardId: 1, OrgId: users[0].OrgId, UserId: users[1].Id, Permission: models.PERMISSION_EDIT})
- So(err, ShouldBeNil)
- err = SavePreferences(&models.SavePreferencesCommand{UserId: users[1].Id, OrgId: users[0].OrgId, HomeDashboardId: 1, Theme: "dark"})
- So(err, ShouldBeNil)
- Convey("when the user is deleted", func() {
- err = DeleteUser(&models.DeleteUserCommand{UserId: users[1].Id})
- So(err, ShouldBeNil)
- Convey("Should delete connected org users and permissions", func() {
- query := &models.GetOrgUsersQuery{OrgId: users[0].OrgId}
- err = GetOrgUsersForTest(query)
- So(err, ShouldBeNil)
- So(len(query.Result), ShouldEqual, 1)
- permQuery := &models.GetDashboardAclInfoListQuery{DashboardId: 1, OrgId: users[0].OrgId}
- err = GetDashboardAclInfoList(permQuery)
- So(err, ShouldBeNil)
- So(len(permQuery.Result), ShouldEqual, 0)
- prefsQuery := &models.GetPreferencesQuery{OrgId: users[0].OrgId, UserId: users[1].Id}
- err = GetPreferences(prefsQuery)
- So(err, ShouldBeNil)
- So(prefsQuery.Result.OrgId, ShouldEqual, 0)
- So(prefsQuery.Result.UserId, ShouldEqual, 0)
- })
- })
- Convey("when retreiving signed in user for orgId=0 result should return active org id", func() {
- ss.CacheService.Flush()
- query := &models.GetSignedInUserQuery{OrgId: users[1].OrgId, UserId: users[1].Id}
- err := ss.GetSignedInUserWithCache(query)
- So(err, ShouldBeNil)
- So(query.Result, ShouldNotBeNil)
- So(query.OrgId, ShouldEqual, users[1].OrgId)
- err = SetUsingOrg(&models.SetUsingOrgCommand{UserId: users[1].Id, OrgId: users[0].OrgId})
- So(err, ShouldBeNil)
- query = &models.GetSignedInUserQuery{OrgId: 0, UserId: users[1].Id}
- err = ss.GetSignedInUserWithCache(query)
- So(err, ShouldBeNil)
- So(query.Result, ShouldNotBeNil)
- So(query.Result.OrgId, ShouldEqual, users[0].OrgId)
- cacheKey := newSignedInUserCacheKey(query.Result.OrgId, query.UserId)
- _, found := ss.CacheService.Get(cacheKey)
- So(found, ShouldBeTrue)
- })
- })
- Convey("When batch disabling users", func() {
- Convey("Should disable all users", func() {
- disableCmd := models.BatchDisableUsersCommand{
- UserIds: []int64{1, 2, 3, 4, 5},
- IsDisabled: true,
- }
- err := BatchDisableUsers(&disableCmd)
- So(err, ShouldBeNil)
- query := &models.SearchUsersQuery{IsDisabled: "true"}
- err = SearchUsers(query)
- So(err, ShouldBeNil)
- So(query.Result.TotalCount, ShouldEqual, 5)
- })
- Convey("Should enable all users", func() {
- ss = InitTestDB(t)
- createFiveTestUsers(func(i int) *models.CreateUserCommand {
- return &models.CreateUserCommand{
- Email: fmt.Sprint("user", i, "@test.com"),
- Name: fmt.Sprint("user", i),
- Login: fmt.Sprint("loginuser", i),
- IsDisabled: true,
- }
- })
- disableCmd := models.BatchDisableUsersCommand{
- UserIds: []int64{1, 2, 3, 4, 5},
- IsDisabled: false,
- }
- err := BatchDisableUsers(&disableCmd)
- So(err, ShouldBeNil)
- query := &models.SearchUsersQuery{IsDisabled: "false"}
- err = SearchUsers(query)
- So(err, ShouldBeNil)
- So(query.Result.TotalCount, ShouldEqual, 5)
- })
- Convey("Should disable only specific users", func() {
- ss = InitTestDB(t)
- users = createFiveTestUsers(func(i int) *models.CreateUserCommand {
- return &models.CreateUserCommand{
- Email: fmt.Sprint("user", i, "@test.com"),
- Name: fmt.Sprint("user", i),
- Login: fmt.Sprint("loginuser", i),
- IsDisabled: false,
- }
- })
- userIdsToDisable := []int64{}
- for i := 0; i < 3; i++ {
- userIdsToDisable = append(userIdsToDisable, users[i].Id)
- }
- disableCmd := models.BatchDisableUsersCommand{
- UserIds: userIdsToDisable,
- IsDisabled: true,
- }
- err := BatchDisableUsers(&disableCmd)
- So(err, ShouldBeNil)
- query := models.SearchUsersQuery{}
- err = SearchUsers(&query)
- So(err, ShouldBeNil)
- So(query.Result.TotalCount, ShouldEqual, 5)
- for _, user := range query.Result.Users {
- shouldBeDisabled := false
- // Check if user id is in the userIdsToDisable list
- for _, disabledUserId := range userIdsToDisable {
- if user.Id == disabledUserId {
- So(user.IsDisabled, ShouldBeTrue)
- shouldBeDisabled = true
- }
- }
- // Otherwise user shouldn't be disabled
- if !shouldBeDisabled {
- So(user.IsDisabled, ShouldBeFalse)
- }
- }
- })
- // Since previous tests were destructive
- ss = InitTestDB(t)
- users = createFiveTestUsers(func(i int) *models.CreateUserCommand {
- return &models.CreateUserCommand{
- Email: fmt.Sprint("user", i, "@test.com"),
- Name: fmt.Sprint("user", i),
- Login: fmt.Sprint("loginuser", i),
- IsDisabled: false,
- }
- })
- })
- Convey("When searching users", func() {
- // Find a user to set tokens on
- login := "loginuser0"
- // Calling GetUserByAuthInfoQuery on an existing user will populate an entry in the user_auth table
- // Make the first log-in during the past
- getTime = func() time.Time { return time.Now().AddDate(0, 0, -2) }
- query := &models.GetUserByAuthInfoQuery{Login: login, AuthModule: "test1", AuthId: "test1"}
- err := GetUserByAuthInfo(query)
- getTime = time.Now
- So(err, ShouldBeNil)
- So(query.Result.Login, ShouldEqual, login)
- // Add a second auth module for this user
- // Have this module's last log-in be more recent
- getTime = func() time.Time { return time.Now().AddDate(0, 0, -1) }
- query = &models.GetUserByAuthInfoQuery{Login: login, AuthModule: "test2", AuthId: "test2"}
- err = GetUserByAuthInfo(query)
- getTime = time.Now
- So(err, ShouldBeNil)
- So(query.Result.Login, ShouldEqual, login)
- Convey("Should return the only most recently used auth_module", func() {
- searchUserQuery := &models.SearchUsersQuery{}
- err = SearchUsers(searchUserQuery)
- So(err, ShouldBeNil)
- So(searchUserQuery.Result.Users, ShouldHaveLength, 5)
- for _, user := range searchUserQuery.Result.Users {
- if user.Login == login {
- So(user.AuthModule, ShouldHaveLength, 1)
- So(user.AuthModule[0], ShouldEqual, "test2")
- }
- }
- // "log in" again with the first auth module
- updateAuthCmd := &models.UpdateAuthInfoCommand{UserId: query.Result.Id, AuthModule: "test1", AuthId: "test1"}
- err = UpdateAuthInfo(updateAuthCmd)
- So(err, ShouldBeNil)
- searchUserQuery = &models.SearchUsersQuery{}
- err = SearchUsers(searchUserQuery)
- So(err, ShouldBeNil)
- for _, user := range searchUserQuery.Result.Users {
- if user.Login == login {
- So(user.AuthModule, ShouldHaveLength, 1)
- So(user.AuthModule[0], ShouldEqual, "test1")
- }
- }
- })
- })
- })
- Convey("Given one grafana admin user", func() {
- var err error
- createUserCmd := &models.CreateUserCommand{
- Email: fmt.Sprint("admin", "@test.com"),
- Name: fmt.Sprint("admin"),
- Login: fmt.Sprint("admin"),
- IsAdmin: true,
- }
- err = CreateUser(context.Background(), createUserCmd)
- So(err, ShouldBeNil)
- Convey("Cannot make themselves a non-admin", func() {
- updateUserPermsCmd := models.UpdateUserPermissionsCommand{IsGrafanaAdmin: false, UserId: 1}
- updatePermsError := UpdateUserPermissions(&updateUserPermsCmd)
- So(updatePermsError, ShouldEqual, models.ErrLastGrafanaAdmin)
- query := models.GetUserByIdQuery{Id: createUserCmd.Result.Id}
- getUserError := GetUserById(&query)
- So(getUserError, ShouldBeNil)
- So(query.Result.IsAdmin, ShouldEqual, true)
- })
- })
- })
- }
- func GetOrgUsersForTest(query *models.GetOrgUsersQuery) error {
- query.Result = make([]*models.OrgUserDTO, 0)
- sess := x.Table("org_user")
- sess.Join("LEFT ", x.Dialect().Quote("user"), fmt.Sprintf("org_user.user_id=%s.id", x.Dialect().Quote("user")))
- sess.Where("org_user.org_id=?", query.OrgId)
- sess.Cols("org_user.org_id", "org_user.user_id", "user.email", "user.login", "org_user.role")
- err := sess.Find(&query.Result)
- return err
- }
- func createFiveTestUsers(fn func(i int) *models.CreateUserCommand) []models.User {
- var err error
- var cmd *models.CreateUserCommand
- users := []models.User{}
- for i := 0; i < 5; i++ {
- cmd = fn(i)
- err = CreateUser(context.Background(), cmd)
- users = append(users, cmd.Result)
- So(err, ShouldBeNil)
- }
- return users
- }
|