Cleanup of RunCoverageTest

oppia · Jun 20, 2024 · 8e28f1d · 8e28f1d
1 parent fdaf9cb
commit 8e28f1d
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 46 deletions.
diff --git a/scripts/src/java/org/oppia/android/scripts/coverage/RunCoverage.kt b/scripts/src/java/org/oppia/android/scripts/coverage/RunCoverage.kt
@@ -80,15 +80,11 @@ class RunCoverage(
     val isExempted = testFileExemptionList.contains(filePath)
     if (isExempted) {
       println("This file is exempted from having a test file. Hence No coverage!")
-//      return null
       return emptyList()
     }
 
-//    val bazelClient = BazelClient(rootDirectory, commandExecutor)
     val testFilePaths = findTestFile(repoRoot, filePath)
-//    return findTestFile(repoRoot, filePath)
     val testTargets = bazelClient.retrieveBazelTargets(testFilePaths)
-//    val testTargets = bazelClient.retrieveBazelTargets(testFilePaths)
 
     for (testTarget in testTargets) {
       val coverageData = RunCoverageForTestTarget(
@@ -100,17 +96,6 @@ class RunCoverage(
       coverageDataList.add(coverageData)
     }
     return coverageDataList
-
-    //this works
-    /*val covdat = RunCoverageForTestTarget(
-      rootDirectory,
-      "//coverage/test/java/com/example:test",
-      commandExecutor,
-      scriptBgDispatcher
-    ).runCoverage()
-    coverageDataList.add(covdat!!)
-    return coverageDataList*/
-//    return covdat
   }
 
   fun findTestFile(repoRoot: String, filePath: String): List<String> {

diff --git a/scripts/src/javatests/org/oppia/android/scripts/coverage/RunCoverageTest.kt b/scripts/src/javatests/org/oppia/android/scripts/coverage/RunCoverageTest.kt
@@ -222,28 +222,6 @@ class RunCoverageTest {
       longCommandExecutor,
       scriptBgDispatcher).execute()
 
-    /*val expectedResult =
-      "["+"SF:coverage/main/java/com/example/TwoSum.kt\n" +
-        "FN:7,com/example/TwoSum${'$'}Companion::sumNumbers (II)Ljava/lang/Object;\n" +
-        "FN:3,com/example/TwoSum::<init> ()V\n" +
-        "FNDA:1,com/example/TwoSum${'$'}Companion::sumNumbers (II)Ljava/lang/Object;\n" +
-        "FNDA:0,com/example/TwoSum::<init> ()V\n" +
-        "FNF:2\n" +
-        "FNH:1\n" +
-        "BRDA:7,0,0,1\n" +
-        "BRDA:7,0,1,1\n" +
-        "BRDA:7,0,2,1\n" +
-        "BRDA:7,0,3,1\n" +
-        "BRF:4\n" +
-        "BRH:4\n" +
-        "DA:3,0\n" +
-        "DA:7,1\n" +
-        "DA:8,1\n" +
-        "DA:10,1\n" +
-        "LH:3\n" +
-        "LF:4\n" +
-        "end_of_record\n"+"]"*/
-
       val expectedResultList = listOf(
         "SF:coverage/main/java/com/example/TwoSum.kt\n"+
         "FN:7,com/example/TwoSum${'$'}Companion::sumNumbers (II)Ljava/lang/Object;\n"+
@@ -275,13 +253,4 @@ class RunCoverageTest {
       scriptBgDispatcher, processTimeout = 5, processTimeoutUnit = TimeUnit.MINUTES
     )
   }
-
-  /** Runs the run_coverage. */
-  private fun runScript(filePath: String) {
-    RunCoverage(
-      "${tempFolder.root}",
-      filePath,
-      commandExecutor,
-      scriptBgDispatcher).execute()
-  }
 }