me/offline-twitter
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Unread notifications count should be specific to a single user

Browse Source
This commit is contained in:
Alessio 2024-09-02 16:25:54 -07:00
parent 9c7174a84a
commit 72b547f6aa
2 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/persistence/notification_queries.go
View File

@ -116,9 +116,9 @@ func (p Profile) CheckNotificationScrapesNeeded(trove TweetTrove) []Notification
return ret
}
func (p Profile) GetUnreadNotificationsCount(since_sort_index int64) int {
func (p Profile) GetUnreadNotificationsCount(u_id UserID, since_sort_index int64) int {
var ret int
err := p.DB.Get(&ret, `select count(*) from notifications where sort_index > ?`, since_sort_index)
err := p.DB.Get(&ret, `select count(*) from notifications where sort_index > ? and user_id = ?`, since_sort_index, u_id)
if err != nil {
panic(err)
}

3
pkg/persistence/notification_queries_test.go
View File

@ -8,6 +8,7 @@ import (
"github.com/stretchr/testify/require"
"gitlab.com/offline-twitter/twitter_offline_engine/pkg/persistence"
. "gitlab.com/offline-twitter/twitter_offline_engine/pkg/scraper"
)
func TestSaveAndLoadNotification(t *testing.T) {
@ -32,6 +33,6 @@ func TestGetUnreadNotificationsCount(t *testing.T) {
profile, err := persistence.LoadProfile("../../sample_data/profile")
require.NoError(err)
unread_notifs_count := profile.GetUnreadNotificationsCount(1724372973735)
unread_notifs_count := profile.GetUnreadNotificationsCount(UserID(1488963321701171204), 1724372973735)
assert.Equal(2, unread_notifs_count)
}