Merge pull request rails#52108 from fresh-eggs/7-2-stable

[`7-2-stable` Backport] Fix action-text-attachment HTML escaping regression test

Author: rafaelfranca

actiontext/test/unit/attachment_test.rb

@@ -79,13 +79,10 @@ class ActionText::AttachmentTest < ActiveSupport::TestCase
     end
   end
 
-  test "sanitizes HTML content attachment" do
-    attachment = attachment_from_html('<action-text-attachment content-type="text/html" content="<img src=\&quot;.\&quot; onerror=alert>"></action-text-attachment>')
-    attachable = attachment.attachable
+  test "to_trix_html sanitizes action-text HTML content attachment" do
+    attachment = ActionText::Content.new("<action-text-attachment content-type=\"text/html\" content=\"<img src=. onerror='alert(location)' />\"></action-text-attachment>")
 
-    ActionText::Content.with_renderer MessagesController.renderer do
-      assert_equal "<img src=\"\\%22.\\%22\">", attachable.to_html.strip
-    end
+    assert_equal "<figure data-trix-attachment=\"{&quot;contentType&quot;:&quot;text/html&quot;,&quot;content&quot;:&quot;<img src=\\&quot;.\\&quot;>&quot;}\"></figure>", attachment.to_trix_html
   end
 
   test "defaults trix partial to model partial" do