Project

General

Profile

Development #51327

misc: interdire l'appel à data_class() sur un {form,card]def léger

Added by Benjamin Dauvergne 3 days ago. Updated 2 days ago.

Status:
En cours
Priority:
Normal
Target version:
-
Start date:
22 Feb 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Sait on jamais.


Files

History

#1

Updated by Benjamin Dauvergne 3 days ago

#2

Updated by Benjamin Dauvergne 3 days ago

Le fait que les tests passent constituera déjà une vérification que la situation n'existe pas de manière évidente actuellement.

#3

Updated by Benjamin Dauvergne 2 days ago

1. ne sert pas à grand chose pour l'instant, mais ça garantit qu'on ne fait pas n'importe quoi avec data_class()
2. parce que je soupçonne qu'on est pas aidé par NFS et qu'on est un peu trop permissif avec les EOFError je ne les laisse plus passer dans storage_load() sauf si on est bien sûr qu'on est sur un pickle vieux format (où o.field devrait forcément être une liste)
3. parce que je ne voyais pas les exceptions dans 1.

#4

Updated by Frédéric Péters 2 days ago

C'est rouge mais je commente quand même :

  • 0001 : en plus de lever l'erreur il y a modification pour créer des lightweight là où ça n'était pas le cas, il ne faudrait pas ça dans de patch.
  • 0003 : pas fan du tout de changer de manière si totale le comportement, je préfère vraiment garder les tests se comporter comme l'application, et qu'un problème y soit annoncé comme une erreur 500.
#5

Updated by Frédéric Péters 2 days ago

  • Patch proposed changed from Yes to No
  • Status changed from Solution proposée to En cours
#6

Updated by Benjamin Dauvergne 2 days ago

Oui après d'autres tests sur le SaaS HDS je me fourvoie certainement, il n'y a pas de NFS là bas et l'écriture atomique marche parfaitement (j'ai lancé plein de process qui font open(.tmp.<random>).write(' ' * 10000).flush().rename('x'), je n'ai jamais réussi à trouver autre chose que ma chaîne de 10000 caractères dans 'x' avec d'autres process concurrents qui lisent).

Je vais juste continuer sur l'aspect surveiller un peu plus quand on utilise dataclass() sur un FormDef léger.

#7

Updated by Benjamin Dauvergne 2 days ago

Frédéric Péters a écrit :

  • 0003 : pas fan du tout de changer de manière si totale le comportement, je préfère vraiment garder les tests se comporter comme l'application, et qu'un problème y soit annoncé comme une erreur 500.

Oui je comprends, je ne pensais pas le pousser à la fin, dans l'absolu ça pourrait être sympa d'avoir quelque chose pour chaîne l'exception initiale à celle émise par webtest pour la 500.

Also available in: Atom PDF