inserting null on an autoincrement primary key prevents cleanup

Offen

#58 erstellt am 14.6.2023 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 38/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: php
Bereich: databases, testing-qa

Rechercherichtung

Beginne damit, den Fall haveInDatabase(table, ['id' => null, ...]) mit einem autoincrement-Primärschlüssel nachzubilden, und verfolge dann, wie die eingefügte ID für das teardown cleanup aufgezeichnet wird. Als erledigt gilt die Aufgabe, wenn der generierte Primärschlüsselwert erhalten bleibt und die eingefügte Zeile während des Cleanups entfernt wird, ohne Fälle zu beeinträchtigen, in denen die autoincrement-Spalte nicht der Primärschlüssel ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

When I have a table with a primary key column that's also increment, it's valid to set it to null on the insert statement. the last insert it will then return the auto increment value, which is the primary key in that case.

As already noticed in #44, the auto increment column doesn't have to be part of the primary key, so the return value of last insert id ironically doesn't necessarily belong to the primary key. However, #44 also broke the case, which one might perceive as more common, that the last insert id may also be the primary key.

So, haveInDatabase(table, ['id' => null, ...]) with id as autoincrement and pk will lead to the id being set to null for cleanup, which results in the row not being removed at teardown.

I have prepared a fix for this case, which hopefully is specific enough to not interfere with other use cases.

Vorherrschende Sprache: PHP
Sterne: 23
Forks: 31
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus Codeception/module-db

haveInDatabase() is not usable inside a transaction on PostgreSQL Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 42/100

Codeception/module-db#87 · Erstellt am 09.6.2025
Using DB module in testing with transactions Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100

Codeception/module-db#85 · Erstellt am 25.2.2025
Make optional the auto-erase of the records, added by `haveInDatabase()` Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

Codeception/module-db#68 · 1 Kommentar · Erstellt am 10.5.2024
Fix a confusion between 'cleanup' and 'skip_cleanup_if_failed' configuration params Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

Codeception/module-db#67 · Erstellt am 09.5.2024
[DB error] PostgreSQL, for tables without sequences get "Undefined table: ERROR: relation '%_seq' does not exist" Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

Codeception/module-db#54 · Erstellt am 31.3.2023

Alle Issues in Codeception/module-db