1 files changed, 8 insertions, 12 deletions
diff --git a/test/unit/lib/redmine/reaction_test.rb b/test/unit/lib/redmine/reaction_test.rb
index bed4600d0..f3228a3bd 100644
--- a/test/unit/lib/redmine/reaction_test.rb
+++ b/test/unit/lib/redmine/reaction_test.rb
@@ -42,7 +42,6 @@ class Redmine::ReactionTest < ActiveSupport::TestCase
     Issue.preload_reaction_details([issue1, issue2])
 
     expected_issue1_reaction_detail = Reaction::Detail.new(
-      reaction_count: 3,
       visible_users: [users(:users_003), users(:users_002), users(:users_001)],
       user_reaction: reactions(:reaction_002)
     )
@@ -53,7 +52,6 @@ class Redmine::ReactionTest < ActiveSupport::TestCase
 
       # Even when an object has no reactions, an empty ReactionDetail is set.
       assert_equal Reaction::Detail.new(
-        reaction_count: 0,
         visible_users: [],
         user_reaction: nil
       ), issue2.reaction_detail
@@ -107,7 +105,6 @@ class Redmine::ReactionTest < ActiveSupport::TestCase
     assert_nil message7.instance_variable_get(:@reaction_detail)
 
     assert_equal Reaction::Detail.new(
-      reaction_count: 1,
       visible_users: [users(:users_002)],
       user_reaction: reactions(:reaction_009)
     ), message7.reaction_detail
@@ -122,7 +119,6 @@ class Redmine::ReactionTest < ActiveSupport::TestCase
     comment1.load_reaction_detail
 
     assert_equal Reaction::Detail.new(
-      reaction_count: 1,
       visible_users: [users(:users_002)],
       user_reaction: nil
     ), comment1.reaction_detail
@@ -153,7 +149,7 @@ class Redmine::ReactionTest < ActiveSupport::TestCase
     assert_not Redmine::Reaction.visible?(object, user)
   end
 
-  test 'writable? returns true for various reactable objects when user is logged in, object is visible, and project is active' do
+  test 'editable? returns true for various reactable objects when user is logged in, object is visible, and project is active' do
     reactable_objects = {
       issue: issues(:issues_007),
       message: messages(:messages_001),
@@ -164,30 +160,30 @@ class Redmine::ReactionTest < ActiveSupport::TestCase
     user = users(:users_002)
 
     reactable_objects.each do |type, object|
-      assert Redmine::Reaction.writable?(object, user), "Expected writable? to return true for #{type}"
+      assert Redmine::Reaction.editable?(object, user), "Expected editable? to return true for #{type}"
     end
   end
 
-  test 'writable? returns false when user is not logged in' do
+  test 'editable? returns false when user is not logged in' do
     object = issues(:issues_007)
     user = User.anonymous
 
-    assert_not Redmine::Reaction.writable?(object, user)
+    assert_not Redmine::Reaction.editable?(object, user)
   end
 
-  test 'writable? returns false when project is inactive' do
+  test 'editable? returns false when project is inactive' do
     object = issues(:issues_007)
     user = users(:users_002)
     object.project.update!(status: Project::STATUS_ARCHIVED)
 
-    assert_not Redmine::Reaction.writable?(object, user)
+    assert_not Redmine::Reaction.editable?(object, user)
   end
 
-  test 'writable? returns false when project is closed' do
+  test 'editable? returns false when project is closed' do
     object = issues(:issues_007)
     user = users(:users_002)
     object.project.update!(status: Project::STATUS_CLOSED)
 
-    assert_not Redmine::Reaction.writable?(object, user)
+    assert_not Redmine::Reaction.editable?(object, user)
   end
 end