cashapp / paparazzi

Render your Android screens without a physical device or emulator
https://cashapp.github.io/paparazzi/
Apache License 2.0
2.28k stars 214 forks source link

[Proposal] Fail tests when screenshots do not match and there needs to be different task to generate html report workflow #1365

Closed kozaxinan closed 6 months ago

kozaxinan commented 6 months ago

Current task setup

graph LR;
testDebugUnitTest--No paparazzi check--->Generate-HTML-Report;
verifyPaparazzi--Just Paparazzi--->Verification;

Suggestion

graph LR;
testDebugUnitTest--All tests including Paparazzi --->Verification;
reportPaparazzi--Generate Html Report--->Report;
verifyPaparazzi-->Deprecated;

While working on the cache issue reported in https://github.com/cashapp/paparazzi/issues/1334, we decided to change the test setup for report and verification for our setup. How likely could the following change be applied in the paparazzi library? I know it can affect backward compatibility and expectations of people already using the paparazzi.

jrodbx commented 6 months ago

Duplicate of https://github.com/cashapp/paparazzi/pull/421.