FIX: `TopicTrackingState.report` not including unread for staff posts.

Follow-up to e15c86e8c5
This commit is contained in:
Alan Guo Xiang Tan 2021-08-12 10:31:50 +08:00
parent 2fb17b7b17
commit 9a60c83535
2 changed files with 24 additions and 1 deletions

View File

@ -385,7 +385,7 @@ class TopicTrackingState
if skip_unread
"1=0"
else
unread_filter_sql
unread_filter_sql(staff: staff)
end
filter_old_unread_sql =

View File

@ -688,4 +688,27 @@ describe TopicTrackingState do
expect(TopicTrackingState.report(post.user)).to be_empty
expect(TopicTrackingState.report(user)).to be_empty
end
describe ".report" do
it "correctly reports topics with staff posts" do
create_post(
raw: "this is a test post",
topic: topic,
user: post.user
)
create_post(
raw: "this is a test post",
topic: topic,
post_type: Post.types[:whisper],
user: user
)
post.user.grant_admin!
state = TopicTrackingState.report(post.user)
expect(state.map(&:topic_id)).to contain_exactly(topic.id)
end
end
end