Codeception / Codeception/module-db

Fix a confusion between 'cleanup' and 'skip_cleanup_if_failed' configuration params

Ouverte
#67 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.