move scala 2 test to sbt-test

scala · Nov 17, 2022 · ced7f55 · ced7f55
1 parent 1a484de
commit ced7f55
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 0 deletions.
diff --git a/tests/pos/i16351/App.scala → sbt-test/scala2-compat/i16351/app/App.scala b/tests/pos/i16351/App.scala → sbt-test/scala2-compat/i16351/app/App.scala
diff --git a/sbt-test/scala2-compat/i16351/build.sbt b/sbt-test/scala2-compat/i16351/build.sbt
@@ -0,0 +1,13 @@
+val scala3Version = sys.props("plugin.scalaVersion")
+val scala2Version = sys.props("plugin.scala2Version")
+
+lazy val lib = project.in(file("lib"))
+  .settings(
+    scalaVersion := scala2Version
+  )
+
+lazy val app = project.in(file("app"))
+  .dependsOn(lib)
+  .settings(
+    scalaVersion := scala3Version
+  )
diff --git a/tests/pos/i16351/lib.scala → sbt-test/scala2-compat/i16351/lib/lib.scala b/tests/pos/i16351/lib.scala → sbt-test/scala2-compat/i16351/lib/lib.scala
diff --git a/sbt-test/scala2-compat/i16351/test b/sbt-test/scala2-compat/i16351/test
@@ -0,0 +1 @@
+> app/compile
diff --git a/tests/init/pos/i9795/A.scala b/tests/init/pos/i9795/A.scala
@@ -0,0 +1,7 @@
+class A:
+  // Safe initialization check only allows capturing `this` either through primary constructor or synthetic `apply`
+  // `Some` case class comes from Scala 2 stdlib, which is not visible, hence the warning
+  // For reference:
+  // https://github.com/lampepfl/dotty/pull/12711
+  // https://github.com/lampepfl/dotty/pull/14283
+  val some = Some(this)