From c0ab2636d699c8544ce03a58f61b95cfd66ff7ce Mon Sep 17 00:00:00 2001 From: Joram Wilander Date: Mon, 18 Jul 2016 11:10:03 -0400 Subject: PLT-2241 Refactored statuses into a more real-time system (#3573) * Refactored statuses into a more real-time system * Updated package.json with correct commit and fixed minor bug * Minor updates to statuses based on feedback * When setting status online, update only LastActivityAt if status already exists --- store/sql_user_store_test.go | 99 +------------------------------------------- 1 file changed, 1 insertion(+), 98 deletions(-) (limited to 'store/sql_user_store_test.go') diff --git a/store/sql_user_store_test.go b/store/sql_user_store_test.go index b6cfbb0ec..4b722b0b3 100644 --- a/store/sql_user_store_test.go +++ b/store/sql_user_store_test.go @@ -129,50 +129,6 @@ func TestUserStoreUpdateUpdateAt(t *testing.T) { } -func TestUserStoreUpdateLastPingAt(t *testing.T) { - Setup() - - u1 := &model.User{} - u1.Email = model.NewId() - Must(store.User().Save(u1)) - Must(store.Team().SaveMember(&model.TeamMember{TeamId: model.NewId(), UserId: u1.Id})) - - if err := (<-store.User().UpdateLastPingAt(u1.Id, 1234567890)).Err; err != nil { - t.Fatal(err) - } - - if r1 := <-store.User().Get(u1.Id); r1.Err != nil { - t.Fatal(r1.Err) - } else { - if r1.Data.(*model.User).LastPingAt != 1234567890 { - t.Fatal("LastPingAt not updated correctly") - } - } - -} - -func TestUserStoreUpdateLastActivityAt(t *testing.T) { - Setup() - - u1 := &model.User{} - u1.Email = model.NewId() - Must(store.User().Save(u1)) - Must(store.Team().SaveMember(&model.TeamMember{TeamId: model.NewId(), UserId: u1.Id})) - - if err := (<-store.User().UpdateLastActivityAt(u1.Id, 1234567890)).Err; err != nil { - t.Fatal(err) - } - - if r1 := <-store.User().Get(u1.Id); r1.Err != nil { - t.Fatal(r1.Err) - } else { - if r1.Data.(*model.User).LastActivityAt != 1234567890 { - t.Fatal("LastActivityAt not updated correctly") - } - } - -} - func TestUserStoreUpdateFailedPasswordAttempts(t *testing.T) { Setup() @@ -189,41 +145,7 @@ func TestUserStoreUpdateFailedPasswordAttempts(t *testing.T) { t.Fatal(r1.Err) } else { if r1.Data.(*model.User).FailedAttempts != 3 { - t.Fatal("LastActivityAt not updated correctly") - } - } - -} - -func TestUserStoreUpdateUserAndSessionActivity(t *testing.T) { - Setup() - - u1 := &model.User{} - u1.Email = model.NewId() - Must(store.User().Save(u1)) - Must(store.Team().SaveMember(&model.TeamMember{TeamId: model.NewId(), UserId: u1.Id})) - - s1 := model.Session{} - s1.UserId = u1.Id - Must(store.Session().Save(&s1)) - - if err := (<-store.User().UpdateUserAndSessionActivity(u1.Id, s1.Id, 1234567890)).Err; err != nil { - t.Fatal(err) - } - - if r1 := <-store.User().Get(u1.Id); r1.Err != nil { - t.Fatal(r1.Err) - } else { - if r1.Data.(*model.User).LastActivityAt != 1234567890 { - t.Fatal("LastActivityAt not updated correctly for user") - } - } - - if r2 := <-store.Session().Get(s1.Id); r2.Err != nil { - t.Fatal(r2.Err) - } else { - if r2.Data.(*model.Session).LastActivityAt != 1234567890 { - t.Fatal("LastActivityAt not updated correctly for session") + t.Fatal("FailedAttempts not updated correctly") } } @@ -268,25 +190,6 @@ func TestUserCount(t *testing.T) { } } -func TestActiveUserCount(t *testing.T) { - Setup() - - u1 := &model.User{} - u1.Email = model.NewId() - Must(store.User().Save(u1)) - Must(store.Team().SaveMember(&model.TeamMember{TeamId: model.NewId(), UserId: u1.Id})) - <-store.User().UpdateLastActivityAt(u1.Id, model.GetMillis()) - - if result := <-store.User().GetTotalActiveUsersCount(); result.Err != nil { - t.Fatal(result.Err) - } else { - count := result.Data.(int64) - if count <= 0 { - t.Fatal() - } - } -} - func TestUserStoreGetAllProfiles(t *testing.T) { Setup() -- cgit v1.2.3-1-g7c22