Rewrite Map.zip. make shure we target Map

Author: MasseGuillaume

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

@@ -9,4 +9,5 @@ class SetMapSrc(set: Set[Int], map: Map[Int, Int]) {
   (set + (2, 3)).map(x => x)
   set + (2, 3) - 4
   map.zip(List())
+  List().zip(List())
 }

scalafix/output/src/main/scala/fix/SetMapSrc.scala

@@ -9,4 +9,5 @@ class SetMapSrc(set: Set[Int], map: Map[Int, Int]) {
   (set + 2 + 3).map(x => x)
   set + 2 + 3 - 4
   map.zip(List()).toMap
+  List().zip(List())
 }

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

@@ -8,6 +8,28 @@ import scala.meta._
 case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
   extends SemanticRule(index, "Scalacollectioncompat_newcollections") {
 
+  // terms dont give us terms https://github.com/scalameta/scalameta/issues/1212
+  // if we have a simple identifier, we can look at his definition at query it's type
+  // this should be improved in future version of scalameta
+  object TypeMatcher {
+    def apply(symbols: Symbol*)(implicit index: SemanticdbIndex): TypeMatcher =
+      new TypeMatcher(symbols: _*)(index)
+  }
+
+  final class TypeMatcher(symbols: Symbol*)(implicit index: SemanticdbIndex) {
+    def unapply(tree: Tree): Boolean = {
+      index.denotation(tree)
+           .map(_.names.headOption.exists(n => symbols.exists(_ == n.symbol)))
+           .getOrElse(false)
+    }
+  }
+
+  val CollectionMap: TypeMatcher = TypeMatcher(
+    Symbol("_root_.scala.collection.immutable.Map#"),
+    Symbol("_root_.scala.collection.mutable.Map#"),
+    Symbol("_root_.scala.Predef.Map#")
+  )
+
   def replaceSymbols(ctx: RuleCtx): Patch = {
     ctx.replaceSymbols(
       "scala.Stream" -> "scala.LazyList",
@@ -245,7 +267,7 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
 
   def replaceMapZip(ctx: RuleCtx): Patch = {
     ctx.tree.collect {
-      case ap @ Term.Apply(Term.Select(_, mapZip(_)), List(_)) =>
+      case ap @ Term.Apply(Term.Select(CollectionMap(), mapZip(_)), List(_)) =>
         ctx.addRight(ap, ".toMap")
 
     }.asPatch