pre-commit/flake8
1
0
Fork 0
mirror of https://github.com/PyCQA/flake8.git synced 2026-03-29 10:36:53 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

always use UTF-8 encoding when reading configuration

Browse source
This commit is contained in:
Anthony Sottile 2022-01-20 13:56:34 -05:00
parent ed690dcdd2
commit 9d23faad6d
2 changed files with 28 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/flake8/options/config.py
View file

@ -20,7 +20,7 @@ def _find_config_file(path: str) -> Optional[str]:
for candidate in ("setup.cfg", "tox.ini", ".flake8"):
cfg_path = os.path.join(path, candidate)
try:
cfg.read(cfg_path)
cfg.read(cfg_path, encoding="UTF-8")
except (UnicodeDecodeError, configparser.ParsingError) as e:
LOG.warning("ignoring unparseable config %s: %s", cfg_path, e)
else:
@ -61,7 +61,7 @@ def load_config(
cfg = configparser.RawConfigParser()
if config is not None:
if not cfg.read(config):
if not cfg.read(config, encoding="UTF-8"):
raise exceptions.ExecutionError(
f"The specified config file does not exist: {config}"
)
@ -72,7 +72,7 @@ def load_config(
# TODO: remove this and replace it with configuration modifying plugins
# read the additional configs afterwards
for filename in extra:
cfg.read(filename)
cfg.read(filename, encoding="UTF-8")
return cfg, cfg_dir

25
tests/unit/test_options_config.py
View file

@ -118,6 +118,31 @@ def test_load_config_append_config(tmpdir):
assert cfg_dir == str(tmpdir)
NON_ASCII_CONFIG = "# ☃\n[flake8]\nindent-size=8\n"
def test_load_auto_config_utf8(tmpdir):
tmpdir.join("setup.cfg").write_text(NON_ASCII_CONFIG, encoding="UTF-8")
with tmpdir.as_cwd():
cfg, cfg_dir = config.load_config(None, [], isolated=False)
assert cfg["flake8"]["indent-size"] == "8"
def test_load_explicit_config_utf8(tmpdir):
tmpdir.join("t.cfg").write_text(NON_ASCII_CONFIG, encoding="UTF-8")
with tmpdir.as_cwd():
cfg, cfg_dir = config.load_config("t.cfg", [], isolated=False)
assert cfg["flake8"]["indent-size"] == "8"
def test_load_extra_config_utf8(tmpdir):
tmpdir.join("setup.cfg").write("[flake8]\nindent-size=2\n")
tmpdir.join("t.cfg").write_text(NON_ASCII_CONFIG, encoding="UTF-8")
with tmpdir.as_cwd():
cfg, cfg_dir = config.load_config(None, ["t.cfg"], isolated=False)
assert cfg["flake8"]["indent-size"] == "8"
@pytest.fixture
def opt_manager():
ret = OptionManager(version="123", plugin_versions="", parents=[])