> On Aug. 21, 2014, 4:52 p.m., Maxim Khutornenko wrote: > > src/test/python/apache/aurora/client/cli/test_create.py, lines 256-257 > > <https://reviews.apache.org/r/24953/diff/1/?file=666552#file666552line256> > > > > Any change to add some asserts here that would verify instances_binding > > and cluster_binding values?
Those actually do: without the binds, the config would have invalid strings in it; with the binds, the config ends up complete, and the mock_query checks that the right values were used. - Mark ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24953/#review51218 ----------------------------------------------------------- On Aug. 21, 2014, 4:44 p.m., Mark Chu-Carroll wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24953/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2014, 4:44 p.m.) > > > Review request for Aurora, David McLaughlin and Maxim Khutornenko. > > > Bugs: aurora-659 > https://issues.apache.org/jira/browse/aurora-659 > > > Repository: aurora > > > Description > ------- > > Also, add new unit test to prevent regressions. > > > Diffs > ----- > > src/main/python/apache/aurora/client/cli/context.py > 0e10589c2e8ab6decedb12122a0d8c7a81285e18 > src/main/python/apache/aurora/client/cli/options.py > bb61dbe14e46cfab609e26e36e4e7e176d9d0e50 > src/test/python/apache/aurora/client/cli/test_create.py > ca635bd49acec9596db67839451101a59ba4d761 > src/test/python/apache/aurora/client/cli/util.py > 552537add91a76b17736b456c78034a38940fa67 > > Diff: https://reviews.apache.org/r/24953/diff/ > > > Testing > ------- > > [sun-wukong incubator-aurora (fix_bindings)]$ ./pants > src/test/python/apache/aurora/client/cli:all > Build operating on top level addresses: > set([BuildFileAddress(/Users/mchucarroll/Code/incubator-aurora/src/test/python/apache/aurora/client/cli/BUILD, > all)]) > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 3 items > > src/test/python/apache/aurora/client/cli/test_config_noun.py ... > > =========================== 3 passed in 0.97 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 5 items > > src/test/python/apache/aurora/client/cli/test_sla.py ..... > > =========================== 5 passed in 0.95 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 4 items > > src/test/python/apache/aurora/client/cli/test_bridge.py .... > > =========================== 4 passed in 0.03 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 3 items > > src/test/python/apache/aurora/client/cli/test_task_run.py ... > > =========================== 3 passed in 0.91 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 7 items > > src/test/python/apache/aurora/client/cli/test_cron.py ....... > > =========================== 7 passed in 1.26 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 45 items > > src/test/python/apache/aurora/client/cli/test_cancel_update.py .. > src/test/python/apache/aurora/client/cli/test_create.py ....... > src/test/python/apache/aurora/client/cli/test_diff.py ... > src/test/python/apache/aurora/client/cli/test_kill.py ............. > src/test/python/apache/aurora/client/cli/test_open.py ..... > src/test/python/apache/aurora/client/cli/test_restart.py ..... > src/test/python/apache/aurora/client/cli/test_status.py ....... > src/test/python/apache/aurora/client/cli/test_update.py ... > > ========================== 45 passed in 7.91 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 1 items > > src/test/python/apache/aurora/client/cli/test_inspect.py . > > =========================== 1 passed in 0.73 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 4 items > > src/test/python/apache/aurora/client/cli/test_quota.py .... > > =========================== 4 passed in 0.79 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 4 items > > src/test/python/apache/aurora/client/cli/test_plugins.py .... > > =========================== 4 passed in 0.86 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 2 items > > src/test/python/apache/aurora/client/cli/test_api_from_cli.py .. > > =========================== 2 passed in 0.75 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 2 items > > src/test/python/apache/aurora/client/cli/test_logging.py .. > > =========================== 2 passed in 0.78 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 5 items > > src/test/python/apache/aurora/client/cli/test_help.py ..... > > =========================== 5 passed in 0.71 seconds > =========================== > ============================= test session starts > ============================== > platform darwin -- Python 2.7.5 -- py-1.4.23 -- pytest-2.6.1 > plugins: cov, timeout > collected 9 items > > src/test/python/apache/aurora/client/cli/test_command_hooks.py ......... > > =========================== 9 passed in 0.90 seconds > =========================== > src.test.python.apache.aurora.client.cli.api > ..... SUCCESS > src.test.python.apache.aurora.client.cli.bridge > ..... SUCCESS > src.test.python.apache.aurora.client.cli.command_hooks > ..... SUCCESS > src.test.python.apache.aurora.client.cli.config > ..... SUCCESS > src.test.python.apache.aurora.client.cli.cron > ..... SUCCESS > src.test.python.apache.aurora.client.cli.help > ..... SUCCESS > src.test.python.apache.aurora.client.cli.inspect > ..... SUCCESS > src.test.python.apache.aurora.client.cli.job > ..... SUCCESS > src.test.python.apache.aurora.client.cli.logging > ..... SUCCESS > src.test.python.apache.aurora.client.cli.plugins > ..... SUCCESS > src.test.python.apache.aurora.client.cli.quota > ..... SUCCESS > src.test.python.apache.aurora.client.cli.sla > ..... SUCCESS > src.test.python.apache.aurora.client.cli.task > ..... SUCCESS > [sun-wukong incubator-aurora (fix_bindings)]$ > > > Thanks, > > Mark Chu-Carroll > >