Add test for ExlicitResultTypes fix

This reproduces an issue discovered in a Mill PR * com-lihaoyi/mill#2922
joan38 · Jan 7, 2024 · 3306a36 · 3306a36
1 parent 8179e4f
commit 3306a36
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/itest/src/fix/.scalafix.conf b/itest/src/fix/.scalafix.conf
@@ -3,7 +3,8 @@ rules = [
   DisableSyntax
   LeakingImplicitClassVal
   NoValInForComprehension
-  ProcedureSyntax
+  ProcedureSyntax,
+  ExplicitResultTypes
 ]
 
 DisableSyntax.noVars = true

diff --git a/itest/src/fix/build.sc b/itest/src/fix/build.sc
@@ -6,6 +6,7 @@ import os._
 
 object project extends ScalaModule with ScalafixModule {
   def scalaVersion  = "2.13.10"
+  def scalafixScalaBinaryVersion = mill.scalalib.api.ZincWorkerUtil.scalaBinaryVersion(scalaVersion())
   def scalacOptions = Seq("-Ywarn-unused")
 }
 
@@ -14,6 +15,7 @@ def verify() =
     val fixedScala = read(pwd / "project" / "src" / "Fix.scala")
     val expected = """object Fix {
                      |  def procedure(): Unit = {}
+                     |  def myComplexMethod: Map[Int, String] = 1.to(10).map(i => i -> i.toString).toMap
                      |}
                      |""".stripMargin
     assert(fixedScala == expected)

diff --git a/itest/src/fix/project/src/Fix.scala b/itest/src/fix/project/src/Fix.scala
@@ -1,3 +1,4 @@
 object Fix {
   def procedure() {}
+  def myComplexMethod = 1.to(10).map(i => i -> i.toString).toMap
 }