----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29984/#review68836 -----------------------------------------------------------
Ship it! Master (116ee2d) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Jan. 16, 2015, 9:45 p.m., Joshua Cohen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29984/ > ----------------------------------------------------------- > > (Updated Jan. 16, 2015, 9:45 p.m.) > > > Review request for Aurora, Kevin Sweeney and Brian Wickman. > > > Repository: aurora > > > Description > ------- > > Minor clean up to mock usage in resource manager integration tests. > > We didn't need to wrap the mock disk collector in a lambda, just rather than > asserting on the mock class, we instead needed to assert on its return value > (the instance). > > > Diffs > ----- > > > src/test/python/apache/aurora/executor/common/test_resource_manager_integration.py > bb3045700c2870329e175db44873dcb990226c7a > > Diff: https://reviews.apache.org/r/29984/diff/ > > > Testing > ------- > > ./pants build > src/test/python/apache/aurora/executor/common:resource_manager_integration > > > Thanks, > > Joshua Cohen > >
