Merge pull request scala#97 from MasseGuillaume/bug/to-list-trailing

Make sure foo.to[X] has trailing brackets

Author: julienrf

admin/build.sh

@@ -62,4 +62,4 @@ if [[ "$TRAVIS_TAG" =~ $tagPat ]]; then
   fi
 fi
 
-sbt -Dhttps.protocols=TLSv1.2 -sbt-dir=/home/travis/.sbt ";$crossScalaVersion ;$publishVersion ;$projectPrefix/clean ;$testProjectPrefix/test ;$projectPrefix/publishLocal ;$publishTask"
+sbt -Dhttps.protocols=TLSv1.2 ";$crossScalaVersion ;$publishVersion ;$projectPrefix/clean ;$testProjectPrefix/test ;$projectPrefix/publishLocal ;$publishTask"

scalafix/input/src/main/scala/fix/TraversableSrc.scala

@@ -10,4 +10,9 @@ object TraversableSrc {
     xs.toIterator
     ys.iterator
   }
+
+  def m1(xs: Traversable[Int]): List[Int] =
+    xs.to
+
+  def m2(xs: Traversable[Int]): List[Int] = xs.to
 }

scalafix/output212/src/main/scala/fix/TraversableSrc.scala

@@ -11,4 +11,9 @@ object TraversableSrc {
     xs.iterator
     ys.iterator
   }
+
+  def m1(xs: Iterable[Int]): List[Int] =
+    xs.to
+
+  def m2(xs: Iterable[Int]): List[Int] = xs.to
 }

scalafix/rules/src/main/scala/fix/Stable212Base.scala

@@ -190,7 +190,7 @@ trait Stable212Base extends CrossCompatibility { self: SemanticRule =>
       case iterator(t: Name) =>
         ctx.replaceTree(t, "iterator")
 
-      case t @ toTpe(n: Name) if !handledTo.contains(n) =>
+      case Term.ApplyType(Term.Select(_, t @ toTpe(n: Name)), _) if !handledTo.contains(n) =>
         trailingBrackets(n, ctx).map { case (open, close) =>
           ctx.replaceToken(open, "(") + ctx.replaceToken(close, ")")
         }.asPatch