Upgrade Scala and Scala-JS versions in build and Travis configuration.

Scala:
- 2.11.8 -> 2.11.11
- 2.12.1 -> 2.12.3

Scala-JS:
- 0.6.13 -> 0.6.19

Author: gslowikowski

.travis.yml

@@ -8,9 +8,9 @@ matrix:
     - jdk: oraclejdk7
       scala: 2.10.6
     - jdk: oraclejdk7
-      scala: 2.11.8
+      scala: 2.11.11
     - jdk: oraclejdk8
-      scala: 2.12.0
+      scala: 2.12.3
 
 before_cache:
   - find "$HOME/.sbt/" -name '*.lock' -print0 | xargs -0 rm

build.sbt

@@ -12,8 +12,8 @@ val ScalatestVersion = "3.0.0"
 
 val appSettings = Seq(
     organization := Org,
-    scalaVersion := "2.12.0",
-    crossScalaVersions := Seq("2.10.6", "2.11.8", "2.12.0"),
+    scalaVersion := "2.12.3",
+    crossScalaVersions := Seq("2.10.6", "2.11.11", "2.12.3"),
     fork in Test := false,
     publishMavenStyle := true,
     publishArtifact in Test := false,

project/plugins.sbt

@@ -4,4 +4,4 @@ addSbtPlugin("com.jsuereth" % "sbt-pgp" % "1.0.1")
 
 addSbtPlugin("com.github.gseitz" % "sbt-release" % "1.0.3")
 
-addSbtPlugin("org.scala-js" % "sbt-scalajs" % "0.6.13")
+addSbtPlugin("org.scala-js" % "sbt-scalajs" % "0.6.19")

scalac-scoverage-plugin/src/main/scala/scoverage/plugin.scala

@@ -581,9 +581,6 @@ class ScoverageInstrumentationComponent(val global: Global, extraAfterPhase: Opt
 
         case _: TypeTree => super.transform(tree)
 
-        // if the rhs of a val is a literal we can just leave it
-        case v: ValDef if v.rhs.isInstanceOf[Literal] => tree
-
         /**
           * We can ignore lazy val defs as they are implemented by a generated defdef
           */
@@ -602,6 +599,7 @@ class ScoverageInstrumentationComponent(val global: Global, extraAfterPhase: Opt
 
         // we need to remove the final mod so that we keep the code in order to check its invoked
         case v: ValDef if v.mods.isFinal =>
+          updateLocation(v)
           treeCopy.ValDef(v, v.mods.&~(ModifierFlags.FINAL), v.name, v.tpt, process(v.rhs))
 
         /**

scalac-scoverage-plugin/src/test/scala/scoverage/PluginCoverageTest.scala

@@ -23,7 +23,7 @@ class PluginCoverageTest
     // we expect:
     // instrumenting the default-param which becomes a method call invocation
     // the method makeGreeting is entered.
-    compiler.assertNMeasuredStatements(1)
+    compiler.assertNMeasuredStatements(2)
   }
 
   test("scoverage should skip macros") {
@@ -68,7 +68,7 @@ class PluginCoverageTest
                                    |} """.stripMargin)
     assert(!compiler.reporter.hasErrors)
     // we should have 3 statements - initialising the val, executing println, and executing the parameter
-    compiler.assertNMeasuredStatements(7)
+    compiler.assertNMeasuredStatements(8)
   }
 
   test("scoverage should not instrument the match as a statement") {