From 3aecc36b3ebd9dd72fd45967e7c6c91d3afcfa71 Mon Sep 17 00:00:00 2001 From: Earlopain <14981592+Earlopain@users.noreply.github.com> Date: Tue, 15 Aug 2023 14:13:58 +0200 Subject: [PATCH] [Users] Fix exception when refreshing counts --- app/models/user.rb | 2 +- test/unit/user_test.rb | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/app/models/user.rb b/app/models/user.rb index 7262d7192..330ba8384 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -760,7 +760,7 @@ class User < ApplicationRecord note_count: NoteVersion.for_user(id).count, own_post_replaced_count: PostReplacement.for_uploader_on_approve(id).count, own_post_replaced_penalize_count: PostReplacement.penalized.for_uploader_on_approve(id).count, - post_replacement_rejected_count: post_replacements.rejected.count.count, + post_replacement_rejected_count: post_replacements.rejected.count, ) end end diff --git a/test/unit/user_test.rb b/test/unit/user_test.rb index 3277c663b..8f5e9e2bd 100644 --- a/test/unit/user_test.rb +++ b/test/unit/user_test.rb @@ -303,5 +303,11 @@ class UserTest < ActiveSupport::TestCase assert_equal([user3.id], User.search(name_matches: "bar\*baz").map(&:id)) end end + + context "when fixing counts" do + should "not raise" do + assert_nothing_raised { @user.refresh_counts! } + end + end end end