happo / happo-cypress

A Happo integration with Cypress.io
35 stars 4 forks source link

Associate snap-requests with report sooner #65

Closed trotzig closed 3 years ago

trotzig commented 3 years ago

When there is a nonce, meaning we're in parallel run mode, there is a risk the last step of the happo-cypress wrapper is lost. Might be from a failing test or some internal hickup. In these cases, it would make sense to associate the snap-requests with the report sooner.

This can help fix issues for certain accounts seeing some reports with missing snapshots. I've investigated and found that in some cases there are orphaned snap-requests right around the time for when a report was constructed (that had missing snapshots).