Skip to content

Commit 764c9b7

Browse files
committed
ci: drop dist/pack from test_non_bootstrapped
1 parent 9be5a34 commit 764c9b7

File tree

3 files changed

+10
-1
lines changed

3 files changed

+10
-1
lines changed

.github/workflows/ci.yaml

+6-1
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,13 @@ jobs:
6969
run: cp -vf .github/workflows/repositories /root/.sbt/ ; true
7070

7171
- name: Test
72+
# DON'T add dist/pack!
73+
# Adding dist/pack bootstraps the compiler
74+
# which undermines the point of these tests:
75+
# to quickly run the tests without the cost of bootstrapping
76+
# and also to run tests when the compiler doesn't bootstrap
7277
run: |
73-
./project/scripts/sbt ";dist/pack; compile ;test"
78+
./project/scripts/sbt ";compile ;test"
7479
./project/scripts/cmdTests
7580
7681
test:

compiler/test/dotty/tools/scripting/BashScriptsTests.scala

+2
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import scala.language.unsafeNulls
77
import java.nio.file.Paths
88
import org.junit.{Test, AfterClass}
99
import org.junit.Assert.assertEquals
10+
import org.junit.experimental.categories.Category
1011

1112
import vulpix.TestConfiguration
1213

@@ -156,6 +157,7 @@ class BashScriptsTests:
156157
* verify that scriptPath.sc sees a valid script.path property,
157158
* and that it's value is the path to "scriptPath.sc".
158159
*/
160+
@Category(Array(classOf[BootstrappedOnlyTests]))
159161
@Test def verifyScriptPathProperty =
160162
val scriptFile = testFiles.find(_.getName == "scriptPath.sc").get
161163
val expected = s"${scriptFile.getName}"

compiler/test/dotty/tools/scripting/ExpressionTest.scala

+2
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import scala.language.unsafeNulls
77
import java.nio.file.Paths
88
import org.junit.{Test, AfterClass}
99
import org.junit.Assert.assertEquals
10+
import org.junit.experimental.categories.Category
1011

1112
import vulpix.TestConfiguration
1213

@@ -15,6 +16,7 @@ import ScriptTestEnv.*
1516
/**
1617
* +. test scala -e <expression>
1718
*/
19+
@Category(Array(classOf[BootstrappedOnlyTests]))
1820
class ExpressionTest:
1921
/*
2022
* verify -e <expression> works.

0 commit comments

Comments
 (0)