Re: Review Request 24852: Add command output tests for "job create", "job killall", "job kill"

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24852/ --- (Updated Aug. 22, 2014, 11:04 a.m.) Review request for Aurora, David McLaughlin

Re: Review Request 24852: Add command output tests for "job create", "job killall", "job kill"

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24852/#review50994 --- Ship it! Ship It! - David McLaughlin On Aug. 19, 2014, 3:22 p.m.

Re: Review Request 24852: Add command output tests for "job create", "job killall", "job kill"

> On Aug. 19, 2014, 3:47 p.m., Maxim Khutornenko wrote: > > src/test/python/apache/aurora/client/cli/test_create.py, lines 233-234 > > > > > > This is the existing test, right? Diff shows it in green for some > > reas

Re: Review Request 24852: Add command output tests for "job create", "job killall", "job kill"

> On Aug. 19, 2014, 11:47 a.m., Maxim Khutornenko wrote: > > src/test/python/apache/aurora/client/cli/test_create.py, lines 233-234 > > > > > > This is the existing test, right? Diff shows it in green for some > > rea

Re: Review Request 24852: Add command output tests for "job create", "job killall", "job kill"

> On Aug. 19, 2014, 3:47 p.m., Maxim Khutornenko wrote: > > src/test/python/apache/aurora/client/cli/test_create.py, lines 233-234 > > > > > > This is the existing test, right? Diff shows it in green for some > > reas

Re: Review Request 24852: Add command output tests for "job create", "job killall", "job kill"

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24852/#review50981 --- Ship it! Ship It! - Maxim Khutornenko On Aug. 19, 2014, 3:22 p.m

Re: Review Request 24852: Add command output tests for "job create", "job killall", "job kill"

> On Aug. 19, 2014, 11:47 a.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/client/cli/jobs.py, lines 119-121 > > > > > > General question: what happens to a message returned by the response in > > case

Re: Review Request 24852: Add command output tests for "job create", "job killall", "job kill"

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24852/#review50974 --- src/main/python/apache/aurora/client/cli/jobs.py

Review Request 24852: Add command output tests for "job create", "job killall", "job kill"

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24852/ --- Review request for Aurora, David McLaughlin and Maxim Khutornenko. Bugs: aurora