Produits Entr'ouvert » Passerelle

Tu n'as normalement pas le droit d'utiliser (?i) c'est de la syntaxe regexp python et pas javascript (https://json-schema.org/understanding-json-schema/reference/regular_expressions.html); on va préférer [Oo][Uu][Ii], aussi la regexp match n'import quoi qui contien oui, il faudrait ajouter ^...$.

Benjamin Dauvergne a écrit :

Tu n'as normalement pas le droit d'utiliser (?i) c'est de la syntaxe regexp python et pas javascript (https://json-schema.org/understanding-json-schema/reference/regular_expressions.html); on va préférer [Oo][Uu][Ii], aussi la regexp match n'import quoi qui contien oui, il faudrait ajouter ^...$.

Aussi j'ai l'impression que tu es trop permissive, dans encode_bool tu devrais finir par retourner obj pas '', et laisser xmlschema finir la validation.

Ne manque-t-il pas la définition de schema_date_optional ?

alors c'était fait exprès mais il faut que je retrouve pourquoi ... je finis de me prendre la tête sur le #38543 et je cherche la raison :)

donc c'est fait exprès:
je ne veux pas passer là-dedans: https://git.entrouvert.org/passerelle.git/tree/passerelle/utils/xml.py?h=wip/38464-toulouse-axel-null-and-bool#n147 pour avoir de façon automatique la contruction DATEType = DATEREQUIREDType + empty-type sans avoir besoin de définir le type json à la main.

Si je définis le schema OUINONType, c'est parce que OUINONType n'est PAS la combinaison de OUINONREQUIREDType + empty-type dans les xsd fournis ...

Par contre je dois définir les encode/decode des 4 types OUINON et DATE pour bien passer là-dedans:
https://git.entrouvert.org/passerelle.git/tree/passerelle/utils/xml.py?h=wip/38464-toulouse-axel-null-and-bool#n101
https://git.entrouvert.org/passerelle.git/tree/passerelle/utils/xml.py?h=wip/38464-toulouse-axel-null-and-bool#n109

Ok pour schema, effectivement. Cependant si j'ajoute ce test :

--- a/tests/test_toulouse_axel_schema.py
+++ b/tests/test_toulouse_axel_schema.py
@@ -46,6 +46,8 @@ def test_date_mapping(date_type):
     json_data = schema.decode(xml_data)
     assert json_data['DATE'] == '2019-12-12'

+    if date_type == 'DATEType':
+        xml_data = schema.encode({'PORTAIL': {'DATE': ''}})
+        assert xml_data.find('DATE').text == ''
+        json_data = schema.decode(xml_data)
+        assert json_data['DATE'] == None

Ca crashe dès l'encode parce que '' n'est pas une date. Il faut que j'ajoute ceci pour que ça passe :

--- a/passerelle/contrib/toulouse_axel/models.py
+++ b/passerelle/contrib/toulouse_axel/models.py
@@ -55,10 +55,19 @@ class AxelSchema(JSONSchemaFromXMLSchema):
             'pattern': '[0-9]{4}-[0-9]{2}-[0-9]{2}',
         }

     def encode_date_optional(self, obj):
+        if not obj:
+            return ''
         return self.encode_date(obj)

Je me dis que mon test est faux, que ce genre de XML n'est pas attendu ...? (Oui, je suis dans la vase, sans doute)

Bon c'est sans doute encore un peu mon esprit tordu mais j'avoue ne pas comprendre que :

    xml_data = schema.encode({'PORTAIL': {'DATE': 'foobar'}})
    assert xml_data.find('DATE').text == 'foobar'

ne plante pas. Ca veut dire qu'on peut construire un XML foireux, qui ne suive pas le format de date imposé par le schéma ?

Finalement j'ai un peu le même soucis avec les tests sur bool: FOOBAR, 42, OUIFOOBAR ou FOONONBAR, ça devrait pas passer ?...

Bon, sans doute que c'est que dans ma tête embrumée et que tout ça est sans aucune importance dans l'usage exact qui sera fait de ce principe dans le connecteur... j'arrive pas encore bien à le percevoir. L'impression que je pinaille pour rien du tout.

Benjamin Dauvergne a écrit :

Aussi j'ai l'impression que tu es trop permissive, dans encode_bool tu devrais finir par retourner obj pas '', et laisser xmlschema finir la validation.

voila pourquoi je renvoie un truc foireux :)