Merge pull request #2556 from ViewComponent/fix-segfault-compiled-source

fix bug where part of template annotation line was not stripped

Author: joelhawksley

docs/CHANGELOG.md

@@ -10,9 +10,9 @@ nav_order: 6
 
 ## main
 
-* Fix segfault when Ruby coverage is enabled but template annotations are disabled.
+* Fix segfaults when Ruby coverage is enabled.
 
-    *George Holborn*
+    *George Holborn*, *Joel Hawksley*
 
 * Add `protocol` parameter to `with_request_url` test helper to enable testing with HTTPS protocol.
 

lib/view_component/template.rb

@@ -65,7 +65,7 @@ def compiled_source
         result = super
         # Strip the annotation line to maintain correct line numbers when coverage
         # is running (avoids segfault from negative lineno)
-        result = result.sub(/\A[^\n]*\n/, "") if @strip_annotation_line
+        result = result.partition(";").last if @strip_annotation_line
         result
       end
     end

test/sandbox/test/mailer_test.rb

@@ -6,7 +6,6 @@ class MailerTest < ActionMailer::TestCase
   def test_rendering_component_in_an_action_mailer
     result = TestMailer.test_email.deliver_now.body.to_s
     assert_includes result, "<div>Hello world!</div>"
-    assert_includes result, "test_email.html.erb"
   end
 
   def test_rendering_component_in_an_action_mailer_with_asset_component