Skip to content
This repository was archived by the owner on Dec 3, 2020. It is now read-only.

Commit 3dc318c

Browse files
author
Daniel Reigada
committed
Do not filter languages with unknown extensions
1 parent b2cdeb4 commit 3dc318c

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

src/main/scala/com/codacy/parsers/implementation/CoberturaParser.scala

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,23 @@
11
package com.codacy.parsers.implementation
22

33
import java.io.File
4-
import java.util.Locale
54
import java.text.NumberFormat
5+
import java.util.Locale
66

77
import com.codacy.api.{CoverageFileReport, CoverageReport, Language}
88
import com.codacy.parsers.XMLCoverageParser
99
import com.codacy.parsers.util.LanguageUtils
1010

11-
import scala.xml.Node
1211
import scala.util.Try
12+
import scala.xml.Node
1313

1414
class CoberturaParser(val language: Language.Value, val rootProject: File, val coverageReport: File) extends XMLCoverageParser {
1515

1616
val rootProjectDir = sanitiseFilename(rootProject.getAbsolutePath + File.separator)
17-
lazy val allFiles = recursiveListFiles(rootProject)(f => f.getName.endsWith(LanguageUtils.getExtension(language)))
17+
18+
lazy val allFiles = recursiveListFiles(rootProject) { file =>
19+
LanguageUtils.getExtension(language).fold(true)(file.getName.endsWith(_))
20+
}.map(file => sanitiseFilename(file.getAbsolutePath))
1821

1922
private[this] def convertToFloat(str: String): Try[Float] = {
2023
Try(str.toFloat).recoverWith {
@@ -79,7 +82,7 @@ class CoberturaParser(val language: Language.Value, val rootProject: File, val c
7982
key -> value
8083
}
8184

82-
allFiles.map(f => sanitiseFilename(f.getAbsolutePath)).find(f => f.endsWith(sourceFilename)).map {
85+
allFiles.find(f => f.endsWith(sourceFilename)).map {
8386
filename =>
8487
CoverageFileReport(stripRoot(filename), fileHit, lineHitMap)
8588
}

src/main/scala/com/codacy/parsers/util/LanguageUtils.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ import com.codacy.api.Language
44

55
object LanguageUtils {
66

7-
def getExtension(language: Language.Value): String = {
8-
language match {
7+
def getExtension(language: Language.Value): Option[String] = {
8+
Option(language).collect {
99
case Language.Java => ".java"
1010
case Language.Python => ".py"
1111
case Language.Scala => ".scala"

0 commit comments

Comments
 (0)