mirror of
https://github.com/discourse/discourse.git
synced 2024-11-23 06:04:11 +08:00
6e161d3e75
The most common thing that we do with fab! is:
fab!(:thing) { Fabricate(:thing) }
This commit adds a shorthand for this which is just simply:
fab!(:thing)
i.e. If you omit the block, then, by default, you'll get a `Fabricate`d object using the fabricator of the same name.
112 lines
3.4 KiB
Ruby
112 lines
3.4 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
RSpec.describe GroupActionLogger do
|
|
subject(:logger) { described_class.new(group_owner, group) }
|
|
|
|
fab!(:group_owner) { Fabricate(:user) }
|
|
fab!(:group)
|
|
fab!(:user)
|
|
|
|
before { group.add_owner(group_owner) }
|
|
|
|
describe "#log_make_user_group_owner" do
|
|
it "should create the right record" do
|
|
logger.log_make_user_group_owner(user)
|
|
|
|
group_history = GroupHistory.last
|
|
|
|
expect(group_history.action).to eq(GroupHistory.actions[:make_user_group_owner])
|
|
expect(group_history.acting_user).to eq(group_owner)
|
|
expect(group_history.target_user).to eq(user)
|
|
end
|
|
end
|
|
|
|
describe "#log_remove_user_as_group_owner" do
|
|
it "should create the right record" do
|
|
logger.log_remove_user_as_group_owner(user)
|
|
|
|
group_history = GroupHistory.last
|
|
|
|
expect(group_history.action).to eq(GroupHistory.actions[:remove_user_as_group_owner])
|
|
expect(group_history.acting_user).to eq(group_owner)
|
|
expect(group_history.target_user).to eq(user)
|
|
end
|
|
end
|
|
|
|
describe "#log_add_user_to_group" do
|
|
context "as a group owner" do
|
|
it "should create the right record" do
|
|
logger.log_add_user_to_group(user)
|
|
|
|
group_history = GroupHistory.last
|
|
|
|
expect(group_history.action).to eq(GroupHistory.actions[:add_user_to_group])
|
|
expect(group_history.acting_user).to eq(group_owner)
|
|
expect(group_history.target_user).to eq(user)
|
|
end
|
|
end
|
|
|
|
context "as a normal user" do
|
|
subject(:logger) { described_class.new(user, group) }
|
|
|
|
before { group.update!(public_admission: true) }
|
|
|
|
it "should create the right record" do
|
|
logger.log_add_user_to_group(user)
|
|
|
|
group_history = GroupHistory.last
|
|
|
|
expect(group_history.action).to eq(GroupHistory.actions[:add_user_to_group])
|
|
expect(group_history.acting_user).to eq(user)
|
|
expect(group_history.target_user).to eq(user)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "#log_remove_user_from_group" do
|
|
context "as group owner" do
|
|
it "should create the right record" do
|
|
logger.log_remove_user_from_group(user)
|
|
|
|
group_history = GroupHistory.last
|
|
|
|
expect(group_history.action).to eq(GroupHistory.actions[:remove_user_from_group])
|
|
expect(group_history.acting_user).to eq(group_owner)
|
|
expect(group_history.target_user).to eq(user)
|
|
end
|
|
end
|
|
|
|
context "as a normal user" do
|
|
subject(:logger) { described_class.new(user, group) }
|
|
|
|
before { group.update!(public_exit: true) }
|
|
|
|
it "should create the right record" do
|
|
logger.log_remove_user_from_group(user)
|
|
|
|
group_history = GroupHistory.last
|
|
|
|
expect(group_history.action).to eq(GroupHistory.actions[:remove_user_from_group])
|
|
expect(group_history.acting_user).to eq(user)
|
|
expect(group_history.target_user).to eq(user)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "#log_change_group_settings" do
|
|
it "should create the right record" do
|
|
group.update!(public_admission: true, created_at: Time.zone.now)
|
|
|
|
expect { logger.log_change_group_settings }.to change { GroupHistory.count }.by(1)
|
|
|
|
group_history = GroupHistory.last
|
|
|
|
expect(group_history.action).to eq(GroupHistory.actions[:change_group_setting])
|
|
expect(group_history.acting_user).to eq(group_owner)
|
|
expect(group_history.subject).to eq("public_admission")
|
|
expect(group_history.prev_value).to eq("f")
|
|
expect(group_history.new_value).to eq("t")
|
|
end
|
|
end
|
|
end
|