diff --git a/app/models/forum_post.rb b/app/models/forum_post.rb index d5e4eeb65..ac7e8dbd4 100644 --- a/app/models/forum_post.rb +++ b/app/models/forum_post.rb @@ -205,7 +205,7 @@ class ForumPost < ApplicationRecord end end - def dtext_shortlink + def dtext_shortlink(**options) "forum ##{id}" end diff --git a/test/functional/moderation_reports_controller_test.rb b/test/functional/moderation_reports_controller_test.rb index b4491a969..c10a6bdff 100644 --- a/test/functional/moderation_reports_controller_test.rb +++ b/test/functional/moderation_reports_controller_test.rb @@ -7,8 +7,10 @@ class ModerationReportsControllerTest < ActionDispatch::IntegrationTest @spammer = create(:user, created_at: 2.weeks.ago) @mod = create(:moderator_user, created_at: 2.weeks.ago) - @spammer.as_current do + as(@spammer) do @comment = create(:comment, creator: @spammer) + @forum_topic = create(:forum_topic, creator: @spammer) + @forum_post = create(:forum_post, topic: @forum_topic, creator: @spammer) end end @@ -50,12 +52,19 @@ class ModerationReportsControllerTest < ActionDispatch::IntegrationTest end context "create action" do - should "create a new moderation report" do + should "create a new moderation report on a comment" do assert_difference("ModerationReport.count", 1) do post_auth moderation_reports_path, @user, params: {:format => "js", :moderation_report => {:model_id => @comment.id, :model_type => "Comment", :reason => "xxx"}} assert_response :success end end + + should "create a new moderation report on a forum post" do + assert_difference("ModerationReport.count", 1) do + post_auth moderation_reports_path, @user, params: { format: "js", moderation_report: { model_id: @forum_post.id, model_type: "ForumPost", reason: "xxx" }} + assert_response :success + end + end end end end