[Android] Add a custom pylintrc for build/android/.

build/android/pylib/remote/device/remote_device_test_run.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Run specific test on specific environment."""
 
 import json
 import logging
 import os
 import re
 import shutil
 import string
 import tempfile
 import time
 import zipfile
 
 from devil.utils import zip_utils
 from pylib.base import base_test_result
 from pylib.base import test_run
 from pylib.remote.device import appurify_constants
 from pylib.remote.device import appurify_sanitized
 from pylib.remote.device import remote_device_helper
 
 _DEVICE_OFFLINE_RE = re.compile('error: device not found')
 _LONG_MSG_RE = re.compile('longMsg=')
 _SHORT_MSG_RE = re.compile('shortMsg=')
 
-
 class RemoteDeviceTestRun(test_run.TestRun):
   """Run tests on a remote device."""
 
   _TEST_RUN_KEY = 'test_run'
   _TEST_RUN_ID_KEY = 'test_run_id'
 
   WAIT_TIME = 5
   COMPLETE = 'complete'
   HEARTBEAT_INTERVAL = 300
 
@@ skipping 35 matching lines @@
     """Run the test."""
     if self._env.trigger:
       with appurify_sanitized.SanitizeLogging(self._env.verbose_count,
                                               logging.WARNING):
         test_start_res = appurify_sanitized.api.tests_run(
             self._env.token, self._env.device_type_id, self._app_id,
             self._test_id)
       remote_device_helper.TestHttpResponse(
         test_start_res, 'Unable to run test.')
       self._test_run_id = test_start_res.json()['response']['test_run_id']
-      logging.info('Test run id: %s' % self._test_run_id)
+      logging.info('Test run id: %s', self._test_run_id)
 
     if self._env.collect:
       current_status = ''
       timeout_counter = 0
       heartbeat_counter = 0
       while self._GetTestStatus(self._test_run_id) != self.COMPLETE:
         if self._results['detailed_status'] != current_status:
           logging.info('Test status: %s', self._results['detailed_status'])
           current_status = self._results['detailed_status']
           timeout_counter = 0
@@ skipping 92 matching lines @@
 
     Args:
       results_path: Path to download appurify results zipfile.
 
     Returns:
       Path to downloaded file.
     """
 
     if self._results_temp_dir is None:
       self._results_temp_dir = tempfile.mkdtemp()
-      logging.info('Downloading results to %s.' % self._results_temp_dir)
+      logging.info('Downloading results to %s.', self._results_temp_dir)
       with appurify_sanitized.SanitizeLogging(self._env.verbose_count,
                                               logging.WARNING):
         appurify_sanitized.utils.wget(self._results['results']['url'],
                                       self._results_temp_dir + '/results')
     if results_path:
       logging.info('Copying results to %s', results_path)
       if not os.path.exists(os.path.dirname(results_path)):
         os.makedirs(os.path.dirname(results_path))
       shutil.copy(self._results_temp_dir + '/results', results_path)
     return self._results_temp_dir + '/results'
@@ skipping 32 matching lines @@
         with zipfile.ZipFile(test_with_deps.name, 'w') as zip_file:
           zip_file.write(test_path, host_test, zipfile.ZIP_DEFLATED)
           for h, _ in data_deps:
             if os.path.isdir(h):
               zip_utils.WriteToZipFile(zip_file, h, '.')
               sdcard_files.extend(os.listdir(h))
             else:
               zip_utils.WriteToZipFile(zip_file, h, os.path.basename(h))
               sdcard_files.append(os.path.basename(h))
           for a in extra_apks or ():
-            zip_utils.WriteToZipFile(zip_file, a, os.path.basename(a));
+            zip_utils.WriteToZipFile(zip_file, a, os.path.basename(a))
             additional_apks.append(os.path.basename(a))
 
         config['sdcard_files'] = ','.join(sdcard_files)
         config['host_test'] = host_test
         if additional_apks:
           config['additional_apks'] = ','.join(additional_apks)
         self._test_id = self._UploadTestToDevice(
             'robotium', test_with_deps.name, app_id=self._app_id)
     else:
       self._test_id = self._UploadTestToDevice('robotium', test_path)
 
-    logging.info('Setting config: %s' % config)
+    logging.info('Setting config: %s', config)
     appurify_configs = {}
     if self._env.network_config:
       appurify_configs['network'] = self._env.network_config
     self._SetTestConfig('robotium', config, **appurify_configs)
 
   def _UploadAppToDevice(self, app_path):
     """Upload app to device."""
     logging.info('Uploading %s to remote service as %s.', app_path,
                  self._test_instance.suite)
     with open(app_path, 'rb') as apk_src:
       with appurify_sanitized.SanitizeLogging(self._env.verbose_count,
                                               logging.WARNING):
         upload_results = appurify_sanitized.api.apps_upload(
             self._env.token, apk_src, 'raw', name=self._test_instance.suite)
       remote_device_helper.TestHttpResponse(
           upload_results, 'Unable to upload %s.' % app_path)
       return upload_results.json()['response']['app_id']
 
   def _UploadTestToDevice(self, test_type, test_path, app_id=None):
     """Upload test to device
     Args:
       test_type: Type of test that is being uploaded. Ex. uirobot, gtest..
     """
-    logging.info('Uploading %s to remote service.' % test_path)
+    logging.info('Uploading %s to remote service.', test_path)
     with open(test_path, 'rb') as test_src:
       with appurify_sanitized.SanitizeLogging(self._env.verbose_count,
                                               logging.WARNING):
         upload_results = appurify_sanitized.api.tests_upload(
             self._env.token, test_src, 'raw', test_type, app_id=app_id)
       remote_device_helper.TestHttpResponse(upload_results,
           'Unable to upload %s.' % test_path)
       return upload_results.json()['response']['test_id']
 
   def _SetTestConfig(self, runner_type, runner_configs,
@@ skipping 82 matching lines @@
               base_test_result.ResultType.UNKNOWN))
       elif self._DidDeviceGoOffline():
         self._LogLogcat()
         self._LogAdbTraceLog()
         raise remote_device_helper.RemoteDeviceError(
             'Remote service unable to reach device.', is_infra_error=True)
       else:
         results.AddResult(base_test_result.BaseTestResult(
             'Remote Service detected error.',
             base_test_result.ResultType.UNKNOWN))