fix(formatter): allow parameters to break in angular test wrappers

[Boshen]

Summary

• Fixed function parameters inside angular test wrappers (inject, async, fakeAsync, waitForAsync) not breaking across lines when they exceed print width
• Made is_angular_test_wrapper public and added a check to exclude angular wrappers from the "hug" parameter layout
• JS conformance improved from 96.85% to 96.98%

Test plan

• cargo run -p oxc_prettier_conformance -- --filter angularjs_inject passes
• cargo run -p oxc_prettier_conformance -- --filter angular passes (all angular test files)
• cargo test -p oxc_formatter passes

🤖 Generated with Claude Code

[Dunqing]

Nice!

[Dunqing]

Merge activity

• Jan 16, 6:51 AM UTC: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Jan 16, 7:00 AM UTC: This pull request can not be added to the Graphite merge queue. Please try rebasing and resubmitting to merge when ready.
• Jan 16, 7:00 AM UTC: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..
• Jan 16, 7:00 AM UTC: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Jan 16, 7:41 AM UTC: Dunqing added this pull request to the Graphite merge queue.
• Jan 16, 7:47 AM UTC: Merged by the Graphite merge queue.

[Copilot]

Pull request overview

This PR fixes a formatting issue where function parameters inside Angular test wrappers (inject, async, fakeAsync, waitForAsync) were not breaking across lines when they exceeded the print width. The fix involves making the is_angular_test_wrapper function public and adding a check to exclude Angular wrappers from the "hug" parameter layout, allowing their inner function parameters to break normally.

Changes:

• Made is_angular_test_wrapper function public in call_expression.rs
• Added logic to exclude Angular test wrappers from parameter hugging in parameters.rs
• Improved JS/TS conformance test results (removed 2 failing TS test files and 1 failing JS test file)

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
crates/oxc_formatter/src/utils/call_expression.rs	Made is_angular_test_wrapper function public for use in parameter formatting
crates/oxc_formatter/src/print/parameters.rs	Added import and logic to exclude Angular test wrappers from parameter hugging, allowing their parameters to break
tasks/prettier_conformance/snapshots/prettier.js.snap.md	Updated JS conformance stats showing angularjs_inject.js test now passes
tasks/prettier_conformance/snapshots/prettier.ts.snap.md	Updated TS conformance stats removing two previously failing union comment tests

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codspeed-hq]

CodSpeed Performance Report

Merging this PR will not alter performance

_{Comparing fix/formatter-angular-test-wrapper-params (1eaa955) with main (f69b6da)}

Summary

✅ 38 untouched benchmarks
⏩ 7 skipped benchmarks¹

Footnotes

1. 7 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩