Skip to content

Commit c1d70b4

Browse files
authored
Merge pull request #3164 from Gedochao/maintenance/adjust-tests-for-Scala-3.3.4
Adjust tests for Scala 3.3.4
2 parents c9c659c + 36e6543 commit c1d70b4

File tree

3 files changed

+22
-13
lines changed

3 files changed

+22
-13
lines changed

modules/integration/src/test/scala/scala/cli/integration/DocTestDefinitions.scala

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,9 @@ abstract class DocTestDefinitions extends ScalaCliSuite with TestScalaVersionArg
4242
)
4343
else if (
4444
actualScalaVersion.coursierVersion >= "3.5.0".coursierVersion ||
45+
(actualScalaVersion.coursierVersion >= "3.3.4".coursierVersion &&
46+
actualScalaVersion.coursierVersion < "3.4.0".coursierVersion) ||
47+
actualScalaVersion.startsWith("3.3.4") ||
4548
actualScalaVersion.startsWith("3.5")
4649
)
4750
Seq(

modules/integration/src/test/scala/scala/cli/integration/PackageTestDefinitions.scala

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -912,6 +912,9 @@ abstract class PackageTestDefinitions extends ScalaCliSuite with TestScalaVersio
912912
)
913913
else if (
914914
actualScalaVersion.coursierVersion >= "3.5.0".coursierVersion ||
915+
(actualScalaVersion.coursierVersion >= "3.3.4".coursierVersion &&
916+
actualScalaVersion.coursierVersion < "3.4.0".coursierVersion) ||
917+
actualScalaVersion.startsWith("3.3.4") ||
915918
actualScalaVersion.startsWith("3.5")
916919
)
917920
Seq(

modules/integration/src/test/scala/scala/cli/integration/SipScalaTests.scala

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -907,18 +907,21 @@ class SipScalaTests extends ScalaCliSuite with SbtTestHelper with MillTestHelper
907907
}
908908
}
909909

910-
test("scalac help respects --cli-default-scala-version") {
911-
TestInputs.empty.fromRoot { root =>
912-
val sv = Constants.scala3NextRc
913-
val launcherVersionOverrideHelp =
914-
os.proc(TestUtil.cli, "--cli-default-scala-version", sv, "--scalac-help")
915-
.call(cwd = root).out.trim()
916-
val standardVersionOverrideHelp =
917-
os.proc(TestUtil.cli, "--scalac-help", "-S", sv)
918-
.call(cwd = root).out.trim()
919-
val migrationPrefix = sv.take(2) + (sv.charAt(2).asDigit + 1).toString
920-
expect(launcherVersionOverrideHelp.contains(s"$migrationPrefix-migration"))
921-
expect(launcherVersionOverrideHelp == standardVersionOverrideHelp)
910+
// this check is just to ensure this isn't being run for LTS RC jobs
911+
// should be adjusted when a new LTS line is released
912+
if (!Constants.scala3NextRc.startsWith(Constants.scala3LtsPrefix))
913+
test("scalac help respects --cli-default-scala-version") {
914+
TestInputs.empty.fromRoot { root =>
915+
val sv = Constants.scala3NextRc
916+
val launcherVersionOverrideHelp =
917+
os.proc(TestUtil.cli, "--cli-default-scala-version", sv, "--scalac-help")
918+
.call(cwd = root).out.trim()
919+
val standardVersionOverrideHelp =
920+
os.proc(TestUtil.cli, "--scalac-help", "-S", sv)
921+
.call(cwd = root).out.trim()
922+
val migrationPrefix = sv.take(2) + (sv.charAt(2).asDigit + 1).toString
923+
expect(launcherVersionOverrideHelp.contains(s"$migrationPrefix-migration"))
924+
expect(launcherVersionOverrideHelp == standardVersionOverrideHelp)
925+
}
922926
}
923-
}
924927
}

0 commit comments

Comments
 (0)