2015-10-11 17:41:23 +08:00
|
|
|
require 'rails_helper'
|
2013-04-29 14:33:24 +08:00
|
|
|
|
|
|
|
describe Group do
|
2013-05-06 12:49:56 +08:00
|
|
|
|
2016-01-08 18:53:52 +08:00
|
|
|
describe '#builtin' do
|
|
|
|
context "verify enum sequence" do
|
|
|
|
before do
|
|
|
|
@builtin = Group.builtin
|
|
|
|
end
|
|
|
|
|
|
|
|
it "'moderators' should be at 1st position" do
|
|
|
|
expect(@builtin[:moderators]).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "'trust_level_2' should be at 4th position" do
|
|
|
|
expect(@builtin[:trust_level_2]).to eq(4)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-06-17 08:46:30 +08:00
|
|
|
# UGLY but perf is horrible with this callback
|
|
|
|
before do
|
|
|
|
User.set_callback(:create, :after, :ensure_in_trust_level_group)
|
|
|
|
end
|
|
|
|
after do
|
|
|
|
User.skip_callback(:create, :after, :ensure_in_trust_level_group)
|
|
|
|
end
|
|
|
|
|
2013-05-09 15:37:34 +08:00
|
|
|
describe "validation" do
|
|
|
|
let(:group) { build(:group) }
|
|
|
|
|
|
|
|
it "is invalid for blank" do
|
|
|
|
group.name = ""
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(group.valid?).to eq false
|
2013-05-09 15:37:34 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it "is valid for a longer name" do
|
|
|
|
group.name = "this_is_a_name"
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(group.valid?).to eq true
|
2013-05-09 15:37:34 +08:00
|
|
|
end
|
2013-06-17 11:43:06 +08:00
|
|
|
|
|
|
|
it "is invalid for non names" do
|
|
|
|
group.name = "this is_a_name"
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(group.valid?).to eq false
|
2013-06-17 11:43:06 +08:00
|
|
|
end
|
2014-12-16 20:07:15 +08:00
|
|
|
|
|
|
|
it "is invalid for case-insensitive existing names" do
|
|
|
|
build(:group, name: 'this_is_a_name').save
|
|
|
|
group.name = 'This_Is_A_Name'
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(group.valid?).to eq false
|
2014-12-16 20:07:15 +08:00
|
|
|
end
|
2015-11-27 13:35:16 +08:00
|
|
|
|
|
|
|
it "is invalid for poorly formatted domains" do
|
|
|
|
group.automatic_membership_email_domains = "wikipedia.org|*@example.com"
|
|
|
|
expect(group.valid?).to eq false
|
|
|
|
end
|
|
|
|
|
|
|
|
it "is valid for proper domains" do
|
|
|
|
group.automatic_membership_email_domains = "discourse.org|wikipedia.org"
|
|
|
|
expect(group.valid?).to eq true
|
|
|
|
end
|
2015-12-07 19:39:28 +08:00
|
|
|
|
2016-04-09 00:11:58 +08:00
|
|
|
it "is valid for newer TLDs" do
|
|
|
|
group.automatic_membership_email_domains = "discourse.institute"
|
|
|
|
expect(group.valid?).to eq true
|
|
|
|
end
|
|
|
|
|
2015-12-07 19:39:28 +08:00
|
|
|
it "is invalid for bad incoming email" do
|
|
|
|
group.incoming_email = "foo.bar.org"
|
|
|
|
expect(group.valid?).to eq(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "is valid for proper incoming email" do
|
|
|
|
group.incoming_email = "foo@bar.org"
|
|
|
|
expect(group.valid?).to eq(true)
|
|
|
|
end
|
2013-05-09 15:37:34 +08:00
|
|
|
end
|
|
|
|
|
2013-09-06 12:07:23 +08:00
|
|
|
def real_admins
|
|
|
|
Group[:admins].user_ids - [-1]
|
|
|
|
end
|
|
|
|
|
|
|
|
def real_moderators
|
|
|
|
Group[:moderators].user_ids - [-1]
|
|
|
|
end
|
|
|
|
|
|
|
|
def real_staff
|
|
|
|
Group[:staff].user_ids - [-1]
|
|
|
|
end
|
|
|
|
|
2015-04-10 10:17:28 +08:00
|
|
|
it "Correctly handles primary groups" do
|
|
|
|
group = Fabricate(:group, primary_group: true)
|
|
|
|
user = Fabricate(:user)
|
|
|
|
|
|
|
|
group.add(user)
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.primary_group_id).to eq group.id
|
|
|
|
|
|
|
|
group.remove(user)
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.primary_group_id).to eq nil
|
|
|
|
|
|
|
|
group.add(user)
|
|
|
|
group.primary_group = false
|
|
|
|
group.save
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.primary_group_id).to eq nil
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
it "Correctly handles title" do
|
|
|
|
|
|
|
|
group = Fabricate(:group, title: 'Super Awesome')
|
|
|
|
user = Fabricate(:user)
|
|
|
|
|
|
|
|
expect(user.title).to eq nil
|
|
|
|
|
|
|
|
group.add(user)
|
|
|
|
user.reload
|
|
|
|
|
|
|
|
expect(user.title).to eq 'Super Awesome'
|
|
|
|
|
|
|
|
group.title = 'BOOM'
|
|
|
|
group.save
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.title).to eq 'BOOM'
|
|
|
|
|
|
|
|
group.title = nil
|
|
|
|
group.save
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.title).to eq nil
|
|
|
|
|
|
|
|
group.title = "BOB"
|
|
|
|
group.save
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.title).to eq "BOB"
|
|
|
|
|
|
|
|
group.remove(user)
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.title).to eq nil
|
|
|
|
|
|
|
|
group.add(user)
|
|
|
|
group.destroy
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.title).to eq nil
|
|
|
|
|
|
|
|
end
|
|
|
|
|
2016-10-24 10:03:22 +08:00
|
|
|
it "makes sure the everyone group is not visible" do
|
|
|
|
g = Group.refresh_automatic_group!(:everyone)
|
|
|
|
expect(g.visible).to eq(false)
|
|
|
|
end
|
|
|
|
|
2015-04-10 10:17:28 +08:00
|
|
|
it "Correctly handles removal of primary group" do
|
2015-02-09 13:03:09 +08:00
|
|
|
group = Fabricate(:group)
|
|
|
|
user = Fabricate(:user)
|
|
|
|
group.add(user)
|
|
|
|
group.save
|
|
|
|
|
|
|
|
user.primary_group = group
|
|
|
|
user.save
|
|
|
|
|
|
|
|
group.reload
|
|
|
|
|
|
|
|
group.remove(user)
|
|
|
|
group.save
|
|
|
|
|
|
|
|
user.reload
|
|
|
|
expect(user.primary_group).to eq nil
|
|
|
|
end
|
|
|
|
|
2013-05-06 12:49:56 +08:00
|
|
|
it "Can update moderator/staff/admin groups correctly" do
|
2013-05-09 15:37:34 +08:00
|
|
|
|
2013-05-06 12:49:56 +08:00
|
|
|
admin = Fabricate(:admin)
|
|
|
|
moderator = Fabricate(:moderator)
|
|
|
|
|
|
|
|
Group.refresh_automatic_groups!(:admins, :staff, :moderators)
|
|
|
|
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(real_admins).to eq [admin.id]
|
|
|
|
expect(real_moderators).to eq [moderator.id]
|
|
|
|
expect(real_staff.sort).to eq [moderator.id,admin.id].sort
|
2013-05-06 12:49:56 +08:00
|
|
|
|
|
|
|
admin.admin = false
|
|
|
|
admin.save
|
|
|
|
|
|
|
|
Group.refresh_automatic_group!(:admins)
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(real_admins).to be_empty
|
2013-05-06 12:49:56 +08:00
|
|
|
|
|
|
|
moderator.revoke_moderation!
|
|
|
|
|
|
|
|
admin.grant_admin!
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(real_admins).to eq [admin.id]
|
|
|
|
expect(real_staff).to eq [admin.id]
|
2013-05-06 12:49:56 +08:00
|
|
|
|
|
|
|
admin.revoke_admin!
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(real_admins).to be_empty
|
|
|
|
expect(real_staff).to be_empty
|
2013-05-06 12:49:56 +08:00
|
|
|
|
|
|
|
admin.grant_moderation!
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(real_moderators).to eq [admin.id]
|
|
|
|
expect(real_staff).to eq [admin.id]
|
2013-05-06 12:49:56 +08:00
|
|
|
|
|
|
|
admin.revoke_moderation!
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(real_admins).to be_empty
|
|
|
|
expect(real_staff).to eq []
|
2013-05-06 12:49:56 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it "Correctly updates automatic trust level groups" do
|
|
|
|
user = Fabricate(:user)
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(Group[:trust_level_0].user_ids).to include user.id
|
2014-06-17 08:46:30 +08:00
|
|
|
|
2014-09-05 13:20:39 +08:00
|
|
|
user.change_trust_level!(TrustLevel[1])
|
2013-05-06 12:49:56 +08:00
|
|
|
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(Group[:trust_level_1].user_ids).to include user.id
|
2013-05-06 12:49:56 +08:00
|
|
|
|
2014-09-05 13:20:39 +08:00
|
|
|
user.change_trust_level!(TrustLevel[2])
|
2013-05-06 12:49:56 +08:00
|
|
|
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(Group[:trust_level_1].user_ids).to include user.id
|
|
|
|
expect(Group[:trust_level_2].user_ids).to include user.id
|
2013-05-08 08:32:29 +08:00
|
|
|
|
|
|
|
user2 = Fabricate(:coding_horror)
|
2014-09-05 13:20:39 +08:00
|
|
|
user2.change_trust_level!(TrustLevel[3])
|
2013-05-08 08:32:29 +08:00
|
|
|
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(Group[:trust_level_2].user_ids.sort).to eq [-1, user.id, user2.id].sort
|
2013-05-06 12:49:56 +08:00
|
|
|
end
|
|
|
|
|
2013-05-08 13:20:38 +08:00
|
|
|
it "Correctly updates all automatic groups upon request" do
|
2013-06-25 22:15:41 +08:00
|
|
|
Fabricate(:admin)
|
2013-05-08 13:20:38 +08:00
|
|
|
user = Fabricate(:user)
|
2014-09-05 13:20:39 +08:00
|
|
|
user.change_trust_level!(TrustLevel[2])
|
2013-05-08 13:20:38 +08:00
|
|
|
|
|
|
|
Group.exec_sql("update groups set user_count=0 where id = #{Group::AUTO_GROUPS[:trust_level_2]}")
|
|
|
|
|
|
|
|
Group.refresh_automatic_groups!
|
|
|
|
|
|
|
|
groups = Group.includes(:users).to_a
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(groups.count).to eq Group::AUTO_GROUPS.count
|
2013-05-08 13:20:38 +08:00
|
|
|
|
2016-04-09 00:11:58 +08:00
|
|
|
g = groups.find{|grp| grp.id == Group::AUTO_GROUPS[:admins]}
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(g.users.count).to eq 2
|
|
|
|
expect(g.user_count).to eq 2
|
2013-05-08 13:20:38 +08:00
|
|
|
|
2016-04-09 00:11:58 +08:00
|
|
|
g = groups.find{|grp| grp.id == Group::AUTO_GROUPS[:staff]}
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(g.users.count).to eq 2
|
|
|
|
expect(g.user_count).to eq 2
|
2014-06-17 08:46:30 +08:00
|
|
|
|
2016-04-09 00:11:58 +08:00
|
|
|
g = groups.find{|grp| grp.id == Group::AUTO_GROUPS[:trust_level_1]}
|
2014-06-17 08:46:30 +08:00
|
|
|
# admin, system and user
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(g.users.count).to eq 3
|
|
|
|
expect(g.user_count).to eq 3
|
2014-06-17 08:46:30 +08:00
|
|
|
|
2016-04-09 00:11:58 +08:00
|
|
|
g = groups.find{|grp| grp.id == Group::AUTO_GROUPS[:trust_level_2]}
|
2014-06-17 08:46:30 +08:00
|
|
|
# system and user
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(g.users.count).to eq 2
|
|
|
|
expect(g.user_count).to eq 2
|
2013-05-08 13:20:38 +08:00
|
|
|
|
|
|
|
end
|
|
|
|
|
2013-05-09 09:33:56 +08:00
|
|
|
it "can set members via usernames helper" do
|
|
|
|
g = Fabricate(:group)
|
|
|
|
u1 = Fabricate(:user)
|
|
|
|
u2 = Fabricate(:user)
|
|
|
|
u3 = Fabricate(:user)
|
|
|
|
|
|
|
|
g.add(u1)
|
|
|
|
g.save!
|
|
|
|
|
|
|
|
usernames = "#{u2.username},#{u3.username}"
|
|
|
|
|
|
|
|
# no side effects please
|
|
|
|
g.usernames = usernames
|
|
|
|
g.reload
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(g.users.count).to eq 1
|
2013-05-09 09:33:56 +08:00
|
|
|
|
|
|
|
g.usernames = usernames
|
|
|
|
g.save!
|
|
|
|
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(g.usernames.split(",").sort).to eq usernames.split(",").sort
|
2013-05-09 09:33:56 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it "correctly destroys groups" do
|
2013-11-18 23:52:01 +08:00
|
|
|
|
2013-05-09 09:33:56 +08:00
|
|
|
g = Fabricate(:group)
|
|
|
|
u1 = Fabricate(:user)
|
|
|
|
g.add(u1)
|
|
|
|
g.save!
|
|
|
|
|
|
|
|
g.destroy
|
|
|
|
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(User.where(id: u1.id).count).to eq 1
|
|
|
|
expect(GroupUser.where(group_id: g.id).count).to eq 0
|
2013-05-09 09:33:56 +08:00
|
|
|
end
|
|
|
|
|
2014-04-25 21:14:05 +08:00
|
|
|
|
|
|
|
it "has custom fields" do
|
|
|
|
group = Fabricate(:group)
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(group.custom_fields["a"]).to be_nil
|
2014-04-25 21:14:05 +08:00
|
|
|
|
|
|
|
group.custom_fields["hugh"] = "jackman"
|
|
|
|
group.custom_fields["jack"] = "black"
|
|
|
|
group.save
|
|
|
|
|
|
|
|
group = Group.find(group.id)
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(group.custom_fields).to eq({"hugh" => "jackman", "jack" => "black"})
|
2014-04-25 21:14:05 +08:00
|
|
|
end
|
|
|
|
|
2013-07-23 08:10:36 +08:00
|
|
|
it "allows you to lookup a new group by name" do
|
|
|
|
group = Fabricate(:group)
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(group.id).to eq Group[group.name].id
|
|
|
|
expect(group.id).to eq Group[group.name.to_sym].id
|
2013-07-23 08:10:36 +08:00
|
|
|
end
|
|
|
|
|
2014-06-17 08:46:30 +08:00
|
|
|
it "can find desired groups correctly" do
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(Group.desired_trust_level_groups(2).sort).to eq [10,11,12]
|
2014-06-17 08:46:30 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it "correctly handles trust level changes" do
|
|
|
|
user = Fabricate(:user, trust_level: 2)
|
|
|
|
Group.user_trust_level_change!(user.id, 2)
|
|
|
|
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(user.groups.map(&:name).sort).to eq ["trust_level_0","trust_level_1", "trust_level_2"]
|
2014-06-17 08:46:30 +08:00
|
|
|
|
|
|
|
Group.user_trust_level_change!(user.id, 0)
|
|
|
|
user.reload
|
2014-12-31 22:55:03 +08:00
|
|
|
expect(user.groups.map(&:name).sort).to eq ["trust_level_0"]
|
2014-06-17 08:46:30 +08:00
|
|
|
end
|
|
|
|
|
2015-01-09 07:35:52 +08:00
|
|
|
context "group management" do
|
|
|
|
let(:group) {Fabricate(:group)}
|
|
|
|
|
|
|
|
it "by default has no managers" do
|
2015-11-09 21:55:16 +08:00
|
|
|
expect(group.group_users.where('group_users.owner')).to be_empty
|
2015-01-09 07:35:52 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it "multiple managers can be appointed" do
|
|
|
|
2.times do |i|
|
|
|
|
u = Fabricate(:user)
|
2015-11-09 21:55:16 +08:00
|
|
|
group.add_owner(u)
|
2015-01-09 07:35:52 +08:00
|
|
|
end
|
2015-11-09 21:55:16 +08:00
|
|
|
expect(group.group_users.where('group_users.owner').count).to eq(2)
|
2015-01-09 07:35:52 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it "manager has authority to edit membership" do
|
|
|
|
u = Fabricate(:user)
|
|
|
|
expect(Guardian.new(u).can_edit?(group)).to be_falsy
|
2015-11-09 21:55:16 +08:00
|
|
|
group.add_owner(u)
|
2015-01-09 07:35:52 +08:00
|
|
|
expect(Guardian.new(u).can_edit?(group)).to be_truthy
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-09-02 04:52:05 +08:00
|
|
|
it "correctly grants a trust level to members" do
|
|
|
|
group = Fabricate(:group, grant_trust_level: 2)
|
|
|
|
u0 = Fabricate(:user, trust_level: 0)
|
|
|
|
u3 = Fabricate(:user, trust_level: 3)
|
|
|
|
|
|
|
|
group.add(u0)
|
|
|
|
expect(u0.reload.trust_level).to eq(2)
|
|
|
|
|
|
|
|
group.add(u3)
|
|
|
|
expect(u3.reload.trust_level).to eq(3)
|
|
|
|
end
|
|
|
|
|
2016-12-05 16:18:24 +08:00
|
|
|
it 'should cook the bio' do
|
|
|
|
group = Fabricate(:group)
|
|
|
|
group.update_attributes!(bio_raw: 'This is a group for :unicorn: lovers')
|
|
|
|
|
|
|
|
expect(group.bio_cooked).to include("unicorn.png")
|
|
|
|
end
|
|
|
|
|
2013-04-29 14:33:24 +08:00
|
|
|
end
|