From 15c2e6a55f11c4978f7da5c9bc53498b5feb1dd3 Mon Sep 17 00:00:00 2001 From: Jesse Luehrs Date: Thu, 28 Feb 2013 18:50:36 -0600 Subject: get the exit code correct for when no tests are run --- src/main/scala/org/perl8/test/tap/Consumer.scala | 9 +++------ src/main/scala/org/perl8/test/tap/TestBuilder.scala | 5 +---- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/src/main/scala/org/perl8/test/tap/Consumer.scala b/src/main/scala/org/perl8/test/tap/Consumer.scala index b15b03b..c108b6d 100644 --- a/src/main/scala/org/perl8/test/tap/Consumer.scala +++ b/src/main/scala/org/perl8/test/tap/Consumer.scala @@ -110,7 +110,7 @@ object Consumer { ) class TAPResult (val plan: Plan, val results: Seq[TestResult]) { - val correctPlan = plan match { + val matchesPlan = plan match { case NumericPlan(n) => results.length == n case _ => results.length == 0 } @@ -122,13 +122,10 @@ object Consumer { val testsPassed = fails == 0 val success = - correctPlan && testsPassed + matchesPlan && testsPassed val exitCode = - if (success) { - 0 - } - else if (!correctPlan) { + if (!matchesPlan || results.length == 0) { 255 } else { diff --git a/src/main/scala/org/perl8/test/tap/TestBuilder.scala b/src/main/scala/org/perl8/test/tap/TestBuilder.scala index 4ccd396..9a8d2e4 100644 --- a/src/main/scala/org/perl8/test/tap/TestBuilder.scala +++ b/src/main/scala/org/perl8/test/tap/TestBuilder.scala @@ -103,10 +103,7 @@ class TestBuilder private ( state.failCount def exitCode: Int = - if (state.isPassing) { - 0 - } - else if (!state.matchesPlan) { + if (!state.matchesPlan || state.currentTest == 0) { 255 } else { -- cgit v1.2.3-54-g00ecf