Partest rename NegStates

Author: vsalvis

test/dotty/partest/DottyPartestRunner.scala

@@ -101,21 +101,23 @@ class DottyRunner(testFile: File, suiteRunner: SuiteRunner) extends nest.Runner(
     import TestState.{ Crash, Fail }
     import scala.reflect.internal.FatalError
 
-    sealed abstract class State
-    case class FoundFailed() extends State
-    case class FailedWithWrongNErr(expected: String, found: String) extends State
-    case class NoneFailed() extends State
+    // Don't get confused, a neg test passes when compilation fails (optionally
+    // checking the number of compiler errors)
+    sealed abstract class NegTestState
+    case class CompFailed() extends NegTestState
+    case class CompFailedWrongNErr(expected: String, found: String) extends NegTestState
+    case class CompSucceeded() extends NegTestState
 
     def nerrIsOk(reason: String) = {
       import scala.util.matching.Regex
       val nerrFinder = """compilation failed with (\d\d+) errors""".r
       reason match {
         case nerrFinder(found) => 
           SFile(FileOps(testFile) changeExtension "nerr").safeSlurp match {
-            case Some(exp) if (exp != found) => FailedWithWrongNErr(exp, found)
-            case _ => FoundFailed
+            case Some(exp) if (exp != found) => CompFailedWrongNErr(exp, found)
+            case _ => CompFailed
           }
-        case _ => FoundFailed
+        case _ => CompFailed
       }
     }
 
@@ -129,16 +131,16 @@ class DottyRunner(testFile: File, suiteRunner: SuiteRunner) extends nest.Runner(
 
     val failureStates = compFailingRounds.map({ case (result, _) => result match {
       // or, OK, we'll let you crash the compiler with a FatalError if you supply a check file
-      case Crash(_, t, _) if !checkFile.canRead || !t.isInstanceOf[FatalError] => NoneFailed
-      case Fail(_, reason, _) => if (diffIsOk) nerrIsOk(reason) else NoneFailed
-      case _ => if (diffIsOk) FoundFailed else NoneFailed
+      case Crash(_, t, _) if !checkFile.canRead || !t.isInstanceOf[FatalError] => CompSucceeded
+      case Fail(_, reason, _) => if (diffIsOk) nerrIsOk(reason) else CompSucceeded
+      case _ => if (diffIsOk) CompFailed else CompSucceeded
     }})
 
-    if (failureStates.exists({ case FoundFailed => true; case _ => false })) {
+    if (failureStates.exists({ case CompFailed => true; case _ => false })) {
       true
     } else {
       val existsNerr = failureStates.exists({ 
-        case FailedWithWrongNErr(exp, found) => nextTestActionFailing(s"wrong number of compilation errors, expected: $exp, found: $found"); true
+        case CompFailedWrongNErr(exp, found) => nextTestActionFailing(s"wrong number of compilation errors, expected: $exp, found: $found"); true
         case _ => false
       })
       if (existsNerr) false else nextTestActionFailing("expected compilation failure")