From 74bade0bd66702245278e6aedcdc3dd5272f5e17 Mon Sep 17 00:00:00 2001 From: Sourabh Chourasia Date: Thu, 23 Jul 2026 00:36:47 +0530 Subject: [PATCH] chore: [AI-7675] format pre-v6 run_results tests to repo black config `tests/test_vendor/test_run_results_pre_v6.py` (PR #108) was not formatted to the repo's `black` line-length (140), which fails the `pre-commit` gate inside `release.sh`/`tox` and blocked the v0.3.4 PyPI publish in the tag-release workflow. Formatting-only change; tests unchanged semantically (36 pass). This PR carries the `version-bump` label so its merge re-triggers tag-release and publishes 0.3.4 (the previous unpublished v0.3.4 tag is deleted so the workflow can recreate it at this commit). Co-Authored-By: Claude Fable 5 --- tests/test_vendor/test_run_results_pre_v6.py | 18 ++++-------------- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/tests/test_vendor/test_run_results_pre_v6.py b/tests/test_vendor/test_run_results_pre_v6.py index a5d737d..f242f7e 100644 --- a/tests/test_vendor/test_run_results_pre_v6.py +++ b/tests/test_vendor/test_run_results_pre_v6.py @@ -15,9 +15,7 @@ def _module(version: int): - return importlib.import_module( - f"vendor.dbt_artifacts_parser.parsers.run_results.run_results_v{version}" - ) + return importlib.import_module(f"vendor.dbt_artifacts_parser.parsers.run_results.run_results_v{version}") def _result(status: str, unique_id: str) -> dict: @@ -57,10 +55,7 @@ def test_reused_status_parses(self, version): def test_known_statuses_preserve_value(self, version): mod = _module(version) for status in ("success", "error", "skipped"): - assert ( - mod.RunResultOutput(**_result(status, "model.proj.a")).status.value - == status - ) + assert mod.RunResultOutput(**_result(status, "model.proj.a")).status.value == status def test_unknown_future_status_parses(self, version): mod = _module(version) @@ -71,10 +66,7 @@ def test_test_and_freshness_statuses_keep_their_value(self, version): """Test/freshness statuses must keep their `.value` (resolved via Status1/Status2).""" mod = _module(version) for status in ("pass", "fail", "warn", "runtime error"): - assert ( - mod.RunResultOutput(**_result(status, "test.proj.t")).status.value - == status - ) + assert mod.RunResultOutput(**_result(status, "test.proj.t")).status.value == status @pytest.mark.parametrize("version", VERSIONS) @@ -82,9 +74,7 @@ class TestParseRunResultsEntryPointPreV6: """The public `parse_run_results` must parse a full file containing `reused`.""" def test_file_with_reused_result_parses_fully(self, version): - run_results = parse_run_results( - _run_results(version, "success", "reused", "skipped", "pass", "error", "no-op") - ) + run_results = parse_run_results(_run_results(version, "success", "reused", "skipped", "pass", "error", "no-op")) # Previously this whole file was dropped because of the single `reused` row. assert len(run_results.results) == 6 assert {r.status.value for r in run_results.results} == {