mirror of
https://github.com/BerriAI/litellm.git
synced 2026-09-11 22:51:28 +00:00
test: accept the parallel setup sub-make in the lint slot-lock assertion
Some checks failed
Terraform Modules / fmt, validate, test (aws) (push) Has been cancelled
Some checks failed
Terraform Modules / fmt, validate, test (aws) (push) Has been cancelled
The slot-lock test pinned lint-install and lint-fetch-base as declared prerequisites of lint-inner. This branch runs them from lint-inner's recipe instead, under their own -j 2 sub-make, so the pair overlaps while lint-checks keeps its own width. Assert the setup is reached from lint-inner either way, which is what the test is actually for: nothing runs before the slot is held.
This commit is contained in:
parent
0589729e34
commit
7671b79589
1 changed files with 4 additions and 3 deletions
|
|
@ -328,6 +328,7 @@ def test_direct_make_lint_takes_a_slot_before_any_setup() -> None:
|
|||
lint_prerequisites, lint_recipe = _make_rule("lint")
|
||||
assert lint_prerequisites == []
|
||||
assert any("$(GATE_SLOT_LOCK)" in line for line in lint_recipe)
|
||||
inner_prerequisites, _ = _make_rule("lint-inner")
|
||||
assert "lint-install" in inner_prerequisites
|
||||
assert "lint-fetch-base" in inner_prerequisites
|
||||
inner_prerequisites, inner_recipe = _make_rule("lint-inner")
|
||||
inner_work = (*inner_prerequisites, *inner_recipe)
|
||||
assert any("lint-install" in item for item in inner_work)
|
||||
assert any("lint-fetch-base" in item for item in inner_work)
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue