2015-10-11 05:41:23 -04:00
|
|
|
require 'rails_helper'
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
describe UserUpdater do
|
2013-12-10 12:46:35 -05:00
|
|
|
|
|
|
|
let(:acting_user) { Fabricate.build(:user) }
|
|
|
|
|
2015-03-30 19:16:11 -04:00
|
|
|
describe '#update_muted_users' do
|
|
|
|
it 'has no cross talk' do
|
|
|
|
u1 = Fabricate(:user)
|
|
|
|
u2 = Fabricate(:user)
|
|
|
|
u3 = Fabricate(:user)
|
|
|
|
|
|
|
|
updater = UserUpdater.new(u1, u1)
|
|
|
|
updater.update_muted_users("#{u2.username},#{u3.username}")
|
|
|
|
|
|
|
|
updater = UserUpdater.new(u2, u2)
|
|
|
|
updater.update_muted_users("#{u3.username},#{u1.username}")
|
|
|
|
|
|
|
|
updater = UserUpdater.new(u3, u3)
|
|
|
|
updater.update_muted_users("")
|
|
|
|
|
|
|
|
expect(MutedUser.where(user_id: u2.id).count).to eq 2
|
|
|
|
expect(MutedUser.where(user_id: u1.id).count).to eq 2
|
|
|
|
expect(MutedUser.where(user_id: u3.id).count).to eq 0
|
|
|
|
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2013-11-01 17:06:20 -04:00
|
|
|
describe '#update' do
|
|
|
|
it 'saves user' do
|
|
|
|
user = Fabricate(:user, name: 'Billy Bob')
|
2016-02-16 23:46:19 -05:00
|
|
|
updater = UserUpdater.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(name: 'Jim Tom')
|
|
|
|
|
|
|
|
expect(user.reload.name).to eq 'Jim Tom'
|
|
|
|
end
|
|
|
|
|
2016-07-07 22:58:18 -04:00
|
|
|
it 'can update categories and tags' do
|
|
|
|
category = Fabricate(:category)
|
|
|
|
tag = Fabricate(:tag)
|
|
|
|
|
|
|
|
user = Fabricate(:user)
|
|
|
|
updater = UserUpdater.new(acting_user, user)
|
|
|
|
updater.update(watched_tags: [tag.name], muted_category_ids: [category.id])
|
|
|
|
|
|
|
|
expect(TagUser.where(
|
|
|
|
user_id: user.id,
|
|
|
|
tag_id: tag.id,
|
|
|
|
notification_level: TagUser.notification_levels[:watching]
|
|
|
|
).count).to eq(1)
|
|
|
|
|
|
|
|
expect(CategoryUser.where(
|
|
|
|
user_id: user.id,
|
|
|
|
category_id: category.id,
|
|
|
|
notification_level: CategoryUser.notification_levels[:muted]
|
|
|
|
).count).to eq(1)
|
|
|
|
|
|
|
|
end
|
|
|
|
|
2016-02-16 23:46:19 -05:00
|
|
|
it 'updates various fields' do
|
2014-06-10 01:19:08 -04:00
|
|
|
user = Fabricate(:user)
|
2016-02-16 23:46:19 -05:00
|
|
|
updater = UserUpdater.new(acting_user, user)
|
2016-11-15 03:10:20 -05:00
|
|
|
date_of_birth = Time.zone.now
|
2016-02-16 23:46:19 -05:00
|
|
|
|
2017-05-15 12:48:08 -04:00
|
|
|
theme = Theme.create!(user_id: -1, name: "test", user_selectable: true)
|
|
|
|
|
|
|
|
seq = user.user_option.theme_key_seq
|
|
|
|
|
2016-08-01 01:29:28 -04:00
|
|
|
val = updater.update(bio_raw: 'my new bio',
|
2017-07-27 21:20:09 -04:00
|
|
|
email_always: 'true',
|
|
|
|
mailing_list_mode: true,
|
|
|
|
digest_after_minutes: "45",
|
|
|
|
new_topic_duration_minutes: 100,
|
|
|
|
auto_track_topics_after_msecs: 101,
|
|
|
|
notification_level_when_replying: 3,
|
|
|
|
email_in_reply_to: false,
|
|
|
|
date_of_birth: date_of_birth,
|
2017-10-06 03:56:58 -04:00
|
|
|
theme_key: theme.key,
|
|
|
|
allow_private_messages: false)
|
|
|
|
|
2016-08-01 01:29:28 -04:00
|
|
|
expect(val).to be_truthy
|
|
|
|
|
2016-02-16 23:46:19 -05:00
|
|
|
user.reload
|
|
|
|
|
|
|
|
expect(user.user_profile.bio_raw).to eq 'my new bio'
|
|
|
|
expect(user.user_option.email_always).to eq true
|
|
|
|
expect(user.user_option.mailing_list_mode).to eq true
|
2016-03-02 15:26:27 -05:00
|
|
|
expect(user.user_option.digest_after_minutes).to eq 45
|
2016-02-18 00:57:22 -05:00
|
|
|
expect(user.user_option.new_topic_duration_minutes).to eq 100
|
|
|
|
expect(user.user_option.auto_track_topics_after_msecs).to eq 101
|
2016-09-30 12:36:43 -04:00
|
|
|
expect(user.user_option.notification_level_when_replying).to eq 3
|
2016-02-25 08:05:40 -05:00
|
|
|
expect(user.user_option.email_in_reply_to).to eq false
|
2017-05-15 12:48:08 -04:00
|
|
|
expect(user.user_option.theme_key).to eq theme.key
|
2017-07-27 21:20:09 -04:00
|
|
|
expect(user.user_option.theme_key_seq).to eq(seq + 1)
|
2017-10-06 03:56:58 -04:00
|
|
|
expect(user.user_option.allow_private_messages).to eq(false)
|
2016-11-15 03:10:20 -05:00
|
|
|
expect(user.date_of_birth).to eq(date_of_birth.to_date)
|
2014-06-10 01:19:08 -04:00
|
|
|
end
|
|
|
|
|
2016-11-28 09:52:35 -05:00
|
|
|
it "disables email_digests when enabling mailing_list_mode" do
|
|
|
|
user = Fabricate(:user)
|
|
|
|
updater = UserUpdater.new(acting_user, user)
|
|
|
|
|
|
|
|
val = updater.update(mailing_list_mode: true, email_digests: true)
|
|
|
|
expect(val).to be_truthy
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
|
|
|
|
expect(user.user_option.email_digests).to eq false
|
|
|
|
expect(user.user_option.mailing_list_mode).to eq true
|
|
|
|
end
|
|
|
|
|
2016-08-01 01:29:28 -04:00
|
|
|
context 'when sso overrides bio' do
|
|
|
|
it 'does not change bio' do
|
|
|
|
SiteSetting.enable_sso = true
|
|
|
|
SiteSetting.sso_overrides_bio = true
|
|
|
|
|
2013-11-01 17:06:20 -04:00
|
|
|
user = Fabricate(:user)
|
2016-02-16 23:46:19 -05:00
|
|
|
updater = UserUpdater.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
2016-08-01 01:29:28 -04:00
|
|
|
expect(updater.update(bio_raw: "new bio")).to be_truthy
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.user_profile.bio_raw).not_to eq 'new bio'
|
2013-11-01 17:06:20 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when update fails' do
|
|
|
|
it 'returns false' do
|
|
|
|
user = Fabricate(:user)
|
|
|
|
user.stubs(save: false)
|
2016-02-16 23:46:19 -05:00
|
|
|
updater = UserUpdater.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
2014-09-25 11:44:48 -04:00
|
|
|
expect(updater.update).to be_falsey
|
2013-11-01 17:06:20 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with permission to update title' do
|
|
|
|
it 'allows user to change title' do
|
|
|
|
user = Fabricate(:user, title: 'Emperor')
|
|
|
|
guardian = stub
|
|
|
|
guardian.stubs(:can_grant_title?).with(user).returns(true)
|
2013-12-10 12:46:35 -05:00
|
|
|
Guardian.stubs(:new).with(acting_user).returns(guardian)
|
2016-02-16 23:46:19 -05:00
|
|
|
updater = UserUpdater.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(title: 'Minion')
|
|
|
|
|
|
|
|
expect(user.reload.title).to eq 'Minion'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'without permission to update title' do
|
|
|
|
it 'does not allow user to change title' do
|
|
|
|
user = Fabricate(:user, title: 'Emperor')
|
|
|
|
guardian = stub
|
|
|
|
guardian.stubs(:can_grant_title?).with(user).returns(false)
|
2013-12-10 12:46:35 -05:00
|
|
|
Guardian.stubs(:new).with(acting_user).returns(guardian)
|
2017-02-27 15:02:00 -05:00
|
|
|
updater = UserUpdater.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(title: 'Minion')
|
|
|
|
|
|
|
|
expect(user.reload.title).not_to eq 'Minion'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when website includes http' do
|
|
|
|
it 'does not add http before updating' do
|
|
|
|
user = Fabricate(:user)
|
2017-02-27 15:02:00 -05:00
|
|
|
updater = UserUpdater.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(website: 'http://example.com')
|
|
|
|
|
2014-06-07 00:54:32 -04:00
|
|
|
expect(user.reload.user_profile.website).to eq 'http://example.com'
|
2013-11-01 17:06:20 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when website does not include http' do
|
|
|
|
it 'adds http before updating' do
|
|
|
|
user = Fabricate(:user)
|
2017-02-27 15:02:00 -05:00
|
|
|
updater = UserUpdater.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(website: 'example.com')
|
|
|
|
|
2014-06-07 00:54:32 -04:00
|
|
|
expect(user.reload.user_profile.website).to eq 'http://example.com'
|
2013-11-01 17:06:20 -04:00
|
|
|
end
|
|
|
|
end
|
2014-09-17 13:09:39 -04:00
|
|
|
|
|
|
|
context 'when custom_fields is empty string' do
|
|
|
|
it "update is successful" do
|
|
|
|
user = Fabricate(:user)
|
2017-07-27 21:20:09 -04:00
|
|
|
user.custom_fields = { 'import_username' => 'my_old_username' }
|
2014-09-17 13:09:39 -04:00
|
|
|
user.save
|
2017-02-27 15:02:00 -05:00
|
|
|
updater = UserUpdater.new(acting_user, user)
|
2014-09-17 13:09:39 -04:00
|
|
|
|
|
|
|
updater.update(website: 'example.com', custom_fields: '')
|
2017-07-27 21:20:09 -04:00
|
|
|
expect(user.reload.custom_fields).to eq('import_username' => 'my_old_username')
|
2014-09-17 13:09:39 -04:00
|
|
|
end
|
|
|
|
end
|
2017-02-23 00:48:57 -05:00
|
|
|
|
|
|
|
it "logs the action" do
|
2017-03-01 03:12:17 -05:00
|
|
|
user_without_name = Fabricate(:user, name: nil)
|
2017-02-23 00:48:57 -05:00
|
|
|
user = Fabricate(:user, name: 'Billy Bob')
|
2017-10-22 23:16:14 -04:00
|
|
|
expect do
|
|
|
|
UserUpdater.new(acting_user, user).update(name: 'Jim Tom')
|
|
|
|
end.to change { UserHistory.count }.by(1)
|
|
|
|
|
|
|
|
expect do
|
|
|
|
UserUpdater.new(acting_user, user).update(name: 'JiM TOm')
|
|
|
|
end.to_not change { UserHistory.count }
|
|
|
|
|
|
|
|
expect do
|
|
|
|
UserUpdater.new(acting_user, user_without_name).update(bio_raw: 'foo bar')
|
|
|
|
end.to_not change { UserHistory.count }
|
|
|
|
|
|
|
|
expect do
|
|
|
|
UserUpdater.new(acting_user, user_without_name).update(name: 'Jim Tom')
|
|
|
|
end.to change { UserHistory.count }.by(1)
|
|
|
|
|
|
|
|
expect do
|
|
|
|
UserUpdater.new(acting_user, user).update(name: '')
|
|
|
|
end.to change { UserHistory.count }.by(1)
|
2017-02-23 00:48:57 -05:00
|
|
|
end
|
2013-11-01 17:06:20 -04:00
|
|
|
end
|
|
|
|
end
|