fixes for offline XML parsing and verbose logging

Author: mrfyda

build.sbt

@@ -2,7 +2,7 @@ import Dependencies._
 
 name := "sbt-codacy-coverage"
 
-version := "1.0.0"
+version := "1.0.1"
 
 scalaVersion := "2.10.4"
 

project/Dependencies.scala

@@ -2,9 +2,9 @@ import sbt._
 
 object Dependencies {
 
-  lazy val playJson = "com.typesafe.play" %% "play-json" % "2.3.6"
-  lazy val playWs = "com.typesafe.play" %% "play-ws" % "2.3.6"
-  lazy val jGit = "org.eclipse.jgit" % "org.eclipse.jgit" % "3.4.1.201406201815-r"
-  lazy val scalaTest = "org.scalatest" %% "scalatest" % "2.2.1" % "test"
+  lazy val playJson = "com.typesafe.play" %% "play-json" % "2.3.7"
+  lazy val playWs = "com.typesafe.play" %% "play-ws" % "2.3.7"
+  lazy val jGit = "org.eclipse.jgit" % "org.eclipse.jgit" % "3.6.2.201501210735-r"
+  lazy val scalaTest = "org.scalatest" %% "scalatest" % "2.2.4" % "test"
 
 }

project/build.properties

@@ -1 +1 @@
-sbt.version=0.13.6
+sbt.version=0.13.7

project/plugins.sbt

@@ -1,9 +1,5 @@
 import sbt._
 
-ivyLoggingLevel := UpdateLogging.Full
-
-logLevel := sbt.Level.Error
-
 resolvers ++= Seq(
   DefaultMavenRepository,
   "Typesafe Repository" at "http://repo.typesafe.com/typesafe/releases/",
@@ -12,12 +8,8 @@ resolvers ++= Seq(
   Classpaths.sbtPluginReleases
 )
 
-addSbtPlugin("com.github.mpeltonen" % "sbt-idea" % "1.6.0")
-
-addSbtPlugin("com.typesafe.sbteclipse" % "sbteclipse-plugin" % "2.4.0")
-
 addSbtPlugin("org.xerial.sbt" % "sbt-sonatype" % "0.2.1")
 
 addSbtPlugin("com.typesafe.sbt" % "sbt-pgp" % "0.8.3")
 
-addSbtPlugin("com.timushev.sbt" % "sbt-updates" % "0.1.6")
+addSbtPlugin("com.timushev.sbt" % "sbt-updates" % "0.1.8")

src/main/resources/logback-test.xml

@@ -0,0 +1,14 @@
+<configuration>
+
+    <appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
+        <!-- encoders are assigned the type
+             ch.qos.logback.classic.encoder.PatternLayoutEncoder by default -->
+        <encoder>
+            <pattern>%d{HH:mm:ss.SSS} [%thread] %-5level %logger{36} - %msg%n</pattern>
+        </encoder>
+    </appender>
+
+    <root level="error">
+        <appender-ref ref="STDOUT"/>
+    </root>
+</configuration>

src/main/resources/logback.xml

@@ -4,7 +4,21 @@ import java.io.File
 
 import com.codacy.api.{CodacyCoverageFileReport, CodacyCoverageReport}
 
-import scala.xml.{Node, XML}
+import scala.xml.factory.XMLLoader
+import scala.xml.{SAXParser, Elem, Node}
+
+object XML extends XMLLoader[Elem] {
+  override def parser: SAXParser = {
+    val f = javax.xml.parsers.SAXParserFactory.newInstance()
+    f.setNamespaceAware(false)
+    f.setValidating(false)
+    f.setFeature("http://xml.org/sax/features/namespaces", false)
+    f.setFeature("http://xml.org/sax/features/validation", false)
+    f.setFeature("http://apache.org/xml/features/nonvalidating/load-dtd-grammar", false)
+    f.setFeature("http://apache.org/xml/features/nonvalidating/load-external-dtd", false)
+    f.newSAXParser()
+  }
+}
 
 class CoberturaParser(coberturaFile: File, rootProject: File) {
 

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

@@ -8,7 +8,7 @@ import scala.collection.mutable.ArrayBuffer
 
 class CoberturaReaderTest extends WordSpec with BeforeAndAfterAll with Matchers {
 
-  val reader = new CoberturaParser(new File("src/test/resources/test_cobertura.xml"), new File(""), new File(""))
+  val reader = new CoberturaParser(new File("src/test/resources/test_cobertura.xml"), new File(""))
 
   "CoberturaReader" when {
     "reading a Cobertura file" should {
@@ -19,7 +19,7 @@ class CoberturaReaderTest extends WordSpec with BeforeAndAfterAll with Matchers
         fileReport.fileReports.map(_.filename).head should endWith("src/test/resources/TestSourceFile.scala")
         fileReport.fileReports.map(_.total).head should equal(87)
         fileReport.fileReports.map(_.coverage) should equal(
-          ArrayBuffer(Map("4" -> 1, "9" -> 1, "5" -> 1, "10" -> 1, "6" -> 2), Map("1" -> 1, "2" -> 1, "3" -> 1))
+          ArrayBuffer(Map(4 -> 1, 9 -> 1, 5 -> 1, 10 -> 1, 6 -> 2), Map(1 -> 1, 2 -> 1, 3 -> 1))
         )
       }