Skip to content

Commit 7224aaa

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 9e880b2 commit 7224aaa

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)