Skip to content

dx: move test_reports tests to sibling file (part of #10053) - #10127

Open
TrapsterDK wants to merge 4 commits into
mainfrom
split-exec-tests
Open

TrapsterDK wants to merge 4 commits into
mainfrom
split-exec-tests

Conversation

@TrapsterDK

@TrapsterDK TrapsterDK commented Nov 9, 2026 •

Copy link
Copy Markdown
Collaborator

Closes #10053.

Pure moves, no behavior change:

  • In-file tests of test_reports.rs, umbrella.rs, update_consumer.rs, update_version.rs, common.rs and audit.rs move to sibling *_tests.rs files.
  • In-file tests of plan/src/slots.rs, deploy/release/src/lib.rs, deploy/rules/src/oci.rs, update/src/backend.rs and adopt/src/dependency_sets.rs move to sibling *_tests.rs files.
  • test_reports.rs splits into test_reports/{account,collect,report}.rs.

Tested: bazel test //cli/cli:dx_cli_test, //cli/plan:dx_plan_test, //cli/adopt:dx_adopt_test, //deploy/rules:dx_deploy_tools_test and //deploy/release:dx_release_tools_test passed. //cli/update:dx_update_test fails only root_sets_tests::root_module_declares_exactly_the_expected_sets, a test this PR does not touch. Nested-Bazel and consumer tests not run.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Xut2hwESun1HL2eSVd1d5B

@TrapsterDK
TrapsterDK marked this pull request as ready for review November 9, 2026 16:25
@TrapsterDK TrapsterDK added the needs-review Touches review paths; needs a second-session Review: comment label Nov 9, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review Touches review paths; needs a second-session Review: comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dx: split exec modules over 2000 lines

1 participant