inserting null on an autoincrement primary key prevents cleanup
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 38/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- php
- Área
- databases, testing-qa
Línea de trabajo
Comienza reproduciendo el caso haveInDatabase(table, ['id' => null, ...]) con una clave primaria autoincremental y, después, sigue cómo se registra el ID insertado para el teardown cleanup. Se considera terminado cuando se conserva el valor de la clave primaria generada y la fila insertada se elimina durante el cleanup, sin romper los casos en los que la columna autoincremental no es la clave primaria.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- PHP
- Estrellas
- 23
- Forks
- 31
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Codeception/module-db
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
Codeception/module-db#87 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Codeception/module-db#85 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Codeception/module-db#68 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Codeception/module-db#67 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Codeception/module-db#54 ·
Todos los issues de Codeception/module-db
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
-
Admissions heading updates Abiertoa11y admissions.uiowa.edu needs grooming SiteImprove best practice
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Save States Menu Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Error on exporting the error log Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
pluginsGLPI/datainjection#673 ·