diff --git a/tests/test_shaker.py b/tests/test_shaker.py index c867809..75ed3f4 100644 --- a/tests/test_shaker.py +++ b/tests/test_shaker.py @@ -9,15 +9,15 @@ def parse(tmp_path: Path, content: str) -> list[Any]: - f = tmp_path / "test.scad" - f.write_text(content) - return getASTfromFile(str(f), process_includes=False) or [] + test_file = tmp_path / "test.scad" + test_file.write_text(content) + return getASTfromFile(str(test_file), process_includes=False) or [] def make_pool(tmp_path: Path, content: str) -> dict[str, list]: """Parse content and return a list-valued pool dict. - This mirrors the private ``_add_to_pool`` behavior in ``openscad_packer.shaker``. + This mirrors the private ``_add_to_pool`` behavior in ``openscad_packer.packer``. """ nodes = parse(tmp_path, content) pool: dict[str, list] = {} @@ -98,8 +98,10 @@ def test_collects_chained_calls(self, tmp_path): def test_does_not_collect_non_identifier_primary_calls(self, tmp_path): # (function(x) x*2)(5) — PrimaryCall where left is a FunctionLiteral, not Identifier names = collect_called_names(parse(tmp_path, "y = (function(x) x*2)(5);")) - # No Identifier on the left side, so nothing collected from the call itself - assert "function" not in names + # The callee is a FunctionLiteral (not an Identifier), so the call site + # contributes nothing. The body x*2 has no inner calls either, so the + # full result must be empty. + assert names == set() class TestComputeReachable: