From 70e5f00241473c27a3008959ce08832c75e76ba8 Mon Sep 17 00:00:00 2001 From: Chris Date: Fri, 6 Oct 2017 11:08:59 -0700 Subject: store/storetest package (#7588) * prerequisites * storetest package --- store/sqlstore/status_store_test.go | 99 ++----------------------------------- 1 file changed, 4 insertions(+), 95 deletions(-) (limited to 'store/sqlstore/status_store_test.go') diff --git a/store/sqlstore/status_store_test.go b/store/sqlstore/status_store_test.go index 92374ff7a..ddfedeb18 100644 --- a/store/sqlstore/status_store_test.go +++ b/store/sqlstore/status_store_test.go @@ -1,4 +1,4 @@ -// Copyright (c) 2016-present Mattermost, Inc. All Rights Reserved. +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. // See License.txt for license information. package sqlstore @@ -6,100 +6,9 @@ package sqlstore import ( "testing" - "github.com/mattermost/mattermost-server/model" - "github.com/mattermost/mattermost-server/store" + "github.com/mattermost/mattermost-server/store/storetest" ) -func TestSqlStatusStore(t *testing.T) { - ss := Setup() - - status := &model.Status{UserId: model.NewId(), Status: model.STATUS_ONLINE, Manual: false, LastActivityAt: 0, ActiveChannel: ""} - - if err := (<-ss.Status().SaveOrUpdate(status)).Err; err != nil { - t.Fatal(err) - } - - status.LastActivityAt = 10 - - if err := (<-ss.Status().SaveOrUpdate(status)).Err; err != nil { - t.Fatal(err) - } - - if err := (<-ss.Status().Get(status.UserId)).Err; err != nil { - t.Fatal(err) - } - - status2 := &model.Status{UserId: model.NewId(), Status: model.STATUS_AWAY, Manual: false, LastActivityAt: 0, ActiveChannel: ""} - if err := (<-ss.Status().SaveOrUpdate(status2)).Err; err != nil { - t.Fatal(err) - } - - status3 := &model.Status{UserId: model.NewId(), Status: model.STATUS_OFFLINE, Manual: false, LastActivityAt: 0, ActiveChannel: ""} - if err := (<-ss.Status().SaveOrUpdate(status3)).Err; err != nil { - t.Fatal(err) - } - - if result := <-ss.Status().GetOnlineAway(); result.Err != nil { - t.Fatal(result.Err) - } else { - statuses := result.Data.([]*model.Status) - for _, status := range statuses { - if status.Status == model.STATUS_OFFLINE { - t.Fatal("should not have returned offline statuses") - } - } - } - - if result := <-ss.Status().GetOnline(); result.Err != nil { - t.Fatal(result.Err) - } else { - statuses := result.Data.([]*model.Status) - for _, status := range statuses { - if status.Status != model.STATUS_ONLINE { - t.Fatal("should not have returned offline statuses") - } - } - } - - if result := <-ss.Status().GetByIds([]string{status.UserId, "junk"}); result.Err != nil { - t.Fatal(result.Err) - } else { - statuses := result.Data.([]*model.Status) - if len(statuses) != 1 { - t.Fatal("should only have 1 status") - } - } - - if err := (<-ss.Status().ResetAll()).Err; err != nil { - t.Fatal(err) - } - - if result := <-ss.Status().Get(status.UserId); result.Err != nil { - t.Fatal(result.Err) - } else { - status := result.Data.(*model.Status) - if status.Status != model.STATUS_OFFLINE { - t.Fatal("should be offline") - } - } - - if result := <-ss.Status().UpdateLastActivityAt(status.UserId, 10); result.Err != nil { - t.Fatal(result.Err) - } -} - -func TestActiveUserCount(t *testing.T) { - ss := Setup() - - status := &model.Status{UserId: model.NewId(), Status: model.STATUS_ONLINE, Manual: false, LastActivityAt: model.GetMillis(), ActiveChannel: ""} - store.Must(ss.Status().SaveOrUpdate(status)) - - if result := <-ss.Status().GetTotalActiveUsersCount(); result.Err != nil { - t.Fatal(result.Err) - } else { - count := result.Data.(int64) - if count <= 0 { - t.Fatal() - } - } +func TestStatusStore(t *testing.T) { + StoreTest(t, storetest.TestStatusStore) } -- cgit v1.2.3-1-g7c22