Codeception / Codeception/module-db
Fix a confusion between 'cleanup' and 'skip_cleanup_if_failed' configuration params
- Langage dominant
- PHP
- Étoiles
- 23
- Forks
- 31
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
#### Problem
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).
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
L’issue mentionne les paramètres de configuration `cleanup` et `skip_cleanup_if_failed`, mais n’identifie ni fichiers ni tests. Commencez par localiser leurs définitions et leur documentation, puis déterminez comment les deux comportements sont décrits et où un futur changement de nom affecterait les utilisateurs. Le travail est terminé lorsque les deux responsabilités de cleanup sont clairement distinguées et que le changement de nom proposé est tranché pour une version majeure.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- php
- Domaine
- database, documentation
- Type d'issue
- Documentation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100