Code cleanup and fix lint checks for missing spaces

oppia · Jul 6, 2024 · 45067f7 · 45067f7
1 parent 5f9ae7a
commit 45067f7
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/scripts/src/java/org/oppia/android/scripts/common/BazelClient.kt b/scripts/src/java/org/oppia/android/scripts/common/BazelClient.kt
@@ -142,11 +142,9 @@ class BazelClient(private val rootDirectory: File, private val commandExecutor:
    *     or null if the coverage data file could not be parsed
    */
   fun runCoverageForTestTarget(bazelTestTarget: String): List<String>? {
-//    val computeInstrumentation = bazelTestTarget.split("/").let { "//${it[2]}/..." }
     val instrumentation = bazelTestTarget.split(":")[0]
-    val computeInstrumentation = instrumentation.split("/").let {"//${it[2]}/..."}
+    val computeInstrumentation = instrumentation.split("/").let { "//${it[2]}/..." }
     println("compute: $computeInstrumentation")
-//    val computeInstrumentation = "//"
     val coverageCommandOutputLines = executeBazelCommand(
       "coverage",
       bazelTestTarget,

diff --git a/scripts/src/java/org/oppia/android/scripts/coverage/CoverageReporter.kt b/scripts/src/java/org/oppia/android/scripts/coverage/CoverageReporter.kt
@@ -213,7 +213,6 @@ class CoverageReporter(
   }
 
   private fun computeCoverageRatio(): Float {
-//    val report = coverageReportList.getOrNull(0)
     return if (coverageReport.linesFound != 0) {
       coverageReport.linesHit.toFloat() / coverageReport.linesFound.toFloat()
     } else {

diff --git a/scripts/src/java/org/oppia/android/scripts/coverage/RunCoverage.kt b/scripts/src/java/org/oppia/android/scripts/coverage/RunCoverage.kt
@@ -116,16 +116,14 @@ class RunCoverage(
       }
 
     val testFilePaths = findTestFile(repoRoot, filePath)
-    println("Test file paths: $testFilePaths")
     if (testFilePaths.isEmpty()) {
       error("No appropriate test file found for $filePath")
     }
 
     val testTargets = bazelClient.retrieveBazelTargets(testFilePaths)
-    println("Test Targets: $testTargets")
 
     /*since I couldn't actually find any multi test target : file ones to test
-    * I am probably for now introducing mock data to test multi aggregated coverage report
+    * I am for now introducing mock data to test multi aggregated coverage report
     * also that's going to save me a light year :|
     * */