Skip to content

Remove redundant test_standalone and tool_standalone crates - #4101

Merged
Kenny Kerr (kennykerr) merged 3 commits into
masterfrom
copilot/remove-redundant-tests
Mar 28, 2026
Merged

Remove redundant test_standalone and tool_standalone crates#4101
Kenny Kerr (kennykerr) merged 3 commits into
masterfrom
copilot/remove-redundant-tests

Conversation

Copilot AI commented Mar 28, 2026

Copy link
Copy Markdown
Contributor

test_standalone and tool_standalone are fully superseded by test_bindgen and the broader test suite. Removes them to reduce CI time.

Why they're redundant:

  • Every bindgen output mode tested by test_standalone (--sys, --flat, --no-deps, --vtbl, --derive) is already covered by test_bindgen modules (bool_sys, struct_sys/win, interface_cpp_sys, derive_struct, struct_arch_a/w, etc.)
  • All runtime Windows API calls (CoCreateGuid, SysAllocString, Uri::CreateUri, Calendar::new(), etc.) are covered by test_core, test_strings, and the winrt/ crates
  • tool_standalone/src/main.rs already carried // TODO: move these into the new bindgen test/tool

Removed:

  • crates/tests/misc/standalone/ — 33 pre-generated b_*.rs binding files + runtime tests (~4,100 lines)
  • crates/tools/standalone/ — the regeneration tool that produced those files (~260 lines)
  • Auto-regenerated test.yml and clippy.yml via cargo run -p tool_yml
  • Removed standalone from the tool matrix in .github/workflows/gen.yml

CI savings: 4 steps removed per platform × 5 platforms = 20 fewer invocations per run (test + clippy, for both test_standalone and tool_standalone), plus the gen.yml matrix is reduced by one entry.


⌨️ Start Copilot coding agent tasks without leaving your editor — available in VS Code, Visual Studio, JetBrains IDEs and Eclipse.

@kennykerr

Copy link
Copy Markdown
Collaborator

Copilot you need to update the gen.yml workflow to remove the test.

Copilot AI commented Mar 28, 2026

Copy link
Copy Markdown
Contributor Author

Copilot you need to update the gen.yml workflow to remove the test.

Done — removed standalone from the tool matrix in .github/workflows/gen.yml in ca87172.

@kennykerr
Kenny Kerr (kennykerr) merged commit 805c374 into master Mar 28, 2026
36 checks passed
@kennykerr
Kenny Kerr (kennykerr) deleted the copilot/remove-redundant-tests branch March 28, 2026 11:28
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.

2 participants