> On March 20, 2015, 10:33 p.m., Joshua Cohen wrote:
> > src/test/python/apache/aurora/client/cli/test_supdate.py, line 636
> > <https://reviews.apache.org/r/32313/diff/5/?file=901848#file901848line636>
> >
> >     Yes, let us add stuff! Stuff is great and junk!

Stuff=added.  This pointed out that i neglected to update these tests to the 
new style.  Key result: mock.patch import is now gone!


- Bill


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


On March 20, 2015, 10:31 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32313/
> -----------------------------------------------------------
> 
> (Updated March 20, 2015, 10:31 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen and Zameer Manji.
> 
> 
> Bugs: AURORA-1206
>     https://issues.apache.org/jira/browse/AURORA-1206
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> One change i snuck in here is in `cli/__init__.py`.  This makes the 
> subcommand help include the description:
> 
> ```
> $ aurora update info -h
> usage: aurora update info [-h] [--write-json] [--verbose]
>                           [--skip-hooks hook,hook,...]
>                           CLUSTER/ROLE/ENV/NAME [ID]
> 
> Display detailed status information about a scheduler-driven in-progress
> update. If no update ID is provided, information will be displayed about the
> active update for the job.
> 
> positional arguments:
>   CLUSTER/ROLE/ENV/NAME
>                         Fully specified job key, in CLUSTER/ROLE/ENV/NAME
>                         format
>   ID                    Update identifier provided by the scheduler when an
>                         update was started.
> 
> optional arguments:
>   -h, --help            show this help message and exit
>   --write-json          Generate command output in JSON format
>   --verbose, -v         Show verbose output
>   --skip-hooks hook,hook,...
>                         A comma-separated list of command hook names that
>                         should be skipped. If the hooks cannot be skipped,
>                         then the command will be aborted
> ```
> 
> Prior to this change, the description was only displayed in the parent 
> command's help text.
> 
> 
> Diffs
> -----
> 
>   docs/client-commands.md fe13fb380010f20e704608c4e82db0e5b93298ab 
>   src/main/python/apache/aurora/client/cli/__init__.py 
> 6a0c129bc5d5dac8d8d393705a69586c9918983d 
>   src/main/python/apache/aurora/client/cli/update.py 
> 830ef4424fe46bc8c14456492f29dea681cf5200 
>   src/test/python/apache/aurora/client/cli/test_supdate.py 
> f9acbdfd65adb252f3059717a6bc1a1f4ba39c44 
>   src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh 
> 320c1fbeee0161528745edd38360cd1fd5d53104 
> 
> Diff: https://reviews.apache.org/r/32313/diff/
> 
> 
> Testing
> -------
> 
> I have converted all test cases in `test_supdate.py` to use the 'new style' 
> non-integration testing, which removed a ton of boilerplate.
> 
> I also corrected some holes in the end-to-end tests, wherein `test` and 
> conditions could silently fail.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Reply via email to