Replace SLF4J with Sonar logging

Author: RadoBuransky

plugin/src/main/scala/com/buransky/plugins/scoverage/sensor/ScoverageSensor.scala

@@ -26,13 +26,13 @@ import com.buransky.plugins.scoverage.measure.ScalaMetrics
 import com.buransky.plugins.scoverage.util.LogUtil
 import com.buransky.plugins.scoverage.xml.XmlScoverageReportParser
 import com.buransky.plugins.scoverage.{CoveredStatement, DirectoryStatementCoverage, FileStatementCoverage, _}
-import org.slf4j.LoggerFactory
 import org.sonar.api.batch.fs.{FileSystem, InputFile}
 import org.sonar.api.batch.{CoverageExtension, Sensor, SensorContext}
 import org.sonar.api.config.Settings
 import org.sonar.api.measures.{CoreMetrics, CoverageMeasuresBuilder, Measure}
 import org.sonar.api.resources.{File, Project, Resource}
 import org.sonar.api.scan.filesystem.PathResolver
+import org.sonar.api.utils.log.Loggers
 
 import scala.collection.JavaConversions._
 
@@ -43,7 +43,7 @@ import scala.collection.JavaConversions._
  */
 class ScoverageSensor(settings: Settings, pathResolver: PathResolver, fileSystem: FileSystem, scala: Scala)
   extends Sensor with CoverageExtension {
-  private val log = LoggerFactory.getLogger(classOf[ScoverageSensor])
+  private val log = Loggers.get(classOf[ScoverageSensor])
   protected val SCOVERAGE_REPORT_PATH_PROPERTY = "sonar.scoverage.reportPath"
   protected lazy val scoverageReportParser: ScoverageReportParser = XmlScoverageReportParser()
 

plugin/src/main/scala/com/buransky/plugins/scoverage/xml/XmlScoverageReportConstructingParser.scala

@@ -19,23 +19,25 @@
  */
 package com.buransky.plugins.scoverage.xml
 
+import java.io.File
+
 import com.buransky.plugins.scoverage._
+import com.buransky.plugins.scoverage.util.PathUtil
+import org.sonar.api.utils.log.Loggers
+
+import scala.annotation.tailrec
+import scala.collection.mutable
 import scala.io.Source
 import scala.xml.parsing.ConstructingParser
-import scala.xml.{Text, NamespaceBinding, MetaData}
-import org.apache.log4j.Logger
-import scala.collection.mutable
-import scala.annotation.tailrec
-import java.io.File
-import com.buransky.plugins.scoverage.util.PathUtil
+import scala.xml.{MetaData, NamespaceBinding, Text}
 
 /**
  * Scoverage XML parser based on ConstructingParser provided by standard Scala library.
  *
  * @author Rado Buransky
  */
 class XmlScoverageReportConstructingParser(source: Source) extends ConstructingParser(source, false) {
-  private val log = Logger.getLogger(classOf[XmlScoverageReportConstructingParser])
+  private val log = Loggers.get(classOf[XmlScoverageReportConstructingParser])
 
   private val CLASS_ELEMENT = "class"
   private val FILENAME_ATTRIBUTE = "filename"

plugin/src/main/scala/com/buransky/plugins/scoverage/xml/XmlScoverageReportParser.scala

@@ -19,18 +19,19 @@
  */
 package com.buransky.plugins.scoverage.xml
 
-import scala.io.Source
-import com.buransky.plugins.scoverage.{ProjectStatementCoverage, ScoverageReportParser, ScoverageException}
-import org.apache.log4j.Logger
 import com.buransky.plugins.scoverage.util.LogUtil
+import com.buransky.plugins.scoverage.{ProjectStatementCoverage, ScoverageException, ScoverageReportParser}
+import org.sonar.api.utils.log.Loggers
+
+import scala.io.Source
 
 /**
  * Bridge between parser implementation and coverage provider.
  *
  * @author Rado Buransky
  */
 class XmlScoverageReportParser extends ScoverageReportParser {
-  private val log = Logger.getLogger(classOf[XmlScoverageReportParser])
+  private val log = Loggers.get(classOf[XmlScoverageReportParser])
 
   def parse(reportFilePath: String): ProjectStatementCoverage = {
     require(reportFilePath != null)

plugin/src/test/scala/com/buransky/plugins/scoverage/sensor/TestSensorContext.scala

@@ -23,8 +23,15 @@ import java.lang.Double
 import java.util.Date
 import java.{io, util}
 
-import org.sonar.api.batch.fs.{InputFile, InputPath}
-import org.sonar.api.batch.{Event, SensorContext}
+import org.sonar.api.batch.fs.{FileSystem, InputFile, InputPath}
+import org.sonar.api.batch.rule.ActiveRules
+import org.sonar.api.batch.sensor.dependency.NewDependency
+import org.sonar.api.batch.sensor.duplication.NewDuplication
+import org.sonar.api.batch.sensor.highlighting.NewHighlighting
+import org.sonar.api.batch.sensor.issue.NewIssue
+import org.sonar.api.batch.sensor.measure.NewMeasure
+import org.sonar.api.batch.{AnalysisMode, Event, SensorContext}
+import org.sonar.api.config.Settings
 import org.sonar.api.design.Dependency
 import org.sonar.api.measures.{Measure, MeasuresFilter, Metric}
 import org.sonar.api.resources.{ProjectLink, Resource}
@@ -102,4 +109,22 @@ class TestSensorContext extends SensorContext {
   override def saveMeasure(inputFile: InputFile, metric: Metric[_ <: io.Serializable], value: Double): Measure[_ <: io.Serializable] = ???
 
   override def saveMeasure(inputFile: InputFile, measure: Measure[_ <: io.Serializable]): Measure[_ <: io.Serializable] = ???
+
+  override def newDuplication(): NewDuplication = ???
+
+  override def activeRules(): ActiveRules = ???
+
+  override def newHighlighting(): NewHighlighting = ???
+
+  override def analysisMode(): AnalysisMode = ???
+
+  override def fileSystem(): FileSystem = ???
+
+  override def newDependency(): NewDependency = ???
+
+  override def settings(): Settings = ???
+
+  override def newMeasure[G <: io.Serializable](): NewMeasure[G] = ???
+
+  override def newIssue(): NewIssue = ???
 }