oppia · seanlip · Jun 28, 2024 · Jun 4, 2024 · Jun 5, 2024 · Jun 5, 2024
@@ -25,3 +25,4 @@ bazel-*
 .bazelproject
 .aswb
 *.pb
+coverage_reports
@@ -12,6 +12,7 @@ kt_jvm_library(
     ],
     visibility = ["//scripts:oppia_script_binary_visibility"],
     deps = [
+        ":coverage_reporter",
         ":coverage_runner",
         "//scripts/src/java/org/oppia/android/scripts/common:bazel_client",
         "//scripts/src/java/org/oppia/android/scripts/proto:script_exemptions_java_proto",
@@ -30,3 +31,16 @@ kt_jvm_library(
         "//scripts/src/java/org/oppia/android/scripts/proto:coverage_java_proto",
     ],
 )
+
+kt_jvm_library(
+    name = "coverage_reporter",
+    testonly = True,
+    srcs = [
+        "CoverageReporter.kt",
+    ],
+    visibility = ["//scripts:oppia_script_binary_visibility"],
+    deps = [
+        "//scripts/src/java/org/oppia/android/scripts/common:bazel_client",
+        "//scripts/src/java/org/oppia/android/scripts/proto:coverage_java_proto",
+    ],
+)
@@ -0,0 +1,198 @@
+package org.oppia.android.scripts.coverage
+
+import org.oppia.android.scripts.proto.Coverage
+import org.oppia.android.scripts.proto.CoverageReport
+import java.io.File
+
+/**
+ * Class responsible for generating rich text coverage report.
+ *
+ * @param repoRoot the root directory of the repository
+ * @param coverageReportList the list of coverage data proto
+ * @param reportFormat the format in which the report will be generated
+ */
+class CoverageReporter(
+  private val repoRoot: String,
+  private val coverageReportList: List<CoverageReport>,
+  private val reportFormat: ReportFormat,
+) {
+  private val computedCoverageRatio = computeCoverageRatio()
+  private val formattedCoveragePercentage = "%.2f".format(computedCoverageRatio * 100)
+
+  private val filePath = coverageReportList.firstOrNull()?.filePath ?: "Unknown"
+
+  private val totalLinesFound = coverageReportList.getOrNull(0)?.linesFound ?: 0
+  private val totalLinesHit = coverageReportList.getOrNull(0)?.linesHit ?: 0
+
+  /**
+   * Generates a rich text report for the analysed coverage data based on the specified format.
+   * It supports Markdown and HTML formats.
+   *
+   * @return a pair where the first value is the computed coverage ratio represented in [0, 1]
+   *     and the second value is the generated report text
+   */
+  fun generateRichTextReport(): Pair<Float, String> {
+    println("report format: $reportFormat")
+    return when (reportFormat) {
+      ReportFormat.MARKDOWN -> generateMarkdownReport()
+      ReportFormat.HTML -> generateHtmlReport()
+    }
+  }
+
+  private fun generateMarkdownReport(): Pair<Float, String> {
+    val markdownContent =
+      """
+        ## Coverage Report
+
+        - **Covered File:** $filePath
+        - **Coverage percentage:** $formattedCoveragePercentage% covered
+        - **Line coverage:** $totalLinesHit / $totalLinesFound lines covered
+      """.trimIndent()
+
+    println("\n$markdownContent")
+
+    return Pair(computedCoverageRatio, markdownContent)
+  }
+
+  private fun generateHtmlReport(): Pair<Float, String> {
+    var htmlContent =
+      """
+    <!DOCTYPE html>
+    <html lang="en">
+    <head>
+      <meta charset="UTF-8">
+      <meta name="viewport" content="width=device-width, initial-scale=1.0">
+      <title>Coverage Report</title>
+      <style>
+        body {
+            font-family: Arial, sans-serif;
+            line-height: 1.6;
+            padding: 20px;
+        }
+        table {
+            width: 100%;
+            border-collapse: collapse;
+            margin-bottom: 20px;
+        }
+        th, td {
+            padding: 8px;
+            text-align: left;
+            border-bottom: 1px solid #fdfdfd;
+        }
+        .line-number-col {
+            width: 5%;
+        }
+        .source-code-col {
+            width: 95%;
+        }
+        .covered-line, .not-covered-line, .uncovered-line {
+            white-space: pre-wrap;
+            word-wrap: break-word;
+            box-sizing: border-box;
+            border-radius: 4px;
+            padding: 2px 8px 2px 4px;
+            display: inline-block;
+        }
+        .covered-line {
+            background-color: #c8e6c9; /* Light green */
+        }
+        .not-covered-line {
+            background-color: #ffcdd2; /* Light red */
+        }
+        .uncovered-line {
+            background-color: #fafafa; /* Half white */
+        }
+        .coverage-summary {
+            margin-bottom: 20px;
+        }
+        h2 {
+            text-align: center;
+        }
+        ul {
+            list-style-type: none;
+            padding: 0;
+            text-align: center;
+        }
+        .summary-box {
+            background-color: #f0f0f0;
+            border: 1px solid #ccc;
+            border-radius: 8px;
+            padding: 10px;
+            margin-bottom: 20px;
+            text-align: center;
+        }
+        @media screen and (max-width: 768px) {
+            body {
+                padding: 10px;
+            }
+            table {
+                width: auto;
+            }
+        }
+      </style>
+    </head>
+    <body>
+      <h2>Coverage Report</h2>
+      <div class="summary-box">
+        <ul>
+          <li><strong>Covered File:</strong> $filePath</li>
+          <li><strong>Coverage percentage:</strong> $formattedCoveragePercentage% covered</li>
+          <li><strong>Line coverage:</strong> $totalLinesHit covered / $totalLinesFound found</li>
+        </ul>
+      </div>
+      <table>
+        <thead>
+          <tr>
+            <th class="line-number-col">Line No</th>
+            <th class="source-code-col">Source Code</th>
+          </tr>
+        </thead>
+        <tbody>
+      """.trimIndent()
+
+    val fileContent = File(repoRoot, filePath).readLines()
+    val coverageMap = coverageReportList
+      .firstOrNull()?.coveredLineList?.associateBy { it.lineNumber }
+
+    fileContent.forEachIndexed { index, line ->
+      val lineNumber = index + 1
+      val lineClass = when (coverageMap?.get(lineNumber)?.coverage) {
+        Coverage.FULL -> "covered-line"
+        Coverage.NONE -> "not-covered-line"
+        else -> "uncovered-line"
+      }
+      htmlContent += """
+        <tr>
+            <td>${lineNumber.toString().padStart(4, ' ')}</td>
+            <td class="$lineClass">$line</td>
+        </tr>
+      """.trimIndent()
+    }
+
+    htmlContent += """
+        </tbody>
+      </table>
+    </body>
+    </html>
+    """.trimIndent()
+
+    return Pair(computedCoverageRatio, htmlContent)
+  }
+
+  private fun computeCoverageRatio(): Float {
+    val report = coverageReportList.getOrNull(0)
+    return if (report != null && report.linesFound != 0) {
+      report.linesHit.toFloat() / report.linesFound.toFloat()
+    } else {
+      0f
+    }
+  }
+}
+
+/** Represents the different types of formats available to generate code coverage reports. */
+enum class ReportFormat {
+  /** Indicates that the report should be formatted in .md format. */
+  MARKDOWN,
+  /** Indicates that the report should be formatted in .html format. */
+  HTML
+}
@@ -104,7 +104,10 @@ class CoverageRunner(
 }
 
 private fun extractTargetName(bazelTestTarget: String): String {
-  val targetName = bazelTestTarget.substringAfterLast(":").trim()
+  val targetName = bazelTestTarget
+    .substringAfterLast("/")
+    .substringAfterLast(":")
+    .trim()
   return targetName.removeSuffix("LocalTest").removeSuffix("Test")
 }
 

@@ -19,10 +19,12 @@ import java.util.concurrent.TimeUnit
  * Arguments:
  * - path_to_root: directory path to the root of the Oppia Android repository.
  * - relative_path_to_file: the relative path to the file to analyse coverage
+ * - reportFormat: the format of the coverage report. Defaults to MARKDOWN if not specified.
+ *   Available options: MARKDOWN, HTML.
  *
  * Example:
  *     bazel run //scripts:run_coverage -- $(pwd)
- *     utility/src/main/java/org/oppia/android/util/parser/math/MathModel.kt
+ *     utility/src/main/java/org/oppia/android/util/parser/math/MathModel.kt HTML
  * Example with custom process timeout:
  *     bazel run //scripts:run_coverage -- $(pwd)
  *     utility/src/main/java/org/oppia/android/util/parser/math/MathModel.kt processTimeout=10
@@ -31,6 +33,13 @@ import java.util.concurrent.TimeUnit
 fun main(vararg args: String) {
   val repoRoot = args[0]
   val filePath = args[1]
+  val reportFormat = when (args.getOrNull(2)) {
+    "HTML" -> ReportFormat.HTML
+    "MARKDOWN", null -> ReportFormat.MARKDOWN
+    else -> throw IllegalArgumentException("Unsupported report format: ${args[2]}")
+  }
+
+  val reportOutputPath = getReportOutputPath(repoRoot, filePath, reportFormat)
 
   if (!File(repoRoot, filePath).exists()) {
     error("File doesn't exist: $filePath.")
@@ -45,7 +54,14 @@ fun main(vararg args: String) {
       scriptBgDispatcher, processTimeout = processTimeout, processTimeoutUnit = TimeUnit.MINUTES
     )
 
-    RunCoverage(repoRoot, filePath, commandExecutor, scriptBgDispatcher).execute()
+    RunCoverage(
+      repoRoot,
+      filePath,
+      reportFormat,
+      reportOutputPath,
+      commandExecutor,
+      scriptBgDispatcher
+    ).execute()
   }
 }
 
@@ -60,6 +76,8 @@ fun main(vararg args: String) {
 class RunCoverage(
   private val repoRoot: String,
   private val filePath: String,
+  private val reportFormat: ReportFormat,
+  private val reportOutputPath: String,
   private val commandExecutor: CommandExecutor,
   private val scriptBgDispatcher: ScriptBackgroundCoroutineDispatcher
 ) {
@@ -79,15 +97,14 @@ class RunCoverage(
    * @return a list of lists containing coverage data for each requested test target, if
    *     the file is exempted from having a test file, an empty list is returned
    */
-  fun execute(): List<CoverageReport> {
+  fun execute(): String {
     val testFileExemptionList = loadTestFileExemptionsProto(testFileExemptionTextProto)
       .testFileExemptionList
       .filter { it.testFileNotRequired }
       .map { it.exemptedFilePath }
 
     if (filePath in testFileExemptionList) {
-      println("This file is exempted from having a test file; skipping coverage check.")
-      return emptyList()
+      return "This file is exempted from having a test file; skipping coverage check."
     }
 
     val testFilePaths = findTestFile(repoRoot, filePath)
@@ -97,9 +114,26 @@ class RunCoverage(
 
     val testTargets = bazelClient.retrieveBazelTargets(testFilePaths)
 
-    return testTargets.mapNotNull { testTarget ->
+    val coverageReports = testTargets.mapNotNull { testTarget ->
       runCoverageForTarget(testTarget)
     }
+
+    coverageReports.takeIf { it.isNotEmpty() }?.run {
+      val reporter = CoverageReporter(repoRoot, this, reportFormat)
+      val (computedCoverageRatio, reportText) = reporter.generateRichTextReport()
+
+      File(reportOutputPath).apply {
+        parentFile?.mkdirs()
+        writeText(reportText)
+      }
+
+      if (File(reportOutputPath).exists()) {
+        println("\nComputed Coverage Ratio is: $computedCoverageRatio")
+        println("\nGenerated report at: $reportOutputPath\n")
+      }
+    } ?: println("No coverage reports generated.")
+
+    return reportOutputPath
   }
 
   private fun runCoverageForTarget(testTarget: String): CoverageReport {
@@ -136,6 +170,19 @@ private fun findTestFile(repoRoot: String, filePath: String): List<String> {
     .map { it.relativeTo(repoRootFile).path }
 }
 
+private fun getReportOutputPath(
+  repoRoot: String,
+  filePath: String,
+  reportFormat: ReportFormat
+): String {
+  val fileWithoutExtension = filePath.substringBeforeLast(".")
+  val defaultFilename = when (reportFormat) {
+    ReportFormat.HTML -> "coverage.html"
+    ReportFormat.MARKDOWN -> "coverage.md"
+  }
+  return "$repoRoot/coverage_reports/$fileWithoutExtension/$defaultFilename"
+}
+
 private fun loadTestFileExemptionsProto(testFileExemptiontextProto: String): TestFileExemptions {
   return File("$testFileExemptiontextProto.pb").inputStream().use { stream ->
     TestFileExemptions.newBuilder().also { builder ->

@@ -4,6 +4,19 @@ Tests corresponding to developer scripts that help with obtaining coverage data
 
 load("@io_bazel_rules_kotlin//kotlin:jvm.bzl", "kt_jvm_test")
 
+kt_jvm_test(
+    name = "RunCoverageTest",
+    srcs = ["RunCoverageTest.kt"],
+    deps = [
+        "//scripts:test_file_check_assets",
+        "//scripts/src/java/org/oppia/android/scripts/coverage:run_coverage_lib",
+        "//scripts/src/java/org/oppia/android/scripts/testing:test_bazel_workspace",
+        "//testing:assertion_helpers",
+        "//third_party:com_google_truth_truth",
+        "//third_party:org_jetbrains_kotlin_kotlin-test-junit",
+    ],
+)
+
 kt_jvm_test(
     name = "CoverageRunnerTest",
     srcs = ["CoverageRunnerTest.kt"],
@@ -17,12 +30,10 @@ kt_jvm_test(
 )
 
 kt_jvm_test(
-    name = "RunCoverageTest",
-    srcs = ["RunCoverageTest.kt"],
+    name = "CoverageReporterTest",
+    srcs = ["CoverageReporterTest.kt"],
     deps = [
-        "//scripts:test_file_check_assets",
-        "//scripts/src/java/org/oppia/android/scripts/coverage:run_coverage_lib",
-        "//scripts/src/java/org/oppia/android/scripts/testing:test_bazel_workspace",
+        "//scripts/src/java/org/oppia/android/scripts/coverage:coverage_reporter",
         "//testing:assertion_helpers",
         "//third_party:com_google_truth_truth",
         "//third_party:org_jetbrains_kotlin_kotlin-test-junit",
-Original file line number
+Diff line change
@@ Expand Up / @@ -25,3 +25,4 @@ bazel-* @@
     .bazelproject
     .aswb
     *.pb
+    coverage_reports