Mailing-List: contact reviews-help@aurora.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: reviews@aurora.incubator.apache.org
Content-Type: multipart/alternative;
 boundary="===============0169092950840712251=="
MIME-Version: 1.0
Subject: Re: Review Request 24852: Add command output tests for "job create",
 "job killall", "job kill"
From: "Mark Chu-Carroll" <mchucarroll@twopensource.com>
To: "Maxim Khutornenko" <maxim@apache.org>,
 "David McLaughlin" <david@dmclaughlin.com>
Cc: "Aurora" <reviews@aurora.incubator.apache.org>,
 "Mark Chu-Carroll" <mchucarroll@twopensource.com>
Date: Tue, 19 Aug 2014 17:39:11 -0000
Message-ID: <20140819173911.1309.2478@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: "Mark Chu-Carroll" <noreply@reviews.apache.org>
References: <20140819154726.7609.38223@reviews.apache.org>
In-Reply-To: <20140819154726.7609.38223@reviews.apache.org>
Reply-To: "Mark Chu-Carroll" <mchucarroll@twopensource.com>

--===============0169092950840712251==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


> On Aug. 19, 2014, 11:47 a.m., Maxim Khutornenko wrote:
> > src/test/python/apache/aurora/client/cli/test_create.py, lines 233-234
> > <https://reviews.apache.org/r/24852/diff/1/?file=664291#file664291line233>
> >
> >     This is the existing test, right? Diff shows it in green for some reason...
> 
> Mark Chu-Carroll wrote:
>     No, this is a new test. It's almost identical to an existing test, except that instead of checking API calls, it's checking output.
> 
> Maxim Khutornenko wrote:
>     Please, drop the TODO in that case. No need in multiplying them.

But don't we want to mark this as a spot that needs to be fixed?


- Mark


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24852/#review50974
-----------------------------------------------------------


On Aug. 19, 2014, 11:22 a.m., Mark Chu-Carroll wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/24852/
> -----------------------------------------------------------
> 
> (Updated Aug. 19, 2014, 11:22 a.m.)
> 
> 
> Review request for Aurora, David McLaughlin and Maxim Khutornenko.
> 
> 
> Bugs: aurora-645
>     https://issues.apache.org/jira/browse/aurora-645
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Add tests to ensure that IO behavior of client commands is correct.
> 
> Improve IO behavior in the IO client, so that users get more information.
> Formorely, many commands succeeded silently, when it would be more helpful to actually tell the user that you did something. (This shouldn't affect scripting, because in these commands, the only result that's needed by a scripter is succeess/failure, which is carried by the exit code.)
> 
> 
> Diffs
> -----
> 
>   src/main/python/apache/aurora/client/cli/context.py 0e10589c2e8ab6decedb12122a0d8c7a81285e18 
>   src/main/python/apache/aurora/client/cli/jobs.py 109ce59e3e8dc1c6f4fb918522718cc47867020d 
>   src/test/python/apache/aurora/client/cli/test_create.py ca635bd49acec9596db67839451101a59ba4d761 
>   src/test/python/apache/aurora/client/cli/test_kill.py ee64908855a4960f44ce96c810e69dd105d2ce5d 
>   src/test/python/apache/aurora/client/cli/test_restart.py 14a08e873058374a9bfe4ffbff8ae7279123f264 
> 
> Diff: https://reviews.apache.org/r/24852/diff/
> 
> 
> Testing
> -------
> 
> Ran test suite.
> 
> 
> Thanks,
> 
> Mark Chu-Carroll
> 
>


--===============0169092950840712251==--