discourse/spec/models/user_action_spec.rb

287 lines
8.6 KiB
Ruby
Raw Normal View History

require 'rails_helper'
2013-02-05 14:16:51 -05:00
describe UserAction do
before do
ActiveRecord::Base.observers.enable :all
end
it { is_expected.to validate_presence_of :action_type }
it { is_expected.to validate_presence_of :user_id }
2013-02-05 14:16:51 -05:00
2013-02-25 11:42:20 -05:00
describe 'lists' do
2013-02-05 14:16:51 -05:00
let(:public_post) { Fabricate(:post) }
let(:public_topic) { public_post.topic }
let(:user) { Fabricate(:user) }
let(:private_post) { Fabricate(:post) }
2013-02-25 11:42:20 -05:00
let(:private_topic) do
topic = private_post.topic
topic.update_columns(category_id: nil, archetype: Archetype::private_message)
topic
end
def log_test_action(opts={})
UserAction.log_action!({
2013-02-05 14:16:51 -05:00
action_type: UserAction::NEW_PRIVATE_MESSAGE,
2013-02-25 11:42:20 -05:00
user_id: user.id,
acting_user_id: user.id,
target_topic_id: private_topic.id,
2013-02-25 11:42:20 -05:00
target_post_id: private_post.id,
}.merge(opts))
end
2013-02-05 14:16:51 -05:00
2013-02-25 11:42:20 -05:00
before do
# Create some test data using a helper
log_test_action
log_test_action(action_type: UserAction::GOT_PRIVATE_MESSAGE)
log_test_action(action_type: UserAction::NEW_TOPIC, target_topic_id: public_topic.id, target_post_id: public_post.id)
2013-02-25 11:42:20 -05:00
log_test_action(action_type: UserAction::BOOKMARK)
2013-02-05 14:16:51 -05:00
end
def stats_for_user(viewer=nil)
UserAction.stats(user.id, Guardian.new(viewer)).map{|r| r["action_type"].to_i}.sort
end
2013-02-05 14:16:51 -05:00
def stream_count(viewer=nil)
UserAction.stream(user_id: user.id, guardian: Guardian.new(viewer)).count
end
2013-02-05 14:16:51 -05:00
it 'includes the events correctly' do
mystats = stats_for_user(user)
expecting = [UserAction::NEW_TOPIC, UserAction::NEW_PRIVATE_MESSAGE, UserAction::GOT_PRIVATE_MESSAGE, UserAction::BOOKMARK].sort
expect(mystats).to eq(expecting)
expect(stream_count(user)).to eq(4)
other_stats = stats_for_user
expecting = [UserAction::NEW_TOPIC]
expect(stream_count).to eq(1)
2013-02-05 14:16:51 -05:00
expect(other_stats).to eq(expecting)
2013-02-05 14:16:51 -05:00
2013-07-09 15:20:18 -04:00
public_topic.trash!(user)
expect(stats_for_user).to eq([])
expect(stream_count).to eq(0)
2013-02-25 11:42:20 -05:00
# groups
category = Fabricate(:category, read_restricted: true)
public_topic.recover!
public_topic.category = category
public_topic.save
expect(stats_for_user).to eq([])
expect(stream_count).to eq(0)
group = Fabricate(:group)
u = Fabricate(:coding_horror)
group.add(u)
group.save
category.set_permissions(group => :full)
category.save
expect(stats_for_user(u)).to eq([UserAction::NEW_TOPIC])
expect(stream_count(u)).to eq(1)
2013-02-05 14:16:51 -05:00
# duplicate should not exception out
log_test_action
# recategorize belongs to the right user
category2 = Fabricate(:category)
admin = Fabricate(:admin)
public_post.revise(admin, { category_id: category2.id})
action = UserAction.stream(user_id: public_topic.user_id, guardian: Guardian.new)[0]
expect(action.acting_user_id).to eq(admin.id)
expect(action.action_type).to eq(UserAction::EDIT)
2013-02-05 14:16:51 -05:00
end
2013-02-05 14:16:51 -05:00
end
2013-02-25 11:42:20 -05:00
describe 'when user likes' do
let!(:post) { Fabricate(:post) }
let(:likee) { post.user }
let(:liker) { Fabricate(:coding_horror) }
def likee_stream
UserAction.stream(user_id: likee.id, guardian: Guardian.new)
2013-02-05 14:16:51 -05:00
end
before do
@old_count = likee_stream.count
2013-02-05 14:16:51 -05:00
end
2013-02-25 11:42:20 -05:00
it "creates a new stream entry" do
2013-03-01 07:07:44 -05:00
PostAction.act(liker, post, PostActionType.types[:like])
expect(likee_stream.count).to eq(@old_count + 1)
2013-02-05 14:16:51 -05:00
end
context "successful like" do
2013-02-25 11:42:20 -05:00
before do
2013-03-01 07:07:44 -05:00
PostAction.act(liker, post, PostActionType.types[:like])
@liker_action = liker.user_actions.find_by(action_type: UserAction::LIKE)
@likee_action = likee.user_actions.find_by(action_type: UserAction::WAS_LIKED)
end
it 'should result in correct data assignment' do
expect(@liker_action).not_to eq(nil)
expect(@likee_action).not_to eq(nil)
expect(likee.user_stat.reload.likes_received).to eq(1)
expect(liker.user_stat.reload.likes_given).to eq(1)
PostAction.remove_act(liker, post, PostActionType.types[:like])
expect(likee.user_stat.reload.likes_received).to eq(0)
expect(liker.user_stat.reload.likes_given).to eq(0)
end
end
context "liking a private message" do
before do
post.topic.update_columns(category_id: nil, archetype: Archetype::private_message)
end
it "doesn't add the entry to the stream" do
2013-03-01 07:07:44 -05:00
PostAction.act(liker, post, PostActionType.types[:like])
expect(likee_stream.count).not_to eq(@old_count + 1)
end
end
2013-02-05 14:16:51 -05:00
end
describe 'when a user posts a new topic' do
def process_alerts(post)
PostAlerter.post_created(post)
end
2013-02-25 11:42:20 -05:00
before do
2013-02-05 14:16:51 -05:00
@post = Fabricate(:old_post)
process_alerts(@post)
2013-02-05 14:16:51 -05:00
end
2013-02-25 11:42:20 -05:00
describe 'topic action' do
before do
@action = @post.user.user_actions.find_by(action_type: UserAction::NEW_TOPIC)
2013-02-05 14:16:51 -05:00
end
2013-02-25 11:42:20 -05:00
it 'should exist' do
expect(@action).not_to eq(nil)
expect(@action.created_at).to be_within(1).of(@post.topic.created_at)
2013-02-05 14:16:51 -05:00
end
end
2013-02-25 11:42:20 -05:00
it 'should not log a post user action' do
expect(@post.user.user_actions.find_by(action_type: UserAction::REPLY)).to eq(nil)
2013-02-05 14:16:51 -05:00
end
2013-02-25 11:42:20 -05:00
describe 'when another user posts on the topic' do
before do
2013-02-05 14:16:51 -05:00
@other_user = Fabricate(:coding_horror)
@mentioned = Fabricate(:admin)
@response = Fabricate(:post, reply_to_post_number: 1, topic: @post.topic, user: @other_user, raw: "perhaps @#{@mentioned.username} knows how this works?")
process_alerts(@response)
2013-02-05 14:16:51 -05:00
end
2013-02-25 11:42:20 -05:00
it 'should log user actions correctly' do
expect(@response.user.user_actions.find_by(action_type: UserAction::REPLY)).not_to eq(nil)
expect(@post.user.user_actions.find_by(action_type: UserAction::RESPONSE)).not_to eq(nil)
expect(@mentioned.user_actions.find_by(action_type: UserAction::MENTION)).not_to eq(nil)
expect(@post.user.user_actions.joins(:target_post).where('posts.post_number = 2').count).to eq(1)
2013-02-05 14:16:51 -05:00
end
it 'should not log a double notification for a post edit' do
@response.raw = "here it goes again"
2013-02-25 11:42:20 -05:00
@response.save!
expect(@response.user.user_actions.where(action_type: UserAction::REPLY).count).to eq(1)
2013-02-05 14:16:51 -05:00
end
end
end
describe 'when user bookmarks' do
2013-02-25 11:42:20 -05:00
before do
2013-02-05 14:16:51 -05:00
@post = Fabricate(:post)
@user = @post.user
2013-03-01 07:07:44 -05:00
PostAction.act(@user, @post, PostActionType.types[:bookmark])
@action = @user.user_actions.find_by(action_type: UserAction::BOOKMARK)
2013-02-05 14:16:51 -05:00
end
it 'should create a bookmark action correctly' do
expect(@action.action_type).to eq(UserAction::BOOKMARK)
expect(@action.target_post_id).to eq(@post.id)
expect(@action.acting_user_id).to eq(@user.id)
expect(@action.user_id).to eq(@user.id)
2013-03-01 07:07:44 -05:00
PostAction.remove_act(@user, @post, PostActionType.types[:bookmark])
expect(@user.user_actions.find_by(action_type: UserAction::BOOKMARK)).to eq(nil)
2013-02-05 14:16:51 -05:00
end
end
describe 'private messages' do
let(:user) do
Fabricate(:user)
end
let(:target_user) do
Fabricate(:user)
end
let(:private_message) do
PostCreator.create( user,
raw: 'this is a private message',
title: 'this is the pm title',
target_usernames: target_user.username,
archetype: Archetype::private_message
)
end
let!(:response) do
PostCreator.create(user, raw: 'oops I forgot to mention this', topic_id: private_message.topic_id)
end
let!(:private_message2) do
PostCreator.create( target_user,
raw: 'this is a private message',
title: 'this is the pm title',
target_usernames: user.username,
archetype: Archetype::private_message
)
end
end
2013-08-01 19:59:12 -04:00
describe 'synchronize_target_topic_ids' do
it 'correct target_topic_id' do
post = Fabricate(:post)
action = UserAction.log_action!(
action_type: UserAction::NEW_PRIVATE_MESSAGE,
user_id: post.user.id,
acting_user_id: post.user.id,
target_topic_id: -1,
target_post_id: post.id,
)
UserAction.log_action!(
action_type: UserAction::NEW_PRIVATE_MESSAGE,
user_id: post.user.id,
acting_user_id: post.user.id,
target_topic_id: -2,
target_post_id: post.id,
)
2013-08-01 19:59:12 -04:00
UserAction.synchronize_target_topic_ids
action.reload
expect(action.target_topic_id).to eq(post.topic_id)
end
end
2013-02-05 14:16:51 -05:00
end