facebookarchive / ios-snapshot-test-case

Snapshot view unit tests for iOS

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Tests are failing since 1.2 with drawViewHierarchyInRect:afterScreenUpdates

tapwork opened this issue · comments

Since release 1.2 and the changes with drawViewHierarchyInRect:afterScreenUpdates most of our tests are failing. Release 1.2 is not usable for us.
Why is drawViewHierarchyInRect:afterScreenUpdates necessary? Performance?

The drawViewHierarchyInRect:afterScreenUpdates renders the view more like in real. But that means that they are not compatible to the old rendering.

I made a pull request (#46), which makes it possible to select between these two ways of rendering.

Have just noticed this: definitely like that the new version can render content that previously didn't work (especially fonts in many cases), but I'm having the problem that even after updating my reference images I still don't get matches. Will check out the pull request. I think this may need to be optional with some kind of "high accuracy" mode...

I came to the conclusion to adapt my tests to work with 1.2.
So I am fine with 1.2