Skip to content

Remove unused test plans#6759

Merged
facumenzella merged 1 commit into
mainfrom
facu/remove-unused-test-plans
May 11, 2026
Merged

Remove unused test plans#6759
facumenzella merged 1 commit into
mainfrom
facu/remove-unused-test-plans

Conversation

@facumenzella

@facumenzella facumenzella commented May 10, 2026

Copy link
Copy Markdown
Member

Summary

  • Removes 5 test plan files that are not referenced by any Fastlane lane, CI script, or other automation:
    • AllTests.xctestplan
    • XC-AllTests.xctestplan — leftover from an abandoned Xcode Cloud migration (sets XCODE_CLOUD=1, otherwise identical to CI-AllTests)
    • UnitTests.xctestplan
    • Coverage.xctestplan
    • RevenueCat.xctestplan
  • The active test plans (CI-AllTests, CI-RevenueCat, CI-Snapshots, CI-RevenueCat-Snapshots, Paywall-Screenshots) are untouched.

After this cleanup, I am planning to remove some test targets from the legacy project (leaving them only in the Tuist one)

Test plan

  • Verify CI passes (no lane references the removed files)

🤖 Generated with Claude Code


Note

Low Risk
Low risk cleanup that only deletes unused .xctestplan files; behavior changes only if some external/local workflow still referenced these plans.

Overview
Removes five unused Xcode test plan files under Tests/TestPlans (including legacy XC-AllTests.xctestplan).

No test targets or active CI test plans are modified; this is a repository cleanup to reduce stale configuration.

Reviewed by Cursor Bugbot for commit 88cc028. Bugbot is set up for automated code reviews on this repo. Configure here.

AllTests, XC-AllTests, UnitTests, Coverage, and RevenueCat.xctestplan
are not referenced by any Fastlane lane, CI script, or other automation.
XC-AllTests appears to be a leftover from an abandoned Xcode Cloud
migration (it sets XCODE_CLOUD=1 but is otherwise identical to CI-AllTests).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@facumenzella facumenzella marked this pull request as ready for review May 11, 2026 04:49
@facumenzella facumenzella requested a review from a team as a code owner May 11, 2026 04:49
@facumenzella facumenzella requested a review from ajpallares May 11, 2026 04:49

@ajpallares ajpallares left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the cleanup! 💪

@facumenzella facumenzella merged commit 171b555 into main May 11, 2026
41 of 42 checks passed
@facumenzella facumenzella deleted the facu/remove-unused-test-plans branch May 11, 2026 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants