From ca86a8ddc0c685900755bddf7c2e83bf70f93c07 Mon Sep 17 00:00:00 2001 From: igor-ctrl Date: Wed, 12 Aug 2026 08:25:30 -0500 Subject: [PATCH] fix: keep bundle roundtrip test output in tmp_path test_make_bundle_produces_valid_manifest was the one test in this file that didn't pass output_path to make_bundle(), so it fell through to make_bundle's default (Path.cwd() / f"{profile}-{version}.tar.gz" in src/bcli/bundle/_publish.py:68) and wrote finance-2026.05.07-1.tar.gz into whatever directory pytest was invoked from. Every sibling test in this file already scopes its output under tmp_path; this brings the outlier in line with them. --- tests/test_bundle/test_bundle_roundtrip.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/test_bundle/test_bundle_roundtrip.py b/tests/test_bundle/test_bundle_roundtrip.py index 77c5f9b..a682b72 100644 --- a/tests/test_bundle/test_bundle_roundtrip.py +++ b/tests/test_bundle/test_bundle_roundtrip.py @@ -55,6 +55,7 @@ def test_make_bundle_produces_valid_manifest(tmp_path): version="2026.05.07-1", publisher="ops-bcli-bot", release_notes="initial", + output_path=tmp_path / "finance-2026.05.07-1.tar.gz", ) assert out.is_file() assert manifest.profile == "finance"