----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67842/#review205799 -----------------------------------------------------------
src/python/cli_new/lib/cli/tests/base.py Lines 282 (patched) <https://reviews.apache.org/r/67842/#comment288710> Can we move this below the try below? We should only down the refcount after we are sure the agent is gone. src/python/cli_new/lib/cli/tests/base.py Lines 397 (patched) <https://reviews.apache.org/r/67842/#comment288711> Same cooment as for the agent. - Kevin Klues On July 6, 2018, 10:02 a.m., Armand Grillet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67842/ > ----------------------------------------------------------- > > (Updated July 6, 2018, 10:02 a.m.) > > > Review request for mesos and Kevin Klues. > > > Bugs: MESOS-8955 > https://issues.apache.org/jira/browse/MESOS-8955 > > > Repository: mesos > > > Description > ------- > > Refactored base of Python CLI tests. > > > Diffs > ----- > > src/python/cli_new/lib/cli/tests/base.py > 4ffa27ca95ddf6575fb0a844d6996890bed4d8c9 > > > Diff: https://reviews.apache.org/r/67842/diff/1/ > > > Testing > ------- > > In `src/python/cli_new/`: > ``` > $ source activate > $ mesos-cli-tests > ``` > Removed one of the binary and tried again to check that the error was correct. > > > Thanks, > > Armand Grillet > >
