Add new option "allow failing build results" #25

Merged
merged 2 commits into from May 16, 2013

Conversation

Projects
None yet
3 participants

xcode-plugin already provides the ability to parse the results of running unit tests on an Xcode project by way of configuring a test step. However, since the tests run as a build step, a failing test result will fail the entire build. This may or may not be the desired behavior.

This pull request provides the ability to ignore an error return code from xcodebuild by way of a new option: "Allow failing build results." If checked, xcode-plugin will not treat a non-zero return code from an xcodebuild run as a reason to fail the build step.

This can prove very useful for test build steps configured in tandem with Jenkins' "Publish JUnit test result report" post-build action: if there are failing tests, that action will mark the build unstable, rather than failed. This can help give developers an additional level of information about the status of their build.

@lithium3141 lithium3141 Add new option "allow failing build results"
* Add a new Boolean option to XCodeBuilder.java
* Add equivalent checkbox in config.jelly
* Document new option in help-allowFailingBuildResults.html

Signed-off-by: Tim Ekl <lithium3141@gmail.com>
bbb8093

Jenkins » xcode-plugin #42 FAILURE
Looks like there's a problem with this pull request
(what's this?)

Looks like maven-hpi-plugin at this URL is now found over here - not sure how to correct that for @buildhive, though.

@lithium3141 lithium3141 New option is a Boolean, not a boolean
* Check for null and compare boolean values in XcodeBuilder.java

Signed-off-by: Tim Ekl <lithium3141@gmail.com>
ddcdae6

ddcdae6 fixes a NullPointerException on upgrading from xcode-plugin 1.3.1.

Jenkins » xcode-plugin #43 FAILURE
Looks like there's a problem with this pull request
(what's this?)

Jenkins » xcode-plugin #43 FAILURE
Looks like this commit caused a build failure
(what's this?)

@lacostej lacostej added a commit that referenced this pull request May 16, 2013

@lacostej lacostej Merge pull request #25 from lithium3141/allow-failing-builds
Add new option "allow failing build results"
a2d1da3

@lacostej lacostej merged commit a2d1da3 into jenkinsci:master May 16, 2013

lithium3141 deleted the lithium3141:allow-failing-builds branch May 31, 2013

lithium3141 restored the lithium3141:allow-failing-builds branch May 31, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment