diff --git a/scripts/src/java/org/oppia/android/scripts/common/BazelClient.kt b/scripts/src/java/org/oppia/android/scripts/common/BazelClient.kt index 37b5fe2adb9..44227f614c5 100644 --- 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? { -// 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 index ca71a382896..dd5800b6291 100644 --- 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 index b217eeae829..9df8aa67a90 100644 --- 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 :| * */