Enable tests for SwiftPM #382
Conversation
Some tests are not guarded for UIKit.
Instead of using `image(forResource:)`, use an own `testImage(forResource:)` which allows us to override the lookup behaviour when building w/ SwiftPM.
... which doesn't even have UXImage 🙄 ;-)
chrisballinger
changed the base branch from
feature/zeveisenberg/swiftpm
to
master
|
Thanks for the PR! This is an interesting and awesome workaround for the "SPM packages can't include resources" issue. SE-0271 will resolve this problem at the source, so it might be better to put this on pause until resource bundles are officially supported in SPM. |
|
Looks like this is resolved in Swift 5.3, would love to get this updated and merged once that's out: https://github.com/apple/swift-evolution/blob/master/proposals/0271-package-manager-resources.md |
|
This should now be possible to do without hacks in Xcode 12.2 using the new Swift 5.3 SPM toolchain for bundled resources. |
|
It looks like there's a bug related to bundled resources and test targets specifically. Not sure if this is going to be fixed in Xcode 12.5, but it doesn't seem to be in the betas. swiftlang/swift-package-manager#2905 |
A way to enable tests even when building w/ SwiftPM.
Only two tests fail (on macOS):