You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently we have configuration params cleanup and skip_cleanup_if_failed.
Despite their names sound similar, they are responsible for completely different things: cleanup controls the database re-creation before each test, and skip_cleanup_if_failed controls whether records, added by haveInDatabase() should be erased after the test.
Suggested fix
It would be good to distinguish these two 'cleanups' in documentation and in the names of config params in next major versions.
For example, to rename cleanup to repopulate (since we have populate responsible for initial database creation).
The text was updated successfully, but these errors were encountered:
Problem
Currently we have configuration params
cleanup
andskip_cleanup_if_failed
.Despite their names sound similar, they are responsible for completely different things:
cleanup
controls the database re-creation before each test, andskip_cleanup_if_failed
controls whether records, added byhaveInDatabase()
should be erased after the test.Suggested fix
It would be good to distinguish these two 'cleanups' in documentation and in the names of config params in next major versions.
For example, to rename
cleanup
torepopulate
(since we havepopulate
responsible for initial database creation).The text was updated successfully, but these errors were encountered: