Fix for issue skipoleschris#16. Failing tests when running the cucumber task now cause the build to fail.

Author: skipoleschris

Diff for: README.markdown

@@ -120,7 +120,7 @@ To install the cucumber plugin, add entries to the build plugins file (project/p
 
     resolvers += "Templemore Repository" at "http://templemore.co.uk/repo"
 
-    addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.1")
+    addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.2")
 
 ### Basic Configuration ###
 To add the cucumber plugin settings to a basic project, just add the following to the build.sbt file:
@@ -213,6 +213,10 @@ Requests for features can be posted to the issues list or emailed to the author.
 
 ## Release History ##
 
+### 0.7.2 ###
+
+Fix for issue #16. Failing cucumber tests now cause the build to report an error.
+
 ### 0.7.1 ###
 
 Fix for issue #15. The templemore repository is now added as a resolver to the project when the plugin is added.

Diff for: plugin/src/main/scala/templemore/sbt/cucumber/CucumberPlugin.scala

@@ -10,7 +10,7 @@ import templemore.sbt.util._
  */
 object CucumberPlugin extends Plugin with Integration {
 
-  private val projectVersion = "0.7.1"
+  private val projectVersion = "0.7.2"
 
   type LifecycleCallback = () => Unit
 
@@ -43,7 +43,12 @@ object CucumberPlugin extends Plugin with Integration {
   val cucumberAfter = SettingKey[LifecycleCallback]("cucumber-after")
 
   protected def cucumberTask(argTask: TaskKey[Seq[String]]) =
-    (argTask, cucumberTestSettings, cucumberOptions, cucumberOutput, streams) map(cuke)
+    (argTask, cucumberTestSettings, cucumberOptions, cucumberOutput, streams) map {
+      (args, settings, opt, out, s) => cuke(args, settings, opt, out, s) match {
+        case 0 => 0
+        case _ => sys.error("There were failed tests.")
+      }
+    }
 
   protected def cucumberSettingsTask: Initialize[Task[JvmSettings]] =
     (fullClasspath in Test, cucumberMainClass, streams, cucumberSystemProperties, cucumberJVMOptions, cucumberMaxMemory, cucumberMaxPermGen) map {

Diff for: project/Build.scala

@@ -4,7 +4,7 @@ import Keys._
 object Settings {
   val buildOrganization = "templemore"
   val buildScalaVersion = "2.9.2"
-  val buildVersion      = "0.7.1"
+  val buildVersion      = "0.7.2"
 
   val buildSettings = Defaults.defaultSettings ++
                       Seq (organization  := buildOrganization,

Diff for: testProjects/integrationTestIntegrationProject/project/Build.scala

@@ -5,7 +5,7 @@ import templemore.sbt.cucumber.CucumberPlugin
 object BuildSettings {
   val buildOrganization = "templemore"
   val buildScalaVersion = "2.9.2"
-  val buildVersion      = "0.7.1"
+  val buildVersion      = "0.7.2"
 
   val buildSettings = Defaults.defaultSettings ++
                       Seq (organization := buildOrganization,

Diff for: testProjects/integrationTestIntegrationProject/project/plugin.sbt

@@ -1,5 +1,5 @@
 resolvers += "Templemore Repository" at "http://templemore.co.uk/repo"
 
-addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.1")
+addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.2")
 
 

Diff for: testProjects/multiModuleTestProject/project/Build.scala

@@ -5,7 +5,7 @@ import templemore.sbt.cucumber.CucumberPlugin
 object BuildSettings {
   val buildOrganization = "templemore"
   val buildScalaVersion = "2.9.2"
-  val buildVersion      = "0.7.1"
+  val buildVersion      = "0.7.2"
 
   val buildSettings = Defaults.defaultSettings ++
                       Seq (organization := buildOrganization,

Diff for: testProjects/multiModuleTestProject/project/plugins.sbt

@@ -1,3 +1,3 @@
 resolvers += Resolver.file("Local Repo", file((Path.userHome / ".m2" / "repository").toString))
 
-addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.1")
+addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.2")

Diff for: testProjects/testIntegrationProject/build.sbt

@@ -1,6 +1,6 @@
 name := "test-project"
 
-version := "0.7.1"
+version := "0.7.2"
 
 organization := "templemore"
 

Diff for: testProjects/testIntegrationProject/project/plugin.sbt

@@ -1,5 +1,5 @@
 resolvers += "Templemore Repository" at "http://templemore.co.uk/repo"
 
-addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.1")
+addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.2")
 
 

Diff for: testProjects/testProject/project/plugin.sbt

@@ -1,5 +1,5 @@
 resolvers += "Templemore Repository" at "http://templemore.co.uk/repo"
 
-addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.1")
+addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.2")
 
 

Diff for: testProjects/testProject2_10/build.sbt

@@ -1,6 +1,6 @@
 name := "test-project2_10"
 
-version := "0.7.1"
+version := "0.7.2"
 
 organization := "templemore"
 

Diff for: testProjects/testProject2_10/project/plugin.sbt

@@ -1,3 +1,3 @@
 resolvers += "Templemore Repository" at "http://templemore.co.uk/repo"
 
-addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.1")
+addSbtPlugin("templemore" % "sbt-cucumber-plugin" % "0.7.2")