Merge pull request #5357 from dotty-staging/clean-pickling-blacklist

Remove separate compilation tests for pickling blacklist

Author: nicolasstucki

compiler/test/dotc/pos-test-pickling.blacklist

@@ -1,39 +1,14 @@
-1567
 depfuntype.scala
-false-companion
-i0881
-i1137-1
-i1137-2
-i1137-3
-i1500.scala
 i1753.scala
 i1812.scala
 i1867.scala
-i2551
-i2797a
 i3067.scala
-i3130b.scala
 i3149.scala
-i3422
-i3467
 i3542-1.scala
 i3585.scala
 i3692.scala
-i3898
-i3898b
-i3898c
-i3912-1
-i3912-2
-i3912-3
 i3976.scala
-i4023
-i4023b
-i4023c
 i4125.scala
-i4167
-i4586.scala
-i4734
-i4758
 implicit-dep.scala
 inline-access-levels
 macro-with-array
@@ -53,10 +28,8 @@ t2712-5.scala
 t2726
 t2741
 t284-pos.scala
-t3249
 t3419
 t3452f.scala
-t3486
 t3612.scala
 t3864
 t3999
@@ -67,7 +40,6 @@ t5504
 t6028
 t6976
 t7264
-t7426.scala
 t7532b
 t8062
 typeclass-encoding2.scala

compiler/test/dotc/run-test-pickling.blacklist

@@ -1,100 +1,9 @@
-colltest4
-colltest5
-colltest6
 Course-2002-07.scala
 eff-dependent.scala
-i1140
-i1144
-i4431
-i4455
-i4492
-i4496b.scala
-i4515
-i4515b
-i4734
-i4735
-i4803
-i4803b
-i4803c
-i4803d
-i4803e
-i4803f
-i4947b
-i5119
-i5119b
-i5188a
 i5257.scala
-inline-varargs-1
-implicitShortcut
-inline-case-objects
-inline-option
-inline-macro-staged-interpreter
-inline-tuples-1
-inline-tuples-2
 lazy-implicit-lists.scala
 lazy-implicit-nums.scala
 lazy-traits.scala
-lst
-mixins1
-outerPatternMatch
-paramForwarding_separate
-quote-and-splice
-quote-change-owner
-quote-force
-quote-indexed-map-by-name
-quote-sep-comp
-quote-sep-comp-2
-quote-simple-macro
-quote-unrolled-foreach
-reduce-projections.scala
-t1500c.scala
-t3038c
-t3452a
-t3452b
-t3452b-bcode
-t3452d
-t3452e
-t3452g
-t4317
-t7120
-t7374
-t7859
 t8133
 t8133b
-tasty-argument-tree-1
-tasty-custom-show
-tasty-definitions-1
-tasty-definitions-2
-tasty-definitions-3
-tasty-eval
-tasty-extractors-1
-tasty-extractors-2
-tasty-extractors-3
-tasty-extractors-constants-1
-tasty-extractors-owners
-tasty-extractors-types
-tasty-getfile
-tasty-getfile-implicit-fun-context
-tasty-indexed-map
-tasty-implicit-fun-context-2
-tasty-linenumber
-tasty-linenumber-2
-tasty-location
-tasty-macro-assert
-tasty-positioned
-tasty-subtyping
-transparent
-transparentAccess
-transparentByName.scala
-transparentForeach.scala
-transparentPower
 tuples1.scala
-Tuple.scala
-TupleAbstract.scala
-typelevel-numeric.scala
-typelevel-patmat.scala
-typelevel.scala
-typelevel1.scala
-typelevel3.scala
-xml-interpolation-1
-xml-interpolation-2

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

@@ -90,9 +90,10 @@ class CompilationTests extends ParallelTesting {
     compileFilesInDir("tests/new", defaultOptions) +
     compileFilesInDir("tests/pos-scala2", scala2Mode) +
     compileFilesInDir("tests/pos", defaultOptions) +
+    compileFilesInDir("tests/pos-separate-compilation", defaultOptions) +
     compileFilesInDir("tests/pos-deep-subtype", allowDeepSubtypes) +
     compileFilesInDir("tests/pos-kind-polymorphism", defaultOptions and "-Ykind-polymorphism") +
-    compileDir("tests/pos/i1137-1", defaultOptions) +
+    compileDir("tests/pos-separate-compilation/i1137-1", defaultOptions) +
     compileFile(
       // succeeds despite -Xfatal-warnings because of -nowarn
       "tests/neg-custom-args/fatal-warnings/xfatalWarnings.scala",
@@ -172,6 +173,7 @@ class CompilationTests extends ParallelTesting {
     implicit val testGroup: TestGroup = TestGroup("runAll")
     compileFilesInDir("tests/run-custom-args/Yretain-trees", defaultOptions and "-Yretain-trees") +
     compileFile("tests/run-custom-args/tuple-cons.scala", allowDeepSubtypes) +
+    compileFilesInDir("tests/run-separate-compilation", defaultOptions) +
     compileFile("tests/run-custom-args/i5256.scala", allowDeepSubtypes) +
     compileFilesInDir("tests/run", defaultOptions)
   }.checkRuns()

compiler/test/dotty/tools/vulpix/ParallelTesting.scala

@@ -1130,7 +1130,7 @@ trait ParallelTesting extends RunnerOrchestration { self =>
 
   /** Make sure that directory string is as expected */
   private def checkRequirements(f: String, sourceDir: JFile, outDir: String): Unit = {
-    require(sourceDir.isDirectory && sourceDir.exists, "passed non-directory to `compileFilesInDir`")
+    require(sourceDir.isDirectory && sourceDir.exists, "passed non-directory to `compileFilesInDir`: " + sourceDir)
     require(outDir.last == JFile.separatorChar, "please specify an `outDir` with a trailing file separator")
   }