Skip to content

Commit 242ba21

Browse files
authored
Remove double tests of tests/pos (#18683)
We should not tests all `tests/pos` files twice with another set of flags. Furthermore, in all our tests the result of adding the flags is not checked in any meaningful way. The only tests for this flag (tests/neg-custom-args/fatal-warnings/i17612b/i17612b.scala) does add the flag explicitly and checks the result already. See 46f2db6
2 parents ef28cd6 + b8aae69 commit 242ba21

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

compiler/test/dotty/tools/dotc/CompilationTests.scala

-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ class CompilationTests {
3434
compileFilesInDir("tests/pos", defaultOptions.and("-Ysafe-init")),
3535
compileFilesInDir("tests/pos-deep-subtype", allowDeepSubtypes),
3636
compileFilesInDir("tests/pos-special/sourcepath/outer", defaultOptions.and("-sourcepath", "tests/pos-special/sourcepath")),
37-
compileFilesInDir("tests/pos", defaultOptions.and("-Xlint:private-shadow", "-Xlint:type-parameter-shadow")),
3837
compileFile("tests/pos-special/sourcepath/outer/nested/Test4.scala", defaultOptions.and("-sourcepath", "tests/pos-special/sourcepath")),
3938
compileFilesInDir("tests/pos-scala2", defaultOptions.and("-source", "3.0-migration")),
4039
compileFilesInDir("tests/pos-custom-args/captures", defaultOptions.and("-language:experimental.captureChecking")),

0 commit comments

Comments
 (0)