Rewrite Map.mapValues -> Map.mapValues.toMap

Author: MasseGuillaume

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

@@ -8,4 +8,5 @@ class SetMapSrc(set: Set[Int], map: Map[Int, Int]) {
   map + (2 -> 3, 3 -> 4)
   (set + (2, 3)).map(x => x)
   set + (2, 3) - 4
+  map.mapValues(_ + 1)
 }

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

@@ -8,4 +8,5 @@ class SetMapSrc(set: Set[Int], map: Map[Int, Int]) {
   map + (2 -> 3) + (3 -> 4)
   (set + 2 + 3).map(x => x)
   set + 2 + 3 - 4
+  map.mapValues(_ + 1).toMap
 }

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

@@ -67,6 +67,11 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
       Symbol("_root_.scala.collection.mutable.MapLike.retain.")
     )
 
+  val mapMapValues = 
+    SymbolMatcher.exact(
+      Symbol("_root_.scala.collection.immutable.MapLike#mapValues(Lscala/Function1;)Lscala/collection/immutable/Map;.")
+    )
+
   val retainSet = 
     SymbolMatcher.normalized(
       Symbol("_root_.scala.collection.mutable.SetLike.retain.")
@@ -232,12 +237,20 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
     }.asPatch
   }
 
+
   def replaceIterableSameElements(ctx: RuleCtx): Patch = {
     ctx.tree.collect {
       case Term.Apply(Term.Select(lhs, iterableSameElement(_)), List(_)) =>
         ctx.addRight(lhs, ".iterator")
     }.asPatch
   }
+
+  def replaceMapMapValues(ctx: RuleCtx): Patch = {
+    ctx.tree.collect {
+      case ap @ Term.Apply(Term.Select(_, mapMapValues(_)), List(_)) =>
+        ctx.addRight(ap, ".toMap")
+    }.asPatch
+  }
 
 
   override def fix(ctx: RuleCtx): Patch = {
@@ -254,6 +267,7 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
       replaceSetMapPlus2(ctx) +
       replaceMutSetMapPlus(ctx) +
       replaceMutMapUpdated(ctx) +
-      replaceIterableSameElements(ctx)
+      replaceIterableSameElements(ctx) +
+      replaceMapMapValues(ctx)
   }
 }