----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27058/#review58444 -----------------------------------------------------------
-1: Master (3778330) is red with this patch. ./build-support/jenkins/build.sh Skipping installation of /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/__init__.py (namespace package) Skipping installation of /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/common/__init__.py (namespace package) Installing /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter.common.lang-0.3.0-py2.7-nspkg.pth Successfully installed twitter.checkstyle pyflakes pep8 GitPython twitter.common.app gitdb twitter.common.process twitter.common.log twitter.common.util twitter.common.collections async smmap twitter.common.string twitter.common.options twitter.common.dirutil twitter.common.contextutil twitter.common.lang Cleaning up... F401:ERROR src/test/python/apache/aurora/client/cli/test_task_run.py:024-031 'ScheduledTask' imported but unused |from gen.apache.aurora.api.ttypes import ( | JobKey, | ResponseCode, | ScheduledTask, | ScheduleStatus, | ScheduleStatusResult, | TaskQuery |) E501:ERROR src/test/python/apache/aurora/client/commands/test_maintenance.py:045-046 line too long (101 > 100 characters) | mock_options = Mock(spec_set=['cluster', 'disable_all_hooks', 'duration', 'filename', 'grouping', | 'hosts', 'percentage', 'post_drain_script', 'reason', 'unsafe_hosts_filename', 'verbosity']) - Aurora ReviewBot On Oct. 24, 2014, 10:48 p.m., David McLaughlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27058/ > ----------------------------------------------------------- > > (Updated Oct. 24, 2014, 10:48 p.m.) > > > Review request for Aurora, Mark Chu-Carroll and Zameer Manji. > > > Bugs: AURORA-248 > https://issues.apache.org/jira/browse/AURORA-248 > > > Repository: aurora > > > Description > ------- > > Use of Mock() without a specification considered harmful. I went through and > updated as many mocks as I could. > > Any remaining can be classified as: > > 1) Mocks of classes that cannot be spec'd. Almost all instances of > SchedulerProxy, which uses __getattr__ to delegate to the read or write > client. > 2) Primitives like strings and callback functions or data objects like dicts > and pystachio structs. > 3) Weird mocks that broke code where they really shouldn't have (off the top > of my head - in test_diff.py and commands/test_run.py) - both when they were > spec'd and when they were replaced with real thrift structs. > > > The remaining offenders: > > $ grep -r --include="*.py" "Mock()" src/test/python > src/test/python/apache/aurora/admin/test_host_maintenance.py: > mock_callback = mock.Mock() > src/test/python/apache/aurora/admin/test_host_maintenance.py: > mock_callback = mock.Mock() > src/test/python/apache/aurora/client/api/test_api.py: mock_proxy = Mock() > src/test/python/apache/aurora/client/api/test_api.py: mock_get = Mock() > src/test/python/apache/aurora/client/api/test_api.py: mock_task_config = > Mock() > src/test/python/apache/aurora/client/api/test_job_monitor.py: > self._scheduler = Mock() > src/test/python/apache/aurora/client/api/test_quota_check.py: > self._scheduler = Mock() > src/test/python/apache/aurora/client/api/test_scheduler_client.py: > client._connect_scheduler = mock.MagicMock() > src/test/python/apache/aurora/client/api/test_sla.py: self._scheduler = > Mock() > src/test/python/apache/aurora/client/api/test_task_util.py: scheduler = > Mock() > src/test/python/apache/aurora/client/cli/test_diff.py: job = Mock() > src/test/python/apache/aurora/client/cli/test_diff.py: > job.assignedTask.task.executorConfig.data = Mock() > src/test/python/apache/aurora/client/cli/test_diff.py: > patch('json.loads', return_value=Mock())) as (_, _, subprocess_patch, _): > src/test/python/apache/aurora/client/cli/test_diff.py: > patch('json.loads', return_value=Mock())) as ( > src/test/python/apache/aurora/client/cli/test_diff.py: > patch('json.loads', return_value=Mock())) as ( > src/test/python/apache/aurora/client/cli/test_inspect.py: raw_config = > Mock() > src/test/python/apache/aurora/client/cli/test_inspect.py: mock_task = > Mock() > src/test/python/apache/aurora/client/cli/test_inspect.py: mock_process = > Mock() > src/test/python/apache/aurora/client/cli/test_kill.py: > mock_scheduler_proxy = Mock() > src/test/python/apache/aurora/client/cli/test_kill.py: > mock_scheduler_proxy = Mock() > src/test/python/apache/aurora/client/cli/test_kill.py: > mock_scheduler_proxy = Mock() > src/test/python/apache/aurora/client/cli/test_kill.py: > mock_scheduler_proxy = Mock() > src/test/python/apache/aurora/client/cli/util.py: mock_scheduler_proxy = > Mock() > src/test/python/apache/aurora/client/cli/util.py: mock_scheduler = Mock() > src/test/python/apache/aurora/client/cli/util.py: mock_scheduler_client = > Mock() > src/test/python/apache/aurora/client/cli/util.py: mock_api_factory = Mock() > src/test/python/apache/aurora/client/commands/test_diff.py: job = Mock() > src/test/python/apache/aurora/client/commands/test_diff.py: > job.assignedTask.task.executorConfig.data = Mock() > src/test/python/apache/aurora/client/commands/test_diff.py: > patch('json.loads', return_value=Mock())) as ( > src/test/python/apache/aurora/client/commands/test_diff.py: > patch('json.loads', return_value=Mock())) as ( > src/test/python/apache/aurora/client/commands/test_diff.py: > patch('json.loads', return_value=Mock())) as ( > src/test/python/apache/aurora/client/commands/test_listjobs.py: > mock_options = Mock() > src/test/python/apache/aurora/client/commands/test_listjobs.py: job = > Mock() > src/test/python/apache/aurora/client/commands/test_maintenance.py: > mock_callback = Mock() > src/test/python/apache/aurora/client/commands/test_maintenance.py: > mock_wait = Mock() > src/test/python/apache/aurora/client/commands/util.py: > mock_scheduler_proxy = Mock() > src/test/python/apache/aurora/client/commands/util.py: mock_api_factory = > Mock() > > > Diffs > ----- > > src/test/python/apache/aurora/admin/test_admin_util.py > f5c8c69c1109d15ee3886fb863014c3285240db1 > src/test/python/apache/aurora/client/cli/test_command_hooks.py > 60c75300501c36ac20a97f78ff18b3ca7af30699 > src/test/python/apache/aurora/client/cli/test_cron.py > c7b71c29d44150162fec8066947623fa91815424 > src/test/python/apache/aurora/client/cli/test_diff.py > 78694d7559f2041f27cd2a7e4cb81ca467f63ac2 > src/test/python/apache/aurora/client/cli/test_sla.py > a1a3d8161ba747aa23a5e614e9ae31473d2058c1 > src/test/python/apache/aurora/client/cli/test_task_run.py > 12163df0d2e1e42f2a321603ec10ff9359848216 > src/test/python/apache/aurora/client/cli/util.py > 796c4f9880a0f834a6950472892981e8a6789a97 > src/test/python/apache/aurora/client/commands/test_cancel_update.py > 13aa1fef1d94d46f2837f500606028baa694fa6e > src/test/python/apache/aurora/client/commands/test_create.py > 4a753fb5942555854538047eb947e5465cdff607 > src/test/python/apache/aurora/client/commands/test_diff.py > 9f1d459e51c663b9ad62bbbbb16a8127568662d1 > src/test/python/apache/aurora/client/commands/test_hooks.py > d4d8d3cd15704353d958e1ef6b220eaa37696a4d > src/test/python/apache/aurora/client/commands/test_kill.py > 1e13b926379147295a3a1b3d6ce79a727719dedb > src/test/python/apache/aurora/client/commands/test_maintenance.py > 13d753f6870c9f552903b077e3c38d306ead5bc4 > src/test/python/apache/aurora/client/commands/test_restart.py > efa0849c1f11d9304e2da981dfb9c1d0ff59a15d > src/test/python/apache/aurora/client/commands/test_run.py > 0c395f7a8106acf3d45842a6f536dfb74b71a309 > src/test/python/apache/aurora/client/commands/test_ssh.py > cf9f425b3dd64afe9d8fcfd70495a3c58108824f > src/test/python/apache/aurora/client/commands/test_status.py > 9eb8def26692cf5fbd0c20bc96975125e411f0ba > src/test/python/apache/aurora/client/commands/test_update.py > 555ea0d2727fca61256faf7815945320fcbde55d > > Diff: https://reviews.apache.org/r/27058/diff/ > > > Testing > ------- > > $ ./pants src/test/python/apache/aurora/:all > $ build-support/python/checkstyle-check src/test/ > $ build-support/python/isort-check > > > Thanks, > > David McLaughlin > >