mirror of
https://github.com/discourse/discourse.git
synced 2024-11-29 05:03:58 +08:00
57ea56ee05
We have all these calls to Group.refresh_automatic_groups! littered throughout the tests. Including tests that are seemingly unrelated to groups. This is because automatic group memberships aren't fabricated when making a vanilla user. There are two places where you'd want to use this: You have fabricated a user that needs a certain trust level (which is now based on group membership.) You need the system user to have a certain trust level. In the first case, we can pass refresh_auto_groups: true to the fabricator instead. This is a more lightweight operation that only considers a single user, instead of all users in all groups. The second case is no longer a thing after #25400.
47 lines
1.3 KiB
Ruby
47 lines
1.3 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
RSpec.describe GroupArchivedMessage do
|
|
fab!(:user)
|
|
fab!(:user_2) { Fabricate(:user) }
|
|
|
|
fab!(:group) do
|
|
Fabricate(:group, messageable_level: Group::ALIAS_LEVELS[:everyone]).tap { |g| g.add(user_2) }
|
|
end
|
|
|
|
fab!(:group_message) do
|
|
create_post(
|
|
user: user,
|
|
target_group_names: [group.name],
|
|
archetype: Archetype.private_message,
|
|
).topic
|
|
end
|
|
|
|
describe ".move_to_inbox!" do
|
|
it "should unarchive the topic correctly" do
|
|
described_class.archive!(group.id, group_message)
|
|
|
|
messages =
|
|
MessageBus.track_publish(PrivateMessageTopicTrackingState.group_channel(group.id)) do
|
|
described_class.move_to_inbox!(group.id, group_message)
|
|
end
|
|
|
|
expect(messages.present?).to eq(true)
|
|
|
|
expect(GroupArchivedMessage.exists?(topic: group_message, group: group)).to eq(false)
|
|
end
|
|
end
|
|
|
|
describe ".archive!" do
|
|
it "should archive the topic correctly" do
|
|
messages =
|
|
MessageBus.track_publish(PrivateMessageTopicTrackingState.group_channel(group.id)) do
|
|
described_class.archive!(group.id, group_message)
|
|
end
|
|
|
|
expect(GroupArchivedMessage.exists?(topic: group_message, group: group)).to eq(true)
|
|
|
|
expect(messages.present?).to eq(true)
|
|
end
|
|
end
|
|
end
|