Compare commits
No commits in common. "376495e88f1746893c36c5f7d42c2ef82cb9f5a1" and "c3314b91259bf1fa9f613ab87b1363974c11e75a" have entirely different histories.
376495e88f
...
c3314b9125
1 changed files with 3 additions and 15 deletions
|
|
@ -416,26 +416,14 @@ def test_watchdog_source_has_no_sqlite3_connect():
|
||||||
|
|
||||||
|
|
||||||
def test_watchdog_source_has_no_explicit_process_lookup_error():
|
def test_watchdog_source_has_no_explicit_process_lookup_error():
|
||||||
"""Регрессия: watchdog не должен перехватывать ProcessLookupError явно (decision #357).
|
"""Регрессия: watchdog не должен явно упоминать ProcessLookupError (decision #357).
|
||||||
|
|
||||||
Обработка мёртвых процессов унифицирована через OSError + errno.ESRCH.
|
Обработка мёртвых процессов унифицирована через OSError + errno.ESRCH.
|
||||||
Проверяем через AST: в except-обработчиках не должно быть ProcessLookupError.
|
|
||||||
"""
|
"""
|
||||||
import ast
|
|
||||||
import core.watchdog as wd
|
import core.watchdog as wd
|
||||||
source = inspect.getsource(wd)
|
source = inspect.getsource(wd)
|
||||||
tree = ast.parse(source)
|
assert "ProcessLookupError" not in source, (
|
||||||
|
"REGRESSION: core/watchdog.py явно использует ProcessLookupError — "
|
||||||
process_lookup_in_except = False
|
|
||||||
for node in ast.walk(tree):
|
|
||||||
if isinstance(node, ast.ExceptHandler) and node.type is not None:
|
|
||||||
for name_node in ast.walk(node.type):
|
|
||||||
if isinstance(name_node, ast.Name) and name_node.id == "ProcessLookupError":
|
|
||||||
process_lookup_in_except = True
|
|
||||||
break
|
|
||||||
|
|
||||||
assert not process_lookup_in_except, (
|
|
||||||
"REGRESSION: core/watchdog.py перехватывает ProcessLookupError явно — "
|
|
||||||
"должен использовать OSError + errno.ESRCH"
|
"должен использовать OSError + errno.ESRCH"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue