Added support for nested expressions in new statements

Author: sksamuel

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

@@ -155,7 +155,7 @@ class ScoverageInstrumentationComponent(val global: Global)
     def instrument(tree: Tree, branch: Boolean = false): Tree = {
       safeSource(tree) match {
         case None =>
-          println(s"[warn] [scoverage] Could not instrument [${tree.getClass.getSimpleName}/${tree.symbol}]. No pos.")
+          println(s"[warn] Could not instrument [${tree.getClass.getSimpleName}/${tree.symbol}]. No pos.")
           tree
         case Some(source) =>
           if (tree.pos.isDefined && !isStatementIncluded(tree.pos)) {
@@ -465,6 +465,8 @@ class ScoverageInstrumentationComponent(val global: Global)
          * List(Literal(Constant(3))))
          *
          */
+        case n: New => n
+
         case s@Select(n@New(tpt), name) =>
           instrument(treeCopy.Select(s, n, name))
 

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

@@ -106,8 +106,9 @@ class PluginCoverageTest
 
   test("scoverage should correctly handle new that invokes nested statements") {
     val compiler = ScoverageCompiler.default
-    compiler.compileCodeSnippet( """ object A {
-                                   |  new String(if (System.currentTimeMillis > 1) "yes" else "no")
+    compiler.compileCodeSnippet( """
+                                   | object A {
+                                   |  val value = new java.util.concurrent.CountDownLatch(if (System.currentTimeMillis > 1) 5 else 10)
                                    | } """.stripMargin)
     assert(!compiler.reporter.hasErrors)
     assert(!compiler.reporter.hasWarnings)