Added test that failed HTTP responses do not raise exceptions in artifact cache

Review Request #2644 - Created Aug. 17, 2015 and submitted

Information
Eric Ayers
pants
zundel/add-multiprocess-cache-fail
1979, 2003
1a646bc...
Reviewers
pants-reviews
benjyw, gmalmquist, jsirois

Followon to https://rbcommons.com/s/twitter/r/2624/ which adds a test for issue 1979, where failed result codes from the HTTP server caused a NonfatalArtifactException to escape from the artifact_cache logic and cause the build to fail.

Manually removed the try/except block from artifact_cache.py and the newly added test fails with a NonfatalArtifactCache exception.

CI Passed at https://travis-ci.org/pantsbuild/pants/builds/75937803

John Sirois
Eric Ayers
Review request changed

Status: Closed (submitted)

Change Summary:

Thanks John. Commit c9ce16e

Loading...