> On May 19, 2016, 4:31 p.m., David McLaughlin wrote: > > src/test/python/apache/aurora/client/cli/test_supdate.py, lines 223-226 > > <https://reviews.apache.org/r/47550/diff/1/?file=1387328#file1387328line223> > > > > Can you add tests to show what happens when the instance ranges will be > > empty?
Test scenario test_show_job_update_diff_no_change in test_diff_formatter.py has been added to cover this. - Mehrdad ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47550/#review133966 ----------------------------------------------------------- On May 23, 2016, 9:38 p.m., Mehrdad Nurolahzade wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47550/ > ----------------------------------------------------------- > > (Updated May 23, 2016, 9:38 p.m.) > > > Review request for Aurora. > > > Repository: aurora > > > Description > ------- > > AURORA-1492 Improve "aurora update start" command output > > > Diffs > ----- > > src/main/python/apache/aurora/client/cli/diff_formatter.py PRE-CREATION > src/main/python/apache/aurora/client/cli/jobs.py > e8bc38aaff42579419130b116389d9b9e09122a9 > src/main/python/apache/aurora/client/cli/update.py > eb7e9b0799bb25af652a0d5bde231cfca5bc1510 > src/test/python/apache/aurora/client/cli/test_diff.py > b9e91cf3be6ebd6d99e588ba2a2df304e42fd832 > src/test/python/apache/aurora/client/cli/test_diff_formatter.py > PRE-CREATION > src/test/python/apache/aurora/client/cli/test_supdate.py > 2135ca988ce2f260262145a2168849acd8d14874 > > Diff: https://reviews.apache.org/r/47550/diff/ > > > Testing > ------- > > Automated: ./pants test.pytest --no-fast src/test/python:: > Manual: using vagrant > e2e: src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh > Style: ./build-support/hooks/pre-commit > > Example output: > > aurora update start devcluster/www-data/prod/hello > ./aurora/examples/jobs/updated_hello_world.aurora > INFO] Starting update for: hello > Job update has started. View your update progress at > http://aurora.local:8081/scheduler/www-data/prod/hello/update/df5ba4d7-6a32-42ca-8453-1b172f61a23b > This job update will: > add instances: [2-3] > update instances: [0-1] > > > Thanks, > > Mehrdad Nurolahzade > >