Codeception / Codeception/module-doctrine

Recommend to load fixtures in `function _before()` and set `cleanup: false`?

Ouverte
#35 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
PHP
Étoiles
4
Forks
2
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

I'm observing that with `cleanup: true`, running external commands with `$I->runShellCommand('php bin/console ...');` isn't working. I'm guessing this is due to the transaction, which prevents external processes to "see" the database contents created/modified by the test. Can you confirm this?

If so, I would suggest to change the recommendation at https://codeception.com/docs/modules/Doctrine to `cleanup: false`, and show people how to load their fixtures in `function _before()`:
```php
$I->runShellCommand('php bin/console doctrine:fixtures:load --no-interaction --env=test');
```

IMO the currently recommended way to cleanup after each test doesn't work out, since for *some tests* it's necessary to disable the transaction (see above) - which ultimately leads to a mess.
So I'm figuring the best approach would be to just reload the fixtures before each test.

What do you think? How do you handle that?

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Start by reading the Doctrine module documentation at https://codeception.com/docs/modules/Doctrine and the issue's examples using cleanup, _before(), runShellCommand(), and doctrine:fixtures:load. Reproduce whether cleanup transactions hide test changes from external commands, then determine whether the documentation should recommend cleanup: false and fixture loading before each test.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
php
Domaine
documentation
Type d'issue
Documentation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
30/100

Recevez les nouvelles issues par e-mail

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