Fix assertSnapshot for Swift Testing tests. #916
Conversation
| @_implementationOnly import Testing | ||
| import Testing | ||
|
|
||
| @_spi(Experimental) |
There was a problem hiding this comment.
Heads up @JRR-OSU and @kondratk. This will affect you based on our discussions in #901.
We have had to remove the @_implementationOnly imports for Testing because we want to remove the @_spi restriction from these testing traits. This means that once again Snapshot Testing will not work from UI testing targets. Again, we feel this is a bug for Apple to deal with, as Testing does not work in UI test targets.
The only other fix that comes to mind for this would be to move everything but the code that uses Swift Testing into some kind of "SnapshotTestingCore" module, and then that could be imported into UI test targets when needed. We will discuss this more.
|
Does this mean that attachments won't be available at all when running from Swift Testing? Seems like Swift Testing does not support it yet at all? |
|
@IlyaPuchkaTW That's correct. You can follow their progress here: swiftlang/swift-testing#714 |
Fixes #915