diff --git a/.travis.yml b/.travis.yml index c3c175233..5f4d26dff 100644 --- a/.travis.yml +++ b/.travis.yml @@ -40,11 +40,11 @@ matrix: - python: 3.5 env: VERSION="" TESTS="0" LINT_CHECK="1" - PYLINT_EXPECTED_ERRORS="35" + PYLINT_EXPECTED_ERRORS="36" - python: 3.7-dev env: VERSION="" TESTS="0" LINT_CHECK="1" - PYLINT_EXPECTED_ERRORS="35" + PYLINT_EXPECTED_ERRORS="36" - python: 3.5 env: VERSION="11.0" ODOO_REPO="OCA/OCB" TESTS="1" LINT_CHECK="0" diff --git a/tests/test_repo/broken_module/model.py b/tests/test_repo/broken_module/model.py index cddbda050..89afa6870 100755 --- a/tests/test_repo/broken_module/model.py +++ b/tests/test_repo/broken_module/model.py @@ -1,4 +1,6 @@ # missing coding +import unittest + from openerp.osv import orm from openerp import fields @@ -54,6 +56,11 @@ def method_w1401(self): my_regex_str_good = r'\d' return my_regex_str_bad, my_regex_str_good + # Reproduce issue https://github.com/OCA/pylint-odoo/issues/243 + @unittest.skipIf(lambda self: self.method_w1401(), "") + def my_method(self): + pass + if __name__ == '__main__': diff --git a/travis/cfg/travis_run_pylint.cfg b/travis/cfg/travis_run_pylint.cfg index 7a2b8c592..ae0e96166 100644 --- a/travis/cfg/travis_run_pylint.cfg +++ b/travis/cfg/travis_run_pylint.cfg @@ -31,7 +31,7 @@ enable=anomalous-backslash-in-string, missing-manifest-dependency, pointless-statement, pointless-string-statement, - print-statement, + print-used, redundant-keyword-arg, reimported, relative-import, diff --git a/travis/self_tests b/travis/self_tests index ba518b978..b2e561c0f 100755 --- a/travis/self_tests +++ b/travis/self_tests @@ -37,7 +37,7 @@ EXPECTED_ERRORS = { 'duplicate-key': 2, 'pointless-statement': 1, 'pointless-string-statement': 1, - 'print-statement': 1, + 'print-used': 1, 'redundant-keyword-arg': 1, 'reimported': 4, 'return-in-init': 1, @@ -60,7 +60,6 @@ if PY3K: EXPECTED_ERRORS.update({ 'assignment-from-none': 2, }) - EXPECTED_ERRORS.pop('print-statement') @contextlib.contextmanager def _patch_streams(out):