Codeception / Codeception/module-doctrine

Doctrine2 Module: Is `flush()` before checking the database so necessary?

Aperta
#21 14 commenti 1 reazione 0 assegnatari Vedi su GitHub
Lingua principale
PHP
Stelle
4
Fork
2
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Why do we flush managed Doctrine entities before checking the database in [`Doctrine2::proceedSeeInRepository()`](https://github.com/Codeception/Codeception/blob/bd6ad8af1fa7cd12684f0287374b9fe71a8c958a/src/Codeception/Module/Doctrine2.php#L378)?

In functional tests the Entity Manager of application is used in the Codeception Doctrine2 Module. If we do test a request with validation rejection, and we want to be sure the database was not changed, the test result will be false negative (the request fetches and populates some entity, but didn't save changes to the database, and then `dontSeeInRepository()` flushes the invalid entity to the database and does fail).

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia in src/Codeception/Module/Doctrine2.php, in Doctrine2::proceedSeeInRepository(), soprattutto nel flush prima del controllo del database. Esamina lo scenario di rifiuto della validazione dell'issue e i test esistenti del modulo, quindi verifica che il controllo dei contenuti del repository non renda persistenti le modifiche alle entità gestite recuperate o non valide; i test pertinenti dovrebbero superarsi senza falsi negativi.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
php
Ambito
databases, testing
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.