From 9d13c572504e290b4d71b8c6c8262fdc32cd824f Mon Sep 17 00:00:00 2001
From: Jesse Luehrs <doy@tozt.net>
Date: Thu, 28 Feb 2013 16:59:24 -0600
Subject: eh, skip all with no message isn't worth it, really

---
 src/main/scala/org/perl8/test/package.scala        | 24 +++++-----------------
 src/main/scala/org/perl8/test/tap/Producer.scala   |  8 +-------
 .../scala/org/perl8/test/tap/TestBuilderTest.scala | 14 -------------
 3 files changed, 6 insertions(+), 40 deletions(-)

(limited to 'src')

diff --git a/src/main/scala/org/perl8/test/package.scala b/src/main/scala/org/perl8/test/package.scala
index b18dc2f..d2b8373 100644
--- a/src/main/scala/org/perl8/test/package.scala
+++ b/src/main/scala/org/perl8/test/package.scala
@@ -8,35 +8,21 @@ package object test {
 
   sealed trait Plan {
     val plan:    Int
-    val skipAll: Boolean
-    val message: Option[String]
+    val skipAll: Option[String]
   }
 
   case class NumericPlan (plan: Int) extends Plan {
-    val skipAll = false
-    val message = None
+    val skipAll = None
   }
 
   case object NoPlan extends Plan {
     val plan    = 0
-    val skipAll = false
-    val message = None
+    val skipAll = None
   }
 
-  case class SkipAll (msg: String) extends Plan {
+  case class SkipAll (message: String) extends Plan {
     val plan    = 0
-    val skipAll = true
-    val message = Some(msg)
-  }
-
-  case object SkipAllNoMessage extends Plan {
-    val plan    = 0
-    val skipAll = true
-    val message = None
-  }
-
-  object SkipAll {
-    def apply () = SkipAllNoMessage
+    val skipAll = Some(message)
   }
 
   case class BailOutException (
diff --git a/src/main/scala/org/perl8/test/tap/Producer.scala b/src/main/scala/org/perl8/test/tap/Producer.scala
index 52c72c9..d432856 100644
--- a/src/main/scala/org/perl8/test/tap/Producer.scala
+++ b/src/main/scala/org/perl8/test/tap/Producer.scala
@@ -25,13 +25,7 @@ object Producer {
     message.split("\n").map(m => "# " + m).mkString("\n")
 
   def plan (plan: Plan): String =
-    if (plan.skipAll) {
-      val skip = "1..0 # SKIP"
-      plan.message.map(m => skip + " " + m).getOrElse(skip)
-    }
-    else {
-      "1.." + plan.plan
-    }
+    plan.skipAll.map(m => "1..0 # SKIP " + m).getOrElse("1.." + plan.plan)
 
   def bailOut: String =
     "Bail out!"
diff --git a/src/test/scala/org/perl8/test/tap/TestBuilderTest.scala b/src/test/scala/org/perl8/test/tap/TestBuilderTest.scala
index d59541a..36ffb9b 100644
--- a/src/test/scala/org/perl8/test/tap/TestBuilderTest.scala
+++ b/src/test/scala/org/perl8/test/tap/TestBuilderTest.scala
@@ -187,20 +187,6 @@ class TestBuilderTest extends TestMore {
   }
 
   subtest ("skip all") {
-    val output = new ByteArrayOutputStream
-    Console.withOut(output) {
-      Console.withErr(output) {
-        val builder = new TestBuilder(SkipAll())
-      }
-    }
-
-    val expected =
-      "1..0 # SKIP\n"
-
-    is(output.toString, expected)
-  }
-
-  subtest ("skip all with reason") {
     val output = new ByteArrayOutputStream
     Console.withOut(output) {
       Console.withErr(output) {
-- 
cgit v1.2.3-54-g00ecf