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

Unified Diff: build/android/pylib/instrumentation/test_jar.py

Issue 132463007: Enable presubmit pylint in build/android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 11 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
Index: build/android/pylib/instrumentation/test_jar.py
diff --git a/build/android/pylib/instrumentation/test_jar.py b/build/android/pylib/instrumentation/test_jar.py
index 21ce8402b15a8fd2e50c105f19caeae369b3b212..5678215dc9476d3259330b5f3a5eacdc0b72f370 100644
--- a/build/android/pylib/instrumentation/test_jar.py
+++ b/build/android/pylib/instrumentation/test_jar.py
@@ -127,7 +127,8 @@ class TestJar(object):
def _GetAnnotationMap(self):
return self._annotation_map
- def _IsTestMethod(self, test):
+ @staticmethod
+ def _IsTestMethod(test):
class_name, method = test.split('#')
return class_name.endswith('Test') and method.startswith('test')
@@ -137,7 +138,8 @@ class TestJar(object):
return []
return self._GetAnnotationMap()[test]
- def _AnnotationsMatchFilters(self, annotation_filter_list, annotations):
+ @staticmethod
+ def _AnnotationsMatchFilters(annotation_filter_list, annotations):
"""Checks if annotations match any of the filters."""
if not annotation_filter_list:
return True
@@ -173,7 +175,7 @@ class TestJar(object):
tests_missing_annotations.append(test_method)
return sorted(tests_missing_annotations)
- def _GetAllMatchingTests(self, annotation_filter_list,
+ def GetAllMatchingTests(self, annotation_filter_list,
exclude_annotation_list, test_filter):
"""Get a list of tests matching any of the annotations and the filter.

Powered by Google App Engine
This is Rietveld 408576698