Skip to content

Commit 74bade0

Browse files
Sourabhchrs93claude
andcommitted
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 <noreply@anthropic.com>
1 parent bec9e6c commit 74bade0

1 file changed

Lines changed: 4 additions & 14 deletions

File tree

tests/test_vendor/test_run_results_pre_v6.py

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,7 @@
1515

1616

1717
def _module(version: int):
18-
return importlib.import_module(
19-
f"vendor.dbt_artifacts_parser.parsers.run_results.run_results_v{version}"
20-
)
18+
return importlib.import_module(f"vendor.dbt_artifacts_parser.parsers.run_results.run_results_v{version}")
2119

2220

2321
def _result(status: str, unique_id: str) -> dict:
@@ -57,10 +55,7 @@ def test_reused_status_parses(self, version):
5755
def test_known_statuses_preserve_value(self, version):
5856
mod = _module(version)
5957
for status in ("success", "error", "skipped"):
60-
assert (
61-
mod.RunResultOutput(**_result(status, "model.proj.a")).status.value
62-
== status
63-
)
58+
assert mod.RunResultOutput(**_result(status, "model.proj.a")).status.value == status
6459

6560
def test_unknown_future_status_parses(self, version):
6661
mod = _module(version)
@@ -71,20 +66,15 @@ def test_test_and_freshness_statuses_keep_their_value(self, version):
7166
"""Test/freshness statuses must keep their `.value` (resolved via Status1/Status2)."""
7267
mod = _module(version)
7368
for status in ("pass", "fail", "warn", "runtime error"):
74-
assert (
75-
mod.RunResultOutput(**_result(status, "test.proj.t")).status.value
76-
== status
77-
)
69+
assert mod.RunResultOutput(**_result(status, "test.proj.t")).status.value == status
7870

7971

8072
@pytest.mark.parametrize("version", VERSIONS)
8173
class TestParseRunResultsEntryPointPreV6:
8274
"""The public `parse_run_results` must parse a full file containing `reused`."""
8375

8476
def test_file_with_reused_result_parses_fully(self, version):
85-
run_results = parse_run_results(
86-
_run_results(version, "success", "reused", "skipped", "pass", "error", "no-op")
87-
)
77+
run_results = parse_run_results(_run_results(version, "success", "reused", "skipped", "pass", "error", "no-op"))
8878
# Previously this whole file was dropped because of the single `reused` row.
8979
assert len(run_results.results) == 6
9080
assert {r.status.value for r in run_results.results} == {

0 commit comments

Comments
 (0)