From cd10039c4478b5d48807dbde8f76144430196b10 Mon Sep 17 00:00:00 2001
From: Rd <rd4ramadevi@gmail.com>
Date: Tue, 27 Aug 2024 21:11:02 +0530
Subject: [PATCH] Updated the RunCoverageTest cases to take unescaped Html
 Report text

---
 .../coverage/reporter/CoverageReporter.kt     |  3 +-
 .../scripts/coverage/RunCoverageTest.kt       | 86 ++++++++++++++++---
 .../coverage/reporter/CoverageReporterTest.kt |  8 +-
 3 files changed, 84 insertions(+), 13 deletions(-)

diff --git a/scripts/src/java/org/oppia/android/scripts/coverage/reporter/CoverageReporter.kt b/scripts/src/java/org/oppia/android/scripts/coverage/reporter/CoverageReporter.kt
index 31763392945..f447a58c256 100644
--- a/scripts/src/java/org/oppia/android/scripts/coverage/reporter/CoverageReporter.kt
+++ b/scripts/src/java/org/oppia/android/scripts/coverage/reporter/CoverageReporter.kt
@@ -1,5 +1,6 @@
 package org.oppia.android.scripts.coverage.reporter
 
+import com.google.common.html.HtmlEscapers
 import org.oppia.android.scripts.proto.Coverage
 import org.oppia.android.scripts.proto.CoverageReport
 import org.oppia.android.scripts.proto.CoverageReportContainer
@@ -277,7 +278,7 @@ class CoverageReporter(
                 """
                   <tr>
                     <td class="line-number-row">${lineNumber.toString().padStart(4, ' ')}</td>
-                    <td class="$lineClass">$line</td>
+                    <td class="$lineClass">${HtmlEscapers.htmlEscaper().escape(line)}</td>
                   </tr>
                 """.trimIndent()
               )
diff --git a/scripts/src/javatests/org/oppia/android/scripts/coverage/RunCoverageTest.kt b/scripts/src/javatests/org/oppia/android/scripts/coverage/RunCoverageTest.kt
index 7ef0183f00d..3d28a816c7d 100644
--- a/scripts/src/javatests/org/oppia/android/scripts/coverage/RunCoverageTest.kt
+++ b/scripts/src/javatests/org/oppia/android/scripts/coverage/RunCoverageTest.kt
@@ -317,7 +317,13 @@ class RunCoverageTest {
 
     val expectedResult = getExpectedHtmlText(kotlinFilePath)
 
-    assertThat(readHtmlReport(kotlinFilePath)).isEqualTo(expectedResult)
+    val unescapedHtmlReport = readHtmlReport(kotlinFilePath)
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedHtmlReport).isEqualTo(expectedResult)
   }
 
   @Test
@@ -342,7 +348,13 @@ class RunCoverageTest {
 
     val expectedResult = getExpectedHtmlText(sourceFilePath)
 
-    assertThat(readHtmlReport(sourceFilePath)).isEqualTo(expectedResult)
+    val unescapedHtmlReport = readHtmlReport(sourceFilePath)
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedHtmlReport).isEqualTo(expectedResult)
   }
 
   @Test
@@ -541,7 +553,13 @@ class RunCoverageTest {
 
     val expectedResult = getExpectedHtmlText(filePath)
 
-    assertThat(readHtmlReport(filePath)).isEqualTo(expectedResult)
+    val unescapedHtmlReport = readHtmlReport(filePath)
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedHtmlReport).isEqualTo(expectedResult)
   }
 
   @Test
@@ -1880,10 +1898,20 @@ class RunCoverageTest {
     ).execute()
 
     val expectedResult1 = getExpectedHtmlText(filePathList.get(0))
-    assertThat(readHtmlReport(filePathList.get(0))).isEqualTo(expectedResult1)
+    val unescapedHtmlReport1 = readHtmlReport(filePathList.get(0))
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+    assertThat(unescapedHtmlReport1).isEqualTo(expectedResult1)
 
     val expectedResult2 = getExpectedHtmlText(filePathList.get(1))
-    assertThat(readHtmlReport(filePathList.get(1))).isEqualTo(expectedResult2)
+    val unescapedHtmlReport2 = readHtmlReport(filePathList.get(1))
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+    assertThat(unescapedHtmlReport2).isEqualTo(expectedResult2)
   }
 
   @Test
@@ -1910,7 +1938,13 @@ class RunCoverageTest {
 
     val expectedResult = getExpectedHtmlText(filePathList.get(0))
 
-    assertThat(readHtmlReport(filePathList.get(0))).isEqualTo(expectedResult)
+    val unescapedHtmlReport = readHtmlReport(filePathList.get(0))
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedHtmlReport).isEqualTo(expectedResult)
   }
 
   @Test
@@ -1937,7 +1971,13 @@ class RunCoverageTest {
 
     val expectedResult = getExpectedHtmlText(filePathList.get(0))
 
-    assertThat(readHtmlReport(filePathList.get(0))).isEqualTo(expectedResult)
+    val unescapedHtmlReport = readHtmlReport(filePathList.get(0))
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedHtmlReport).isEqualTo(expectedResult)
   }
 
   @Test
@@ -1964,7 +2004,13 @@ class RunCoverageTest {
 
     val expectedResult = getExpectedHtmlText(filePathList.get(0))
 
-    assertThat(readHtmlReport(filePathList.get(0))).isEqualTo(expectedResult)
+    val unescapedHtmlReport = readHtmlReport(filePathList.get(0))
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedHtmlReport).isEqualTo(expectedResult)
   }
 
   @Test
@@ -2009,7 +2055,13 @@ class RunCoverageTest {
 
     val expectedResult = getExpectedHtmlText(filePathList.get(0))
 
-    assertThat(readHtmlReport(filePathList.get(0))).isEqualTo(expectedResult)
+    val unescapedHtmlReport = readHtmlReport(filePathList.get(0))
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedHtmlReport).isEqualTo(expectedResult)
   }
 
   @Test
@@ -2036,7 +2088,13 @@ class RunCoverageTest {
 
     val expectedResult = getExpectedHtmlText(filePathList.get(0))
 
-    assertThat(readHtmlReport(filePathList.get(0))).isEqualTo(expectedResult)
+    val unescapedHtmlReport = readHtmlReport(filePathList.get(0))
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedHtmlReport).isEqualTo(expectedResult)
   }
 
   @Test
@@ -2262,7 +2320,13 @@ class RunCoverageTest {
     </html>
       """.trimIndent()
 
-    assertThat(readHtmlReport(filePathList.get(0))).isEqualTo(expectedResult)
+    val unescapedHtmlReport = readHtmlReport(filePathList.get(0))
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedHtmlReport).isEqualTo(expectedResult)
   }
 
   @Test
diff --git a/scripts/src/javatests/org/oppia/android/scripts/coverage/reporter/CoverageReporterTest.kt b/scripts/src/javatests/org/oppia/android/scripts/coverage/reporter/CoverageReporterTest.kt
index b0af56a1115..ae06e6a693e 100644
--- a/scripts/src/javatests/org/oppia/android/scripts/coverage/reporter/CoverageReporterTest.kt
+++ b/scripts/src/javatests/org/oppia/android/scripts/coverage/reporter/CoverageReporterTest.kt
@@ -697,7 +697,13 @@ class CoverageReporterTest {
     </html>
       """.trimIndent()
 
-    assertThat(outputReportText).isEqualTo(expectedHtml)
+    val unescapedOutputReportText = outputReportText
+      .replace("&quot;", "\"")
+      .replace("&amp;", "&")
+      .replace("&lt;", "<")
+      .replace("&gt;", ">")
+
+    assertThat(unescapedOutputReportText).isEqualTo(expectedHtml)
   }
 
   @Test