Skip to content

Commit 3ff7045

Browse files
nicolasstuckiWojciechMazur
authored andcommitted
Remove double tests of tests/pos
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 [Cherry-picked b8aae69]
1 parent 8d05ace commit 3ff7045

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

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

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

0 commit comments

Comments
 (0)