Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test de la création de déclaration historique #1449

Merged
merged 5 commits into from
Jan 14, 2025
Merged

Conversation

pletelli
Copy link
Collaborator

@pletelli pletelli commented Jan 10, 2025

J'ai beaucoup réfléchi pour essayer d'éviter cette erreur
Mais je n'y arrive pas malgré :

  • un sleep(20) pour éventuellement attendre que les triggers soient terminés
  • la modification des modèles Ica_ pour que remplacer partout dans les on_delete=models.CASCADE par on_delete=models.DO_NOTHING
  • le modification de l'ordre dans lesquelles les tables sont supprimées

J'ai donc juste supprimé la suppression des tables 👯‍♀️
le schema_editor.delete_model(table) est normalement nécessaire car les tables ne sont pas créés lors des test (elles sont unmanaged). Mais ça n'a pas l'air d'impacter les autres tests qu'elles persistent. Et la database est de toutes façons supprimée en fin de tests.

@pletelli pletelli marked this pull request as ready for review January 14, 2025 10:16
@pletelli pletelli changed the title WIP: Test de la création de déclaration historique Test de la création de déclaration historique Jan 14, 2025
@pletelli pletelli requested a review from alemangui January 14, 2025 10:21

with connection.schema_editor() as schema_editor:
schema_editor.delete_model(IcaEtablissement)
# super().tearDown()
Copy link
Collaborator

Choose a reason for hiding this comment

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

J'imagine qu'il faut qd même appeler le super().tearDown() ?

schema_editor.delete_model(IcaEtablissement)
# super().tearDown()
for table in [IcaVersionDeclaration, IcaComplementAlimentaire, IcaDeclaration, IcaEtablissement]:
for table_item in table.objects.all():
Copy link
Collaborator

Choose a reason for hiding this comment

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

Peut-être un table.objects.all().delete() au lieu de la boucle for

@pletelli pletelli merged commit e570400 into staging Jan 14, 2025
5 checks passed
@pletelli pletelli deleted the test-declaration branch January 14, 2025 13:03
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.

2 participants