Skip to content
This repository was archived by the owner on May 3, 2018. It is now read-only.

Commit efd879c

Browse files
committed
rename PartestTask to SbtPartestTask
`PartestTask` is a very unfortunate name for this, because it already exists in scala-partest; there is: - `scala.tools.partest.PartestTask extends org.apache.tools.ant.Task` in scala-partest, for running partest with ant - `scala.tools.partest.PartestTask extends sbt.testing.Task` for running partest with sbt Depending on the order `scala-partest` and `scala-partest-interface` are added to the dependencies, SBT can get the wrong one and crash...
1 parent 544fe98 commit efd879c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/main/scala/tools/partest/TestingInterface.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ case class Runner(args: Array[String], remoteArgs: Array[String], testClassLoade
4949
* @return an array of <code>Task</code>s
5050
* @throws IllegalStateException if invoked after <code>done</code> has been invoked.
5151
*/
52-
def tasks(taskDefs: Array[TaskDef]): Array[sbt.testing.Task] = taskDefs map (PartestTask(_): sbt.testing.Task)
52+
def tasks(taskDefs: Array[TaskDef]): Array[sbt.testing.Task] = taskDefs map (SbtPartestTask(_): sbt.testing.Task)
5353

5454
/** Indicates the client is done with this <code>Runner</code> instance.
5555
*
@@ -62,7 +62,7 @@ case class Runner(args: Array[String], remoteArgs: Array[String], testClassLoade
6262
*
6363
* TODO: make configurable
6464
*/
65-
case class PartestTask(taskDef: TaskDef) extends Task {
65+
case class SbtPartestTask(taskDef: TaskDef) extends Task {
6666
/** Executes this task, possibly returning to the client new tasks to execute. */
6767
def execute(eventHandler: EventHandler, loggers: Array[Logger]): Array[Task] = {
6868
val forkedCp = scala.util.Properties.javaClassPath

0 commit comments

Comments
 (0)