FIX: include silence_reason when admin sees user (#31493)

Small regression created in this PR
https://github.com/discourse/discourse/pull/30635

The reason is included only when `include_silence_reason` option is
included.
This commit is contained in:
Krzysztof Kotlarek 2025-02-25 14:14:33 +11:00 committed by GitHub
parent f591083407
commit f1a892a64e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 17 additions and 0 deletions

View File

@ -50,6 +50,7 @@ class Admin::UsersController < Admin::StaffController
AdminDetailedUserSerializer,
root: false,
similar_users_count: @user.similar_users.count,
include_silence_reason: true,
)
end

View File

@ -109,6 +109,22 @@ RSpec.describe Admin::UsersController do
end
end
it "returns silence reason when user is silenced" do
silencer =
UserSilencer.new(
user,
admin,
message: :too_many_spam_flags,
reason: "because I said so",
keep_posts: true,
)
silencer.silence
get "/admin/users/#{user.id}.json"
expect(response.status).to eq(200)
expect(response.parsed_body["silence_reason"]).to eq("because I said so")
end
context "with a non-existing user" do
it "returns 404 error" do
get "/admin/users/0.json"