----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29132/#review65430 -----------------------------------------------------------
Ship it! Ship It! - Joe Smith On Dec. 16, 2014, 4:57 p.m., Brian Wickman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29132/ > ----------------------------------------------------------- > > (Updated Dec. 16, 2014, 4:57 p.m.) > > > Review request for Aurora and Kevin Sweeney. > > > Repository: aurora > > > Description > ------- > > get_client_version should never be allowed to run unpatched in test. > > > Diffs > ----- > > src/test/python/apache/aurora/client/cli/test_help.py > 1576edcaf0f59911c1c7aec42052d5ebfe41caac > > Diff: https://reviews.apache.org/r/29132/diff/ > > > Testing > ------- > > get_client_version tries to get the PEX-INFO of sys.argv[0], which is usually > pants when running tests (!) if the format of PEX-INFO changes to a new > version of pants (as is the case for pants 0.0.25), then tests will start > failing with an inscrutible error. instead, just patch get_client_version to > something sensible. > > > Thanks, > > Brian Wickman > >
