fix(tests): zip(strict=True) в гарде паритета — ruff B905
Some checks failed
CI / changes (pull_request) Successful in 9s
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
CI Trade-In / changes (pull_request) Successful in 8s
CI / backend-tests (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Failing after 4m38s
Some checks failed
CI / changes (pull_request) Successful in 9s
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
CI Trade-In / changes (pull_request) Successful in 8s
CI / backend-tests (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Failing after 4m38s
CI поймал то, что локально не проверилось: ruff в этом окружении вызывался не тем интерпретатором и молча не запускался. strict=True здесь не просто успокаивает линтер — равенство длин кортежей уже проверено assert'ом выше, так что строгий zip выражает ровно то же намерение явно. Refs #3063
This commit is contained in:
parent
d3d75801ca
commit
0cce97bd75
1 changed files with 2 additions and 1 deletions
|
|
@ -111,7 +111,8 @@ def test_unchanged_gate_mirrors_the_set_clause() -> None:
|
|||
f"кортежи гейта разной длины: слева {len(left)}, справа {len(right)} — "
|
||||
"сравнение поехало бы по колонкам молча"
|
||||
)
|
||||
for i, (lhs, rhs) in enumerate(zip(left, right)):
|
||||
# strict=True безопасен: равенство длин уже проверено assert выше.
|
||||
for i, (lhs, rhs) in enumerate(zip(left, right, strict=True)):
|
||||
column = lhs.split(".")[-1]
|
||||
if column == "is_active":
|
||||
continue # SET пишет литерал true, справа он же — намеренно
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue