Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2334)

Unified Diff: build/android/pylib/linker/test_case.py

Issue 132463007: Enable presubmit pylint in build/android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase vs tot and only disabling F0401 in specific spots Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/android/pylib/linker/setup.py ('k') | build/android/pylib/linker/test_runner.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/pylib/linker/test_case.py
diff --git a/build/android/pylib/linker/test_case.py b/build/android/pylib/linker/test_case.py
index b4bddc727c749c0cd5979860f45518360397203f..febab92545ca8b8637da80833db5f97e15adebf5 100644
--- a/build/android/pylib/linker/test_case.py
+++ b/build/android/pylib/linker/test_case.py
@@ -33,36 +33,34 @@
build/android/test_runner.py linker
"""
+# pylint: disable=R0201
import logging
import os
import re
-import StringIO
-import subprocess
-import tempfile
import time
from pylib import constants
from pylib import android_commands
-from pylib import flag_changer
from pylib.base import base_test_result
+
ResultType = base_test_result.ResultType
-_PACKAGE_NAME='org.chromium.content_linker_test_apk'
-_ACTIVITY_NAME='.ContentLinkerTestActivity'
-_COMMAND_LINE_FILE='/data/local/tmp/content-linker-test-command-line'
+_PACKAGE_NAME = 'org.chromium.content_linker_test_apk'
+_ACTIVITY_NAME = '.ContentLinkerTestActivity'
+_COMMAND_LINE_FILE = '/data/local/tmp/content-linker-test-command-line'
# Path to the Linker.java source file.
-_LINKER_JAVA_SOURCE_PATH = \
- 'content/public/android/java/src/org/chromium/content/app/Linker.java'
+_LINKER_JAVA_SOURCE_PATH = (
+ 'content/public/android/java/src/org/chromium/content/app/Linker.java')
# A regular expression used to extract the browser shared RELRO configuration
# from the Java source file above.
-_RE_LINKER_BROWSER_CONFIG = \
- re.compile(r'.*BROWSER_SHARED_RELRO_CONFIG\s+=\s+' + \
- 'BROWSER_SHARED_RELRO_CONFIG_(\S+)\s*;.*',
- re.MULTILINE | re.DOTALL)
+_RE_LINKER_BROWSER_CONFIG = re.compile(
+ r'.*BROWSER_SHARED_RELRO_CONFIG\s+=\s+' +
+ 'BROWSER_SHARED_RELRO_CONFIG_(\S+)\s*;.*',
+ re.MULTILINE | re.DOTALL)
# Logcat filters used during each test. Only the 'chromium' one is really
# needed, but the logs are added to the TestResult in case of error, and
@@ -150,7 +148,7 @@ def _CheckLinkerTestStatus(logcat):
return (False, None, None)
-def _WaitForLinkerTestStatus(adb, timeout):
+def _WaitForLinkerTestStatus(_adb, _timeout):
"""Wait up to |timeout| seconds until the full linker test status lines appear
in the logcat being recorded with |adb|.
Args:
@@ -161,6 +159,7 @@ def _WaitForLinkerTestStatus(adb, timeout):
ResultType.TIMEOUT in case of timeout, ResulType.PASS if both status lines
report 'SUCCESS', or ResulType.FAIL otherwise.
"""
+ pass
def _StartActivityAndWaitForLinkerTestStatus(adb, timeout):
@@ -276,7 +275,6 @@ def _CheckLoadAddressRandomization(lib_map_list, process_type):
# For each library, check the randomness of its load addresses.
bad_libs = {}
- success = True
for lib_name, lib_address_list in lib_addr_map.iteritems():
# If all addresses are different, skip to next item.
lib_address_set = set(lib_address_list)
@@ -304,17 +302,17 @@ class LinkerTestCaseBase(object):
"""
self.is_low_memory = is_low_memory
if is_low_memory:
- test_suffix = 'ForLowMemoryDevice'
+ test_suffix = 'ForLowMemoryDevice'
else:
- test_suffix = 'ForRegularDevice'
+ test_suffix = 'ForRegularDevice'
class_name = self.__class__.__name__
self.qualified_name = '%s.%s' % (class_name, test_suffix)
self.tagged_name = self.qualified_name
- def _RunTest(self, adb):
+ def _RunTest(self, _adb):
"""Run the test, must be overriden.
Args:
- adb: An AndroidCommands instance to the device.
+ _adb: An AndroidCommands instance to the device.
Returns:
A (status, log) tuple, where <status> is a ResultType constant, and <log>
is the logcat output captured during the test in case of error, or None
@@ -499,7 +497,7 @@ class LinkerRandomizationTest(LinkerTestCaseBase):
browser_lib_map_list = []
renderer_lib_map_list = []
logs_list = []
- for loop in range(max_loops):
+ for _loop in range(max_loops):
# Start the activity.
result, logs = _StartActivityAndWaitForLinkerTestStatus(adb, timeout=30)
if result == ResultType.TIMEOUT:
« no previous file with comments | « build/android/pylib/linker/setup.py ('k') | build/android/pylib/linker/test_runner.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698