Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove native iOS tests #62

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

okwasniewski
Copy link
Contributor

@okwasniewski okwasniewski commented Sep 27, 2024

Summary:

This PR removes native iOS tests as they are no longer used.

Unfortunately, this change creates a big diff in HelloWorld.xcodeproj/project.pbxproj We should annotate it in upgrade helper. But less code to maintain for users is always better 👍🏻

Manual Instruction for upgrade helper

  1. Open the xcworkspace
  2. Remove the target
  3. Open the Podfile
  4. Remove the section for the Tests
  5. rerun pod install.

Changelog:

[IOS] [REMOVED] - Native iOS tests

Test Plan:

  1. Create a new project
  2. Test if there is no sign of native tests in Xcode
  3. Build the app

@thymikee
Copy link
Member

Can you add manual instructions of what needs to be changed in Xcode, so that it stays in this PR and can be easily referenced / copied over to the Upgrade Helper?

@okwasniewski
Copy link
Contributor Author

okwasniewski commented Sep 27, 2024

@thymikee I've added additional instructions for manual steps in Xcode if users don't want to deal with editing the file manually

@cipolleschi
Copy link
Contributor

Instructions could be simpler: it should be enough to:

  1. Open the xcworkspace
  2. Remove the target
  3. Open the Podfile
  4. Remove the section for the Tests
  5. rerun pod install.

Step 5 should take care of all the other parts.

@okwasniewski
Copy link
Contributor Author

Thanks @cipolleschi, I upgraded the instructions

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.

3 participants