Skip to content

style(tests): format routing fixtures for follow-up repair - #799

Merged
ss-o merged 1 commit into
mainfrom
bug-796-fixture-cleanup
Oct 15, 2026
Merged

ss-o merged 1 commit into
mainfrom
bug-796-fixture-cleanup

Conversation

@ss-o

@ss-o ss-o commented Oct 15, 2026

Copy link
Copy Markdown
Member

Summary

Format the existing Org Routing test file with the pinned Black and isort tools so a subsequent fixture cleanup fix can pass the required formatter checks without mixing formatting and behavior changes.

Refs #796; follow-up to #797's fixture validation.

Verification

  • The import set and ordered non-import Python AST are unchanged.
  • 104 Org Routing, policy-report and knowledge-delivery tests passed.
  • The affected-file Trunk check and whitespace check passed.

Agent handoff

Mechanical prerequisite only. The reproduced detached-maintenance cleanup fix follows in a separate PR.

@ss-o
ss-o requested a review from a team as a code owner October 15, 2026 06:13

@ss-o ss-o left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fallback review under ADR-0026: maintainer elected, no Copilot request on 8cedb15

No actionable findings.

  • Correctness: the import set is unchanged, with only standard-library import ordering adjusted. The ordered non-import Python AST is identical to the base. No fixture, test assertion or production behavior changes.
  • Scope: only the existing Org Routing test file is formatted using its pinned Black and isort tools. The functional cleanup change is separate.
  • Verification: 104 related tests pass on Python 3.14 and Python 3.10.8. The affected-file Trunk check reports no new issues; two inherited Bandit findings remain. All 11 hosted checks pass on this head, and the source signature and whitespace checks pass.
  • Limits: this is self-review. Visual Markdown rendering was not inspected. The formatting prerequisite does not repair the cleanup race by itself.

@ss-o
ss-o merged commit f830dfd into main Oct 15, 2026
11 checks passed
@ss-o
ss-o deleted the bug-796-fixture-cleanup branch October 15, 2026 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant