Skip to content
This repository was archived by the owner on Apr 28, 2023. It is now read-only.

Uninvoked empty snapshot fix #98

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mariya-gfx
Copy link

Using 'readFileMaybe' plugin for storing snapshots for each test run
next to the initial test caller, there is no need of creating an empty
file if no test invokes a snapshot.

@CLAassistant
Copy link

CLAassistant commented Oct 23, 2019

CLA assistant check
All committers have signed the CLA.

Updated evaluateLoadedSnapShots() in index.js to store newly generated
snapshots. Also, global.before() checks on the snapshots content initially to
prevent the creation of an empty snapshot file.
@ghost
Copy link

ghost commented Feb 28, 2020

This is needed in Cypress 4.0 since cy.writeFile() will yield null

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants