| # Copyright (c) 2016-2018, 2020 Claudiu Popa <[email protected]> |
| # Copyright (c) 2016-2017 Derek Gustafson <[email protected]> |
| # Copyright (c) 2019-2020 Pierre Sassoulas <[email protected]> |
| # Copyright (c) 2019 Ashley Whetter <[email protected]> |
| # Copyright (c) 2020 Damien Baty <[email protected]> |
| |
| # Licensed under the GPL: https://www.gnu.org/licenses/old-licenses/gpl-2.0.html |
| # For details: https://github.com/PyCQA/pylint/blob/master/COPYING |
| |
| """Tests for the pylint checker in :mod:`pylint.extensions.bad_builtin |
| """ |
| from os import path as osp |
| |
| import pytest |
| |
| from pylint.extensions.bad_builtin import BadBuiltinChecker |
| from pylint.lint import fix_import_path |
| |
| EXPECTED = [ |
| "Used builtin function 'map'. Using a list comprehension can be clearer.", |
| "Used builtin function 'filter'. Using a list comprehension can be clearer.", |
| ] |
| |
| |
| @pytest.fixture(scope="module") |
| def checker(): |
| return BadBuiltinChecker |
| |
| |
| @pytest.fixture(scope="module") |
| def disable(): |
| return ["I"] |
| |
| |
| def test_types_redefined(linter): |
| elif_test = osp.join(osp.dirname(osp.abspath(__file__)), "data", "bad_builtin.py") |
| with fix_import_path([elif_test]): |
| linter.check([elif_test]) |
| msgs = sorted(linter.reporter.messages, key=lambda item: item.line) |
| assert len(msgs) == 2 |
| for msg, expected in zip(msgs, EXPECTED): |
| assert msg.symbol == "bad-builtin" |
| assert msg.msg == expected |
