Update --suite-name-format default value to preserve

[Copilot]

swiftTestingTestCaseNames supports suiteNameFormat: .preserve to prevent stripping display name strings from @Suite. This PR makes preserve the default for suiteNameFormat, and adds tests to document the interaction between the two rules—specifically that redundantSwiftTestingSuite only removes @Suite("Name") after swiftTestingTestCaseNames strips the string argument, not independently.

Changes

• Options.swift: Changed the default value of suiteNameFormat from .standardIdentifiers to .preserve. This means @Suite("Name") is preserved by default; users must opt into --suite-name-format standard-identifiers or raw-identifiers to have the display name stripped.

• Rules.md: Regenerated to reflect the updated default, now showing "preserve" (default) for --suite-name-format.

• RedundantSwiftTestingSuiteTests: Added testKeepSuiteWithDisplayNameWhenSuiteNameFormatIsPreserve — runs both rules together with suiteNameFormat: .preserve and verifies @Suite("Display Name") is left untouched (the string stays, so the attribute isn't redundant).

• SwiftTestingTestCaseNamesTests: Added testSuitePreserveKeepsDisplayNameWithBothRules — same scenario from the other rule's perspective; both rules active with suiteNameFormat: .preserve leave @Suite("My Feature Tests") unchanged.

• Test updates: Updated tests that previously relied on the .standardIdentifiers default to explicitly pass suiteNameFormat: .standardIdentifiers.

The existing testRemovesTestSuiteWithOnlyStringArgument (both rules, with explicit suiteNameFormat: .standardIdentifiers) covers the opt-in case: swiftTestingTestCaseNames strips the string → redundantSwiftTestingSuite removes the now-empty @Suite.

Original prompt

Ensure the swiftTestingTestCaseNames rule has a way to disable the behavior where a suite's string name is removed. Does the preserve option already work that way? Also include tests that redundantSwiftTestingSuite doesn't remove a suite like @Suite("Name") unless the other rule removes the string argument first.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[calda]

@copilot, let's also make preserve the default for this option please

[calda]

Fixes #2439

[Copilot]

@copilot, let's also make preserve the default for this option please

Done in dedb229. Changed suiteNameFormat default from .standardIdentifiers to .preserve in Options.swift, and updated all tests that relied on the old default to explicitly pass suiteNameFormat: .standardIdentifiers.

[calda]

@copilot please fix test failures

[Copilot]

@copilot please fix test failures

Fixed in e8dbbba. The MetadataTests.testGenerateRulesDocumentation() failure was because Rules.md (auto-generated) still showed standard-identifiers (default) for --suite-name-format. Running the test regenerated it to show preserve (default).

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.29%. Comparing base (f771e19) to head (e8dbbba).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #2441   +/-   ##
========================================
  Coverage    95.28%   95.29%           
========================================
  Files          165      165           
  Lines        25147    25147           
========================================
+ Hits         23962    23964    +2     
+ Misses        1185     1183    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.