Shopify / android-testify

Add screenshots to your Android tests
https://testify.dev
MIT License
231 stars 23 forks source link

Fix Issue #216: You can now define both an exclusion area and an exactness threshold #233

Closed DanielJette closed 3 years ago

DanielJette commented 3 years ago

What does this change accomplish?

Resolves #216

The bug as originally logged speculated that the problem was the PixelCopyCapture and the ExclusionRects being incompatible. The issue was actually with the setExactness method. Since PixelCopyCapture is hardware accelerated, it's susceptible to minor variations in rendering due to differences in GPU capabilities. As such, it's recommended to lower the matching tolerance using setExactness when using PixelCopyCapture. As originally designed, the exactness and exclusion rect comparison methods were separate paths and you could not combine them together.

This PR unifies the two concepts and allows you to mix & match the comparison methods.

Library

Bug fixes

Added

Changes

image