Skip to content

Conversation

@Badatos
Copy link
Collaborator

@Badatos Badatos commented Nov 7, 2025

…to avoid annoying SQLite timeouts

@Badatos Badatos added this to the 4.x milestone Nov 7, 2025
@Badatos Badatos self-assigned this Nov 7, 2025
@LoicBonavent
Copy link
Collaborator

Ah super, j'attendais que quelqu'un se dévoue pour trouver une solution à ces lock tables :) Merci

@Badatos
Copy link
Collaborator Author

Badatos commented Nov 7, 2025

Cette fois ca passe :

Ran 671 tests in 111.445s

(en SQLite on avait du Ran 671 tests in 124.144s donc les temps sont similaires )

@Badatos Badatos marked this pull request as ready for review November 7, 2025 15:48
@Badatos Badatos requested a review from LoicBonavent November 7, 2025 15:48
Copy link
Collaborator

@LoicBonavent LoicBonavent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, cela a été rapide en plus. Parfait, merci !

@Badatos
Copy link
Collaborator Author

Badatos commented Nov 7, 2025

Cool, cela a été rapide en plus. Parfait, merci !

Arf, ça ne passe pas :/

django.db.utils.DataError: (1406, "Data too long for column 'name' at row 1")

@Badatos Badatos marked this pull request as draft November 7, 2025 16:37
@LoicBonavent
Copy link
Collaborator

Il y a vraiment un paquet d'erreurs... Pas si rapide que çà sur le coup :)
Bon week-end

Base automatically changed from dev_v4 to main November 10, 2025 11:03
@Badatos Badatos changed the base branch from main to dev_v4 November 10, 2025 15:09
@Badatos
Copy link
Collaborator Author

Badatos commented Nov 10, 2025

OOOK, alors en effet ca va être un peu plus corsé que prévu.

Si avec SQLite le prochain id est celui qui est disponible, peu importe s'il a déjà été utilisé auparavant, c'est plus stricte avec Mysql, qui autoincremente les id, et un id ne sera jamais utilisé 2X.

Bref, il ne faut donc pas juste chercher l'élément d'id "1" dans les tests juste parce qu'on pense n'avoir créé qu'un élément.
CF correction effectuée à ce sujet dans le pod/recorder test_views.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants