fix: translate_weather обрезает пробелы перед проверкой на пустоту
This commit is contained in:
parent
efee713301
commit
d94b9c16ab
@ -163,11 +163,11 @@ class TestTranslateWeather:
|
|||||||
[
|
[
|
||||||
("", "—"),
|
("", "—"),
|
||||||
(None, "—"),
|
(None, "—"),
|
||||||
(" ", " "), # пробелы не считаются пустыми
|
(" ", "—"), # строка из пробелов тоже считается пустой
|
||||||
],
|
],
|
||||||
)
|
)
|
||||||
def test_translate_empty(self, input_value, expected) -> None:
|
def test_translate_empty(self, input_value, expected) -> None:
|
||||||
"""Пустой или None ввод должен возвращать '—'."""
|
"""Пустой, None или только пробелы должен возвращать '—'."""
|
||||||
assert translate_weather(input_value) == expected
|
assert translate_weather(input_value) == expected
|
||||||
|
|
||||||
def test_translate_unknown_returns_original(self) -> None:
|
def test_translate_unknown_returns_original(self) -> None:
|
||||||
|
|||||||
@ -172,9 +172,9 @@ _WEATHER_MAPPING = [
|
|||||||
|
|
||||||
|
|
||||||
def translate_weather(en: Optional[str]) -> str:
|
def translate_weather(en: Optional[str]) -> str:
|
||||||
if not en:
|
if not en or not en.strip():
|
||||||
return "—"
|
return "—"
|
||||||
en_lower = en.lower()
|
en_lower = en.strip().lower()
|
||||||
for key, value in _WEATHER_MAPPING:
|
for key, value in _WEATHER_MAPPING:
|
||||||
if key.lower() in en_lower:
|
if key.lower() in en_lower:
|
||||||
return value
|
return value
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user