swift-snapshot-testing icon indicating copy to clipboard operation
swift-snapshot-testing copied to clipboard

Support for RTL layout testing with Xcode 11 test plans

Open SebastianOsinski opened this issue 6 years ago • 5 comments

This PR adds support for SNAPSHOT_CONFIGURATION_NAME environment variable which can be used in Test Plans configurations. When aforementioned variable exists, another level of directory nesting is added to store separate snapshots for each configuration.

Additionally, to fully support RTL layout testing, .leftToRight traits have been removed from base trait sets for each snapshotting device. This trait override caused RTL UIs to not be correctly rendered (see http://www.openradar.me/radar?id=5044259694575616).

See #168 for more background for this PR.

SebastianOsinski avatar Oct 19 '19 19:10 SebastianOsinski

@mbrandonw @stephencelis Hi! Over a month passed since I opened this PR. Is there any chance to merge it or give any pointers / suggestions :)?

SebastianOsinski avatar Nov 22 '19 13:11 SebastianOsinski

Hey @SebastianOsinski! It'd definitely be nice to add an environment component to test names to distinguish things like Xcode 11 test plans! @mbrandonw and I will thing things over and bike-shed the variable name and where it renders in the filename soon.

stephencelis avatar Dec 10 '19 22:12 stephencelis

@stephencelis Hi! Did you have time to think it over? I'm always ready to update this PR according to your suggestions ;)

SebastianOsinski avatar Jan 20 '20 14:01 SebastianOsinski

Do we have any update on this PR? My team is working on a test project and we highly anticipate this feature. Thanks all for the help!

justinguo avatar Apr 03 '20 19:04 justinguo

bump on this PR

skywalkerdude avatar Nov 01 '20 00:11 skywalkerdude