2019-04-29 20:27:42 -04:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2022-07-27 22:27:38 -04:00
|
|
|
RSpec.describe UserSummary do
|
2016-01-27 19:12:12 -05:00
|
|
|
it "produces secure summaries" do
|
|
|
|
topic = create_post.topic
|
|
|
|
user = topic.user
|
|
|
|
_reply = create_post(user: topic.user, topic: topic)
|
|
|
|
|
|
|
|
summary = UserSummary.new(user, Guardian.new)
|
|
|
|
|
|
|
|
expect(summary.topics.length).to eq(1)
|
|
|
|
expect(summary.replies.length).to eq(1)
|
2018-07-18 16:37:50 -04:00
|
|
|
expect(summary.top_categories.length).to eq(1)
|
|
|
|
expect(summary.top_categories.first[:topic_count]).to eq(1)
|
|
|
|
expect(summary.top_categories.first[:post_count]).to eq(1)
|
2016-01-27 19:12:12 -05:00
|
|
|
|
|
|
|
topic.update_columns(deleted_at: Time.now)
|
|
|
|
|
|
|
|
expect(summary.topics.length).to eq(0)
|
|
|
|
expect(summary.replies.length).to eq(0)
|
2018-07-18 16:37:50 -04:00
|
|
|
expect(summary.top_categories.length).to eq(0)
|
2016-01-27 19:12:12 -05:00
|
|
|
|
|
|
|
topic.update_columns(deleted_at: nil, visible: false)
|
|
|
|
|
|
|
|
expect(summary.topics.length).to eq(0)
|
|
|
|
expect(summary.replies.length).to eq(0)
|
2018-07-18 16:37:50 -04:00
|
|
|
expect(summary.top_categories.length).to eq(0)
|
2016-01-27 19:12:12 -05:00
|
|
|
|
|
|
|
category = Fabricate(:category)
|
|
|
|
topic.update_columns(category_id: category.id, deleted_at: nil, visible: true)
|
|
|
|
|
|
|
|
category.set_permissions(staff: :full)
|
|
|
|
category.save
|
|
|
|
|
|
|
|
expect(summary.topics.length).to eq(0)
|
|
|
|
expect(summary.replies.length).to eq(0)
|
2018-07-18 16:37:50 -04:00
|
|
|
expect(summary.top_categories.length).to eq(0)
|
2016-01-27 19:12:12 -05:00
|
|
|
end
|
|
|
|
|
2018-12-24 14:34:55 -05:00
|
|
|
it "is robust enough to handle bad data" do
|
2019-01-03 12:03:01 -05:00
|
|
|
UserActionManager.enable
|
2018-12-24 14:34:55 -05:00
|
|
|
|
|
|
|
liked_post = create_post
|
|
|
|
user = Fabricate(:user)
|
2019-01-03 12:03:01 -05:00
|
|
|
PostActionCreator.like(user, liked_post)
|
2018-12-24 14:34:55 -05:00
|
|
|
|
|
|
|
users = UserSummary.new(user, Guardian.new).most_liked_users
|
|
|
|
|
|
|
|
expect(users.map(&:id)).to eq([liked_post.user_id])
|
|
|
|
|
|
|
|
# really we should not be corrupting stuff like this
|
|
|
|
# but in production dbs this can happens sometimes I guess
|
|
|
|
liked_post.user.delete
|
|
|
|
|
|
|
|
users = UserSummary.new(user, Guardian.new).most_liked_users
|
|
|
|
expect(users).to eq([])
|
|
|
|
end
|
2021-04-15 06:05:03 -04:00
|
|
|
|
|
|
|
it "includes ordered top categories" do
|
|
|
|
u = Fabricate(:user)
|
|
|
|
|
|
|
|
UserSummary::MAX_SUMMARY_RESULTS.times do
|
|
|
|
c = Fabricate(:category)
|
|
|
|
t = Fabricate(:topic, category: c, user: u)
|
|
|
|
Fabricate(:post, user: u, topic: t)
|
|
|
|
end
|
|
|
|
|
|
|
|
top_category = Fabricate(:category)
|
|
|
|
t = Fabricate(:topic, category: top_category, user: u)
|
|
|
|
Fabricate(:post, user: u, topic: t)
|
|
|
|
Fabricate(:post, user: u, topic: t)
|
|
|
|
|
|
|
|
summary = UserSummary.new(u, Guardian.new)
|
|
|
|
|
|
|
|
expect(summary.top_categories.length).to eq(UserSummary::MAX_SUMMARY_RESULTS)
|
|
|
|
expect(summary.top_categories.first[:id]).to eq(top_category.id)
|
|
|
|
end
|
2021-11-18 03:12:03 -05:00
|
|
|
|
|
|
|
it "excludes moderator action posts" do
|
|
|
|
topic = create_post.topic
|
|
|
|
user = topic.user
|
|
|
|
create_post(user: user, topic: topic)
|
|
|
|
Fabricate(:small_action, topic: topic, user: user)
|
|
|
|
|
|
|
|
summary = UserSummary.new(user, Guardian.new)
|
|
|
|
|
|
|
|
expect(summary.topics.length).to eq(1)
|
|
|
|
expect(summary.replies.length).to eq(1)
|
|
|
|
expect(summary.top_categories.length).to eq(1)
|
|
|
|
expect(summary.top_categories.first[:topic_count]).to eq(1)
|
|
|
|
expect(summary.top_categories.first[:post_count]).to eq(1)
|
|
|
|
end
|
PERF: Improve performance of `most_replied_to_users` (#26373)
This PR improves the performance of the `most_replied_to_users` method on the `UserSummary` model.
### Old Query
```ruby
post_query
.joins(
"JOIN posts replies ON posts.topic_id = replies.topic_id AND posts.reply_to_post_number = replies.post_number",
)
# We are removing replies by @user, but we can simplify this by getting the using the user_id on the posts.
.where("replies.user_id <> ?", @user.id)
.group("replies.user_id")
.order("COUNT(*) DESC")
.limit(MAX_SUMMARY_RESULTS)
.pluck("replies.user_id, COUNT(*)")
.each { |r| replied_users[r[0]] = r[1] }
```
### Old Query with corrections
```ruby
post_query
.joins(
"JOIN posts replies ON posts.topic_id = replies.topic_id AND replies.reply_to_post_number = posts.post_number",
)
# Remove replies by @user but instead look on loaded posts (we do this so we don't count self replies)
.where("replies.user_id <> posts.user_id")
.group("replies.user_id")
.order("COUNT(*) DESC")
.limit(MAX_SUMMARY_RESULTS)
.pluck("replies.user_id, COUNT(*)")
.each { |r| replied_users[r[0]] = r[1] }
```
### New Query
```ruby
post_query
.joins(
"JOIN posts replies ON posts.topic_id = replies.topic_id AND posts.reply_to_post_number = replies.post_number",
)
# Only include regular posts in our joins, this makes sure we don't have the bloat of loading private messages
.joins(
"JOIN topics ON replies.topic_id = topics.id AND topics.archetype <> 'private_message'",
)
# Only include visible post types, so exclude posts like whispers, etc
.joins(
"AND replies.post_type IN (#{Topic.visible_post_types(@user, include_moderator_actions: false).join(",")})",
)
.where("replies.user_id <> posts.user_id")
.group("replies.user_id")
.order("COUNT(*) DESC")
.limit(MAX_SUMMARY_RESULTS)
.pluck("replies.user_id, COUNT(*)")
.each { |r| replied_users[r[0]] = r[1] }
```
# Conclusion
`most_replied_to_users` was untested, so I introduced a test for the logic, and have confirmed that it passes on both the new query **AND** the old query.
Thank you @danielwaterworth for the debugging assistance.
2024-04-03 16:20:54 -04:00
|
|
|
|
|
|
|
it "returns the most replied to users" do
|
|
|
|
topic1 = create_post.topic
|
|
|
|
topic1_post = create_post(topic: topic1)
|
|
|
|
topic1_reply =
|
|
|
|
create_post(topic: topic1, reply_to_post_number: topic1_post.post_number, user: topic1.user)
|
|
|
|
|
|
|
|
# Create a second topic by the same user as topic1
|
|
|
|
topic2 = create_post(user: topic1.user).topic
|
|
|
|
topic2_post = create_post(topic: topic2)
|
|
|
|
topic2_reply =
|
|
|
|
create_post(topic: topic2, reply_to_post_number: topic2_post.post_number, user: topic2.user)
|
|
|
|
|
|
|
|
# Don't include replies to whispers
|
|
|
|
topic3 = create_post(user: topic1.user).topic
|
|
|
|
topic3_post = create_post(topic: topic3, post_type: Post.types[:whisper])
|
|
|
|
topic3_reply =
|
|
|
|
create_post(topic: topic3, reply_to_post_number: topic3_post.post_number, user: topic3.user)
|
|
|
|
|
|
|
|
# Don't include replies to private messages
|
|
|
|
replied_to_user = Fabricate(:user)
|
|
|
|
topic4 =
|
|
|
|
create_post(
|
|
|
|
user: topic1.user,
|
|
|
|
archetype: Archetype.private_message,
|
|
|
|
target_usernames: [replied_to_user.username],
|
|
|
|
).topic
|
|
|
|
topic4_post = create_post(topic: topic4, user: replied_to_user)
|
|
|
|
topic4_reply =
|
|
|
|
create_post(topic: topic4, reply_to_post_number: topic4_post.post_number, user: topic4.user)
|
|
|
|
|
|
|
|
user_summary = UserSummary.new(topic1.user, Guardian.new(topic1.user))
|
|
|
|
most_replied_to_users = user_summary.most_replied_to_users
|
|
|
|
|
|
|
|
counts =
|
|
|
|
most_replied_to_users
|
|
|
|
.index_by { |user_with_count| user_with_count[:id] }
|
|
|
|
.transform_values { |c| c[:count] }
|
|
|
|
|
|
|
|
expect(counts).to eq({ topic1_post.user_id => 1, topic2_post.user_id => 1 })
|
|
|
|
end
|
2016-01-27 19:12:12 -05:00
|
|
|
end
|