X-Git-Url: https://git.brokenzipper.com/gitweb?a=blobdiff_plain;f=src%2Fmain%2Fscala%2FSbtTapReporting.scala;h=9345066996c621c06df72c08b10e2625c85583f4;hb=7b47de44e98f157ae5edae5ceafa5249cebe3cfd;hp=45510ac640ab887139f7d1c9b8f8919a648e4995;hpb=37a33da19bc4034f15db3d9468f74293902647ca;p=chaz%2Fsbt-tap diff --git a/src/main/scala/SbtTapReporting.scala b/src/main/scala/SbtTapReporting.scala index 45510ac..9345066 100644 --- a/src/main/scala/SbtTapReporting.scala +++ b/src/main/scala/SbtTapReporting.scala @@ -5,7 +5,7 @@ import org.scalatools.testing.{Event => TEvent, Result => TResult} import java.util.concurrent.atomic.AtomicInteger object SbtTapReporting extends Plugin { - lazy val tapListener = new SbtTapListener + def apply() = new SbtTapListener } /** @@ -34,10 +34,13 @@ class SbtTapListener extends TestsListener { case TResult.Success => writeTapFields("ok", testId.incrementAndGet(), "-", e.testName()) case TResult.Error | TResult.Failure => writeTapFields("not ok", testId.incrementAndGet(), "-", e.testName()) - writeTapFields(stackTraceForError(e.error())) + // According to the TAP spec, as long as there is any kind of whitespace, this output should belong to the + // the test that failed and it should get displayed in the UI. + // TODO:It would be nice if we could report the exact line in the test where this happened. + writeTapFields(" ", stackTraceForError(e.error())) case TResult.Skipped => // it doesn't look like this framework distinguishes between pending and ignored. - writeTapFields("ok", testId.incrementAndGet(), e.testName(), "#", "skip") + writeTapFields("ok", testId.incrementAndGet(), e.testName(), "#", "skip", e.testName()) } } }