fail the build if a state change request returns a failure status code

pact-foundation · Feb 28, 2015 · a13bac9 · a13bac9
1 parent 2eff45b
commit a13bac9
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 3 deletions.
diff --git a/...vider-gradle/src/main/groovy/au/com/dius/pact/provider/gradle/PactVerificationTask.groovy b/...vider-gradle/src/main/groovy/au/com/dius/pact/provider/gradle/PactVerificationTask.groovy
@@ -173,7 +173,17 @@ class PactVerificationTask extends DefaultTask {
                     .reset())
             } else {
                 ProviderClient client = new ProviderClient(provider: providerToVerify)
-                client.makeStateChangeRequest(url, state, consumer.stateChangeUsesBody)
+                def response = client.makeStateChangeRequest(url, state, consumer.stateChangeUsesBody)
+                try {
+                    if (response.statusCode >= 400) {
+                        AnsiConsole.out().println(Ansi.ansi().a('         ').fg(Ansi.Color.RED)
+                            .a('State Change Request Failed - ')
+                            .a(response.statusLine.toString()).reset())
+                        return 'State Change Request Failed - ' + response.statusLine.toString()
+                    }
+                } finally {
+                    response.close()
+                }
             }
             return true
         } catch (e) {

diff --git a/...vm-provider-maven/src/main/groovy/au/com/dius/pact/provider/maven/PactProviderMojo.groovy b/...vm-provider-maven/src/main/groovy/au/com/dius/pact/provider/maven/PactProviderMojo.groovy
@@ -219,7 +219,17 @@ class PactProviderMojo extends AbstractMojo {
                     stateChangeUsesBody = provider.stateChangeUsesBody
                 }
                 ProviderClient client = new ProviderClient(provider: provider)
-                client.makeStateChangeRequest(stateChangeUrl.toString(), state, stateChangeUsesBody)
+                def response = client.makeStateChangeRequest(stateChangeUrl.toString(), state, stateChangeUsesBody)
+                try {
+                    if (response.statusCode >= 400) {
+                        AnsiConsole.out().println(Ansi.ansi().a('         ').fg(Ansi.Color.RED)
+                            .a('State Change Request Failed - ')
+                            .a(response.statusLine.toString()).reset())
+                        return 'State Change Request Failed - ' + response.statusLine.toString()
+                    }
+                } finally {
+                    response.close()
+                }
             }
             return true
         } catch (e) {

diff --git a/pact-jvm-provider/build.gradle b/pact-jvm-provider/build.gradle
@@ -3,5 +3,5 @@ dependencies {
         "org.scalatest:scalatest_${project.scalaVersion}:2.2.1",
         'commons-io:commons-io:2.4', 'org.codehaus.groovy:groovy-all:2.3.7', 'org.fusesource.jansi:jansi:1.11',
         'org.codehaus.groovy.modules.http-builder:http-builder:0.7',
-        'org.apache.httpcomponents:httpclient:4.3.6', 'com.googlecode.java-diff-utils:diffutils:1.3.0'
+        'org.apache.httpcomponents:httpclient:4.4', 'com.googlecode.java-diff-utils:diffutils:1.3.0'
 }