Cleanup: Remove test storekit configuration files when importing through SPM

RevenueCat / purchases-ios

In-app purchases and subscriptions made easy. Support for iOS, watchOS, tvOS, macOS, and visionOS.

https://www.revenuecat.com/

MIT License

2.26k stars 303 forks source link

Cleanup: Remove test storekit configuration files when importing through SPM #3878

Closed aboedo closed 3 months ago

aboedo commented 3 months ago

Alternative approach to #3876, where in this one we only add the Package.swift file. That was the most important part in any case.

I would still love to clean up the project structure, but it's a much bigger change.

RevenueCat-Danger-Bot commented 3 months ago

	1 Message
:book:	Size increase: 0.40 KB

Generated by :no_entry_sign: Danger

aboedo commented 3 months ago

can confirm that it still works:

Now we just need to see whether tests still run correctly (they work on my machine ™️ )

aboedo commented 3 months ago

This has been bugging me for so long it's kind of crazy that the solution was this simple

vegaro commented 2 months ago

I've noticed that after this change, the Tests folder stopped appearing in xcode when opening the SPM project, which is the only way I know to work on RevenueCatUI tests. The xcodeproj doesn't have the RevenueCatUI test files (nor the source files)