From fa5c61e441b88204dbcee334ebf4478b9f0949c5 Mon Sep 17 00:00:00 2001
From: Simon Lecoq <22963968+lowlighter@users.noreply.github.com>
Date: Tue, 21 May 2024 07:37:36 -0400
Subject: [PATCH] fix(cli/coverage): invalid line id in html reporter (#23908)

---
 cli/tools/coverage/reporter.rs      | 2 +-
 tests/integration/coverage_tests.rs | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/cli/tools/coverage/reporter.rs b/cli/tools/coverage/reporter.rs
index 6547f2036d..f359755a92 100644
--- a/cli/tools/coverage/reporter.rs
+++ b/cli/tools/coverage/reporter.rs
@@ -633,7 +633,7 @@ impl HtmlCoverageReporter {
   ) -> String {
     let line_num = file_text.lines().count();
     let line_count = (1..line_num + 1)
-      .map(|i| format!("<a name='L{i}'></a><a href='#{i}'>{i}</a>"))
+      .map(|i| format!("<a name='L{i}'></a><a href='#L{i}'>{i}</a>"))
       .collect::<Vec<_>>()
       .join("\n");
     let line_coverage = (0..line_num)
diff --git a/tests/integration/coverage_tests.rs b/tests/integration/coverage_tests.rs
index 6e9a1454ca..ae732f25f3 100644
--- a/tests/integration/coverage_tests.rs
+++ b/tests/integration/coverage_tests.rs
@@ -486,6 +486,9 @@ fn test_html_reporter() {
   assert_contains!(bar_ts_html, "<h1>Coverage report for bar.ts</h1>");
   // Check <T> in source code is escaped to &lt;T&gt;
   assert_contains!(bar_ts_html, "&lt;T&gt;");
+  // Check that line anchors are correctly referenced by line number links
+  assert_contains!(bar_ts_html, "<a name='L1'></a>");
+  assert_contains!(bar_ts_html, "<a href='#L1'>1</a>");
 
   let baz_index_html = tempdir
     .join("html")