| OLD | NEW |
| 1 # Copyright (C) 2010 Chris Jerdonek (cjerdonek@webkit.org) | 1 # Copyright (C) 2010 Chris Jerdonek (cjerdonek@webkit.org) |
| 2 # | 2 # |
| 3 # Redistribution and use in source and binary forms, with or without | 3 # Redistribution and use in source and binary forms, with or without |
| 4 # modification, are permitted provided that the following conditions | 4 # modification, are permitted provided that the following conditions |
| 5 # are met: | 5 # are met: |
| 6 # 1. Redistributions of source code must retain the above copyright | 6 # 1. Redistributions of source code must retain the above copyright |
| 7 # notice, this list of conditions and the following disclaimer. | 7 # notice, this list of conditions and the following disclaimer. |
| 8 # 2. Redistributions in binary form must reproduce the above copyright | 8 # 2. Redistributions in binary form must reproduce the above copyright |
| 9 # notice, this list of conditions and the following disclaimer in the | 9 # notice, this list of conditions and the following disclaimer in the |
| 10 # documentation and/or other materials provided with the distribution. | 10 # documentation and/or other materials provided with the distribution. |
| 11 # | 11 # |
| 12 # THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND | 12 # THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND |
| 13 # ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED | 13 # ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED |
| 14 # WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE | 14 # WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE |
| 15 # DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE FOR | 15 # DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE FOR |
| 16 # ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL | 16 # ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL |
| 17 # DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR | 17 # DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR |
| 18 # SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER | 18 # SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER |
| 19 # CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, | 19 # CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, |
| 20 # OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | 20 # OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
| 21 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | 21 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
| 22 | 22 |
| 23 """Unit tests for python.py.""" | 23 """Unit tests for python.py.""" |
| 24 | 24 |
| 25 import os | 25 import os |
| 26 import unittest | 26 import unittest |
| 27 | 27 |
| 28 from python import PythonChecker | 28 from webkitpy.style.checkers.python import PythonChecker |
| 29 | 29 |
| 30 | 30 |
| 31 class PythonCheckerTest(unittest.TestCase): | 31 class PythonCheckerTest(unittest.TestCase): |
| 32 | 32 |
| 33 """Tests the PythonChecker class.""" | 33 """Tests the PythonChecker class.""" |
| 34 | 34 |
| 35 def test_init(self): | 35 def test_init(self): |
| 36 """Test __init__() method.""" | 36 """Test __init__() method.""" |
| 37 | 37 |
| 38 def _mock_handle_style_error(self): | 38 def _mock_handle_style_error(self): |
| (...skipping 22 matching lines...) Expand all Loading... |
| 61 self.assertEqual( | 61 self.assertEqual( |
| 62 [ | 62 [ |
| 63 (2, 'pep8/W291', 5, 'trailing whitespace'), | 63 (2, 'pep8/W291', 5, 'trailing whitespace'), |
| 64 (3, 'pep8/E261', 5, 'at least two spaces before inline comment')
, | 64 (3, 'pep8/E261', 5, 'at least two spaces before inline comment')
, |
| 65 (3, 'pep8/E262', 5, "inline comment should start with '# '"), | 65 (3, 'pep8/E262', 5, "inline comment should start with '# '"), |
| 66 (2, 'pylint/C0303(trailing-whitespace)', 5, '[] Trailing whitesp
ace'), | 66 (2, 'pylint/C0303(trailing-whitespace)', 5, '[] Trailing whitesp
ace'), |
| 67 (2, 'pylint/E0602(undefined-variable)', 5, u"[] Undefined variab
le 'error'"), | 67 (2, 'pylint/E0602(undefined-variable)', 5, u"[] Undefined variab
le 'error'"), |
| 68 (3, 'pylint/W0611(unused-import)', 5, '[] Unused import math'), | 68 (3, 'pylint/W0611(unused-import)', 5, '[] Unused import math'), |
| 69 ], | 69 ], |
| 70 errors) | 70 errors) |
| OLD | NEW |