[javascriptlint-commit] SF.net SVN: javascriptlint:[326] trunk/test.py

[<mat...@us...>]

Revision: 326
          http://sourceforge.net/p/javascriptlint/code/326
Author:   matthiasmiller
Date:     2013-10-02 23:16:54 +0000 (Wed, 02 Oct 2013)
Log Message:
-----------
pylint: start work to run pylint on jsl

Modified Paths:
--------------
    trunk/test.py

Modified: trunk/test.py
===================================================================
--- trunk/test.py	2013-10-01 17:12:20 UTC (rev 325)
+++ trunk/test.py	2013-10-02 23:16:54 UTC (rev 326)
@@ -4,6 +4,9 @@
 import re
 import sys
 
+import pylint.lint
+from pylint.reporters.text import TextReporter
+
 import javascriptlint.conf
 import javascriptlint.lint
 
@@ -90,7 +93,93 @@
     all_files.sort()
     return all_files
 
+class _CustomLintReporter(TextReporter):
+    line_format = '{path}({line}): [{msg_id}({symbol}){obj}] {msg}'
+    def __init__(self):
+        TextReporter.__init__(self)      
+        self.msg_count = 0
+
+    def write_message(self, msg):
+        TextReporter.write_message(self, msg)
+        self.msg_count += 1
+
+def _get_python_modules(dir_):
+    for root, dirs, files in os.walk(dir_):
+        if '.svn' in dirs:
+            dirs.remove('.svn')
+        for name in files:
+            if name.endswith('.py'):
+                yield os.path.join(root, name)
+
+def _run_pylint():
+    IGNORE = [
+        'C0111', # Missing docstring
+        'I0011', # Locally disabling warning
+        'R0902', # Too many instance attributes (%s/%s)
+        'R0903', # Too few public methods (%s/%s)
+        'R0904', # Too many public methods (%s/%s)
+        'R0911', # Too many return statements (%s/%s)
+        'R0912', # Too many branches (%s/%s)
+        'R0913', # Too many arguments (%s/%s)
+        'R0914', # Too many local variables (%s/%s)
+        'R0915', # Too many statements (%s/%s)
+        'W0142', # Used * or ** magic
+    ]
+    REVIEW = [
+        'C0103', # Invalid name "%s" (should match %s)
+        'C0202', # Class method should have "cls" as first argument
+        'C0301', # Line too long (%s/%s)
+        'C0303', # Trailing whitespace
+        'C0321', # More than one statement on a single line
+        'C0323', # Operator not followed by a space
+        'C0324', # Comma not followed by a space
+        'C1001', # Old style class
+        'E0602', # Undefined variable %r
+        'E1101', # %s %r has no %r member
+        'E1103', # %s %r has no %r member (but some types could not be inferred)
+        'E1306', # Not enough arguments for format string
+        'F0401', # Cyclic import (%s)
+        'R0201', # Attribute %r defined outside __init__
+        'R0924', # Badly implemented
+        'W0109', # Duplicate key %r in dictionary
+        'W0120', # Else clause on loop without a break statement
+        'W0121', # old-raise-syntax
+        'W0141', # Used builtin function %r
+        'W0201', # Attribute %r defined outside __init__
+        'W0212', # Access to a protected member %s of a client class
+        'W0231', # __init__ method from base class %r is not called
+        'W0232', # Class has no __init__ method
+        'W0301', # Unnecessary semicolon
+        'W0311', # Bad indentation
+        'W0401', # Wildcard import %s
+        'W0403', # Relative import %r
+        'W0511', # TODO
+        'W0611', # Unused import %s
+        'W0612', # unused variable
+        'W0613', # Unused argument %r
+        'W0614', # Unused import %s from wildcard import
+        'W0621', # Redefining name %r from outer scope (line %s)
+        'W0622', # Redefining built-in %r
+        'W0631', # Using possibly undefined loop variable %r
+        'W0632', # unbalanced-tuple-unpacking
+        'W1401', # Anomalous backslash in string
+    ]
+
+    dir_ = os.path.dirname(os.path.abspath(__file__))
+    modules = list(_get_python_modules(dir_))
+    reporter = _CustomLintReporter()
+    pylint.lint.Run([
+        '--reports=n',
+    ] + [
+        '--disable=%s' % code for code in (IGNORE + REVIEW)
+    ] + modules, reporter=reporter, exit=False)
+    if reporter.msg_count:
+        print '\nLint failed!\n'
+        sys.exit(1)
+
 def main():
+    _run_pylint()
+
     haderrors = False
     for file in _get_test_files():
         ext = os.path.splitext(file)[1]
@@ -108,5 +197,9 @@
     sys.exit(haderrors)
 
 if __name__  == '__main__':
-    main()
+    try:
+        main()
+    except KeyboardInterrupt:
+        sys.stderr.write('\n\nCanceled.\n')
+        sys.exit(1)
 

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.