mirror of
https://gh.wpcy.net/https://github.com/discourse/discourse.git
synced 2026-05-04 15:13:41 +08:00
4e304e7130
This commit implements a date-time range selector to staff action log in admin page, making it easier to filter and analyze the data. Additionally, it allows to apply the date-time picker to the exported file as well, reducing the exported file size.
99 lines
2.9 KiB
Ruby
99 lines
2.9 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
RSpec.describe UserHistory do
|
|
describe "#actions" do
|
|
context "when verifying enum sequence" do
|
|
let!(:actions) { described_class.actions }
|
|
|
|
it "'delete_user' should be at 1st position" do
|
|
expect(actions[:delete_user]).to eq(1)
|
|
end
|
|
|
|
it "'change_site_text' should be at 29th position" do
|
|
expect(actions[:change_site_text]).to eq(29)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "#staff_action_records" do
|
|
context "with some records" do
|
|
fab!(:admin)
|
|
|
|
let(:custom_type) { "confirmed_ham" }
|
|
let!(:change_site_setting) do
|
|
UserHistory.create!(
|
|
action: UserHistory.actions[:change_site_setting],
|
|
subject: "title",
|
|
previous_value: "Old",
|
|
new_value: "New",
|
|
)
|
|
end
|
|
let!(:change_trust_level) do
|
|
UserHistory.create!(
|
|
action: UserHistory.actions[:change_trust_level],
|
|
target_user_id: Fabricate(:user).id,
|
|
details: "stuff happened",
|
|
)
|
|
end
|
|
let!(:custom_history) do
|
|
StaffActionLogger.new(admin).log_custom("confirmed_ham", admin_only: true)
|
|
end
|
|
|
|
it "returns all records for admins" do
|
|
records = described_class.staff_action_records(admin).to_a
|
|
expect(records.size).to eq(3)
|
|
end
|
|
|
|
it "doesn't return records to moderators that only admins should see" do
|
|
records = described_class.staff_action_records(Fabricate(:moderator)).to_a
|
|
expect(records).not_to include([change_site_setting])
|
|
end
|
|
|
|
it "filters by action" do
|
|
records =
|
|
described_class.staff_action_records(
|
|
admin,
|
|
action_id: change_site_setting.action_before_type_cast,
|
|
).to_a
|
|
expect(records.size).to eq(1)
|
|
expect(records.first).to eq(change_site_setting)
|
|
end
|
|
|
|
it "filters by action_name" do
|
|
records =
|
|
described_class.staff_action_records(admin, action_name: "change_site_setting").to_a
|
|
expect(records.size).to eq(1)
|
|
expect(records.first).to eq(change_site_setting)
|
|
end
|
|
|
|
it "Uses action_name as custom_type when searching for custom_staff logs" do
|
|
records =
|
|
described_class.staff_action_records(
|
|
admin,
|
|
action_name: custom_type,
|
|
action_id: described_class.actions[:custom_staff],
|
|
).to_a
|
|
|
|
expect(records.size).to eq(1)
|
|
expect(records.first).to eq(custom_history)
|
|
end
|
|
|
|
it "filters by start and/or end date" do
|
|
freeze_time
|
|
|
|
10.times do |i|
|
|
Fabricate(
|
|
:user_history,
|
|
action: UserHistory.actions[:suspend_user],
|
|
created_at: i.days.ago,
|
|
)
|
|
end
|
|
|
|
records =
|
|
described_class.staff_action_records(admin, start_date: 7.days.ago, end_date: 2.days.ago)
|
|
|
|
expect(records.size).to eq(7 - 2 + 1)
|
|
end
|
|
end
|
|
end
|
|
end
|
