Add coverage

tools/telemetry/third_party/coverage/pylintrc

Index: tools/telemetry/third_party/coverage/pylintrc
diff --git a/tools/telemetry/third_party/gsutilz/third_party/boto/pylintrc b/tools/telemetry/third_party/coverage/pylintrc
similarity index 71%
copy from tools/telemetry/third_party/gsutilz/third_party/boto/pylintrc
copy to tools/telemetry/third_party/coverage/pylintrc
index 723cbcb0b90ac93547b0f40db73774552f399125..09ac14166a675c9783fd2c044d01a0c85833746d 100644
--- a/tools/telemetry/third_party/gsutilz/third_party/boto/pylintrc
+++ b/tools/telemetry/third_party/coverage/pylintrc
@@ -1,6 +1,9 @@
+# Licensed under the Apache License: http://www.apache.org/licenses/LICENSE-2.0
+# For details: https://bitbucket.org/ned/coveragepy/src/default/NOTICE.txt
+
 # lint Python modules using external checkers.
 # 
-# This is the main checker controlling the other ones and the reports
+# This is the main checker controling the other ones and the reports
 # generation. It is itself both a raw checker and an astng checker in order
 # to:
 # * handle message activation / deactivation at the module level
@@ -8,19 +11,22 @@
 # 
 [MASTER]
 
-
 # Specify a configuration file.
 #rcfile=
 
+# Python code to execute, usually for sys.path manipulation such as
+# pygtk.require().
+#init-hook=
+
 # Profiled execution.
 profile=no
 
 # Add <file or directory> to the black list. It should be a base name, not a
 # path. You may set this option multiple times.
-ignore=.svn
+ignore=
 
 # Pickle collected data for later comparisons.
-persistent=yes
+persistent=no
 
 # Set the cache size for astng objects.
 cache-size=500
@@ -32,12 +38,12 @@ load-plugins=
 
 [MESSAGES CONTROL]
 
-# Enable only checker(s) with the given id(s). This option conflict with the
+# Enable only checker(s) with the given id(s). This option conflicts with the
 # disable-checker option
 #enable-checker=
 
-# Enable all checker(s) except those with the given id(s). This option conflict
-# with the disable-checker option
+# Enable all checker(s) except those with the given id(s). This option
+# conflicts with the enable-checker option
 #disable-checker=
 
 # Enable all messages in the listed categories.
@@ -47,20 +53,43 @@ load-plugins=
 #disable-msg-cat=
 
 # Enable the message(s) with the given id(s).
-#enable-msg=
+enable=
+#   I0021: Useless suppression
+    I0021
 
 # Disable the message(s) with the given id(s).
-# disable-msg=C0323,W0142,C0301,C0103,C0111,E0213,C0302,C0203,W0703,R0201
-disable-msg=C0301,C0111,C0103,R0201,W0702,C0324
+disable= 
+    spelling,
+# Messages that are just silly:
+#   I0011:106: Locally disabling E1101
+#   W0122: 30:run_python_file: Use of the exec statement
+#   W0142: 31:call_singleton_method: Used * or ** magic
+#   W0232:  6:AnyOldObject: Class has no __init__ method
+#   C0323:311:coverage.report: Operator not followed by a space
+#   C0324: 15: Comma not followed by a space
+#   W0603: 28:call_singleton_method: Using the global statement
+#   W0703:133:CoverageData._read_file: Catch "Exception"
+    I0011,W0122,W0142,W0232,C0323,C0324,W0603,W0703,
+# Messages that may be silly:
+#   R0201: 42:Tracer.stop: Method could be a function
+#   E1103: 26:RunTests.test_run_python_file: Instance of 'file' has no 'getvalue' member (but some types could not be inferred)
+    R0201,E1103,
+# formatting stuff
+    superfluous-parens,bad-continuation,
+# Messages that are noisy for now, eventually maybe we'll turn them on:
+#   C0103:256:coverage.morf_filename: Invalid name "f" (should match [a-z_][a-z0-9_]{2,30}$)
+#   W0212: 86:Reporter.report_files: Access to a protected member _analyze of a client class     
+    C0103,W0212,
+    duplicate-code,
+    cyclic-import
+
+msg-template={path}:{line}: {msg} ({symbol})
 
 [REPORTS]
 
-# set the output format. Available formats are text, parseable, colorized and
-# html
-output-format=colorized
-
-# Include message's id in output
-include-ids=yes
+# set the output format. Available formats are text, parseable, colorized, msvs
+# (visual studio) and html
+output-format=text
 
 # Put messages in a separate file for each module / package specified on the
 # command line instead of printing them on stdout. Reports (if any) will be
@@ -68,7 +97,7 @@ include-ids=yes
 files-output=no
 
 # Tells wether to display a full report or only the messages
-reports=yes
+reports=no
 
 # Python expression which should return a note less than 10 (10 is the highest
 # note).You have access to the variables errors warning, statement which
@@ -87,46 +116,11 @@ comment=no
 # Disable the report(s) with the given id(s).
 #disable-report=
 
-# checks for
-# * unused variables / imports
-# * undefined variables
-# * redefinition of variable from builtins or from an outer scope
-# * use of variable before assigment
-# 
-[VARIABLES]
-
-# Tells wether we should check for unused import in __init__ files.
-init-import=yes
-
-# A regular expression matching names used for dummy variables (i.e. not used).
-dummy-variables-rgx=_|dummy
-
-# List of additional names supposed to be defined in builtins. Remember that
-# you should avoid to define new builtins when possible.
-additional-builtins=
-
-
-# try to find bugs in the code using type inference
-# 
-[TYPECHECK]
-
-# Tells wether missing members accessed in mixin class should be ignored. A
-# mixin class is detected if its name ends with "mixin" (case insensitive).
-ignore-mixin-members=yes
-
-# When zope mode is activated, consider the acquired-members option to ignore
-# access to some undefined attributes.
-zope=no
-
-# List of members which are usually get through zope's acquisition mecanism and
-# so shouldn't trigger E0201 when accessed (need zope=yes to be considered).
-acquired-members=REQUEST,acl_users,aq_parent
-
 
 # checks for :
 # * doc strings
 # * modules / classes / functions / methods / arguments / variables name
-# * number of arguments, local variables, branches, returns and statements in
+# * number of arguments, local variables, branchs, returns and statements in
 # functions, methods
 # * required module attributes
 # * dangerous default values as arguments
@@ -140,13 +134,13 @@ required-attributes=
 
 # Regular expression which should only match functions or classes name which do
 # not require a docstring
-no-docstring-rgx=__.*__
+no-docstring-rgx=__.*__|test[A-Z_].*|setUp|tearDown
 
 # Regular expression which should only match correct module names
 module-rgx=(([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+))$
 
 # Regular expression which should only match correct module level names
-const-rgx=(([A-Z_][A-Z1-9_]*)|(__.*__))$
+const-rgx=(([A-Z_][A-Z0-9_]*)|(__.*__))$
 
 # Regular expression which should only match correct class names
 class-rgx=[A-Z_][a-zA-Z0-9]+$
@@ -155,7 +149,7 @@ class-rgx=[A-Z_][a-zA-Z0-9]+$
 function-rgx=[a-z_][a-z0-9_]{2,30}$
 
 # Regular expression which should only match correct method names
-method-rgx=[a-z_][a-z0-9_]{2,30}$
+method-rgx=[a-z_][a-z0-9_]{2,30}$|setUp|tearDown|test_.*
 
 # Regular expression which should only match correct instance attribute names
 attr-rgx=[a-z_][a-z0-9_]{2,30}$
@@ -177,7 +171,65 @@ good-names=i,j,k,ex,Run,_
 bad-names=foo,bar,baz,toto,tutu,tata
 
 # List of builtins function names that should not be used, separated by a comma
-bad-functions=apply,input
+bad-functions=
+
+
+# try to find bugs in the code using type inference
+# 
+[TYPECHECK]
+
+# Tells wether missing members accessed in mixin class should be ignored. A
+# mixin class is detected if its name ends with "mixin" (case insensitive).
+ignore-mixin-members=yes
+
+# List of classes names for which member attributes should not be checked
+# (useful for classes with attributes dynamicaly set).
+ignored-classes=SQLObject
+
+# When zope mode is activated, consider the acquired-members option to ignore
+# access to some undefined attributes.
+zope=no
+
+# List of members which are usually get through zope's acquisition mecanism and
+# so shouldn't trigger E0201 when accessed (need zope=yes to be considered).
+acquired-members=REQUEST,acl_users,aq_parent
+
+
+# checks for
+# * unused variables / imports
+# * undefined variables
+# * redefinition of variable from builtins or from an outer scope
+# * use of variable before assigment
+# 
+[VARIABLES]
+
+# Tells wether we should check for unused import in __init__ files.
+init-import=no
+
+# A regular expression matching names used for dummy variables (i.e. not used).
+dummy-variables-rgx=_|dummy|unused|.*_unused
+
+# List of additional names supposed to be defined in builtins. Remember that
+# you should avoid to define new builtins when possible.
+additional-builtins=
+
+
+# checks for :
+# * methods without self as first argument
+# * overridden methods signature
+# * access only to existant members via self
+# * attributes not defined in the __init__ method
+# * supported interfaces implementation
+# * unreachable code
+# 
+[CLASSES]
+
+# List of interface methods to ignore, separated by a comma. This is used for
+# instance to not check methods defines in Zope's Interface base class.
+ignore-iface-methods=isImplementedBy,deferred,extends,names,namesAndDescriptions,queryDescriptionFor,getBases,getDescriptionFor,getDoc,getName,getTaggedValue,getTaggedValueTags,isEqualOrExtendedBy,setTaggedValue,isImplementedByInstancesOf,adaptWith,is_implemented_by
+
+# List of method names used to declare (i.e. assign) instance attributes.
+defining-attr-methods=__init__,__new__,setUp,reset
 
 
 # checks for sign of poor/misdesign:
@@ -187,31 +239,31 @@ bad-functions=apply,input
 [DESIGN]
 
 # Maximum number of arguments for function / method
-max-args=12
+max-args=15
 
 # Maximum number of locals for function / method body
-max-locals=30
+max-locals=50
 
 # Maximum number of return / yield for function / method body
-max-returns=12
+max-returns=20
 
 # Maximum number of branch for function / method body
-max-branchs=30
+max-branches=50
 
 # Maximum number of statements in function / method body
-max-statements=60
+max-statements=150
 
 # Maximum number of parents for a class (see R0901).
-max-parents=7
+max-parents=12
 
 # Maximum number of attributes for a class (see R0902).
-max-attributes=20
+max-attributes=40
 
 # Minimum number of public methods for a class (see R0903).
 min-public-methods=0
 
 # Maximum number of public methods for a class (see R0904).
-max-public-methods=20
+max-public-methods=500
 
 
 # checks for
@@ -239,21 +291,32 @@ int-import-graph=
 
 
 # checks for :
-# * methods without self as first argument
-# * overridden methods signature
-# * access only to existant members via self
-# * attributes not defined in the __init__ method
-# * supported interfaces implementation
-# * unreachable code
+# * unauthorized constructions
+# * strict indentation
+# * line length
+# * use of <> instead of !=
 # 
-[CLASSES]
+[FORMAT]
 
-# List of interface methods to ignore, separated by a comma. This is used for
-# instance to not check methods defines in Zope's Interface base class.
-# ignore-iface-methods=isImplementedBy,deferred,extends,names,namesAndDescriptions,queryDescriptionFor,getBases,getDescriptionFor,getDoc,getName,getTaggedValue,getTaggedValueTags,isEqualOrExtendedBy,setTaggedValue,isImplementedByInstancesOf,adaptWith,is_implemented_by
+# Maximum number of characters on a single line.
+max-line-length=100
 
-# List of method names used to declare (i.e. assign) instance attributes.
-defining-attr-methods=__init__,__new__,setUp
+# Maximum number of lines in a module
+max-module-lines=10000
+
+# String used as indentation unit. This is usually " " (4 spaces) or "\t" (1
+# tab).
+indent-string='    '
+
+
+# checks for:
+# * warning notes in the code like FIXME, XXX
+# * PEP 263: source code with non ascii character but no encoding declaration
+# 
+[MISCELLANEOUS]
+
+# List of note tags to take in consideration, separated by a comma.
+notes=FIXME,XXX,TODO
 
 
 # checks for similarities and duplicated code. This computation may be
@@ -263,7 +326,7 @@ defining-attr-methods=__init__,__new__,setUp
 [SIMILARITIES]
 
 # Minimum lines number of a similarity.
-min-similarity-lines=5
+min-similarity-lines=4
 
 # Ignore comments when computing similarities.
 ignore-comments=yes
@@ -271,31 +334,12 @@ ignore-comments=yes
 # Ignore docstrings when computing similarities.
 ignore-docstrings=yes
 
+#
+# SPELLING
+#
 
-# checks for:
-# * warning notes in the code like FIXME, XXX
-# * PEP 263: source code with non ascii character but no encoding declaration
-# 
-[MISCELLANEOUS]
-
-# List of note tags to take in consideration, separated by a comma.
-notes=FIXME,XXX,TODO,BUG:
-
-
-# checks for :
-# * unauthorized constructions
-# * strict indentation
-# * line length
-# * use of <> instead of !=
-# 
-[FORMAT]
-
-# Maximum number of characters on a single line.
-max-line-length=80
-
-# Maximum number of lines in a module
-max-module-lines=1000
-
-# String used as indentation unit. This is usually " " (4 spaces) or "\t" (1
-# tab).
-indent-string='    '
+spelling-dict=en_US
+# pylint doesn't strip the words, so insert a dummy x at the beginning to make
+# the other words work properly.
+# https://bitbucket.org/logilab/pylint/issue/398/spelling-words-need-to-be-stripped-or-the
+spelling-private-dict-file=doc/dict.txt