add import for traceback, and add test to exercise that code path, rm unsed kwargs

Review Request #1943 - Created March 19, 2015 and submitted

Information
Nick Howard (Twitter)
pants
1287
345d7c6...
Reviewers
pants-reviews
jsirois, mateor, stuhood

address Mateo's comments on https://rbcommons.com/s/twitter/r/1868

I've got a TODO to write some more test coverage for various publish paths to better cover the existing paths to catch regressions like this better.

wrote a regression in test_jar_publish, fixed regression. ci baking https://travis-ci.org/pantsbuild/pants/builds/54957239

Nick Howard (Twitter)
John Sirois
Mateo Rodriguez
Nick Howard (Twitter)
Review request changed

Status: Closed (submitted)

Change Summary:

merged at 6d670fab04e937d0885eca4006f2c97d47ad41c5

Loading...