Update npm dependencies

[florimondmanca]

Ça faisait longtemps qu'on avait pas lancé make update_deps

Edit : je n'ai pu mettre à jour que les dépendences npm. Il y a une dépendance PHP qui, lorsqu'elle est mise à jour, déclenche des erreurs 500 sur SaveGeneralInfoController et AddLocationController. Voir fil de la discussion.

[mmarchois]

La dernière fois que j'avais voulu le faire, on avait l'erreur

  The geometry_type of a spatial column cannot be changed (Requested changing  
   type from "POINT" to "GEOMETRY" for column "from_point" in table "location

A voir si le problème persiste toujours

[florimondmanca]

Oui on l'a toujours quand on fait make check

Y'a qq bugs graphiques, je vais repasser sur le changelog du DSFR

[florimondmanca]

Le problème de POINT a l'air lié à doctrine/om 2.16+

Ça crée un problème du côté de jsor/doctrine-postgis : jsor/doctrine-postgis#62

Un fix a été proposé jsor/doctrine-postgis#64

Il fonctionne mais n'a pas encore été mergé. J'ai mis un commentaire pour demander comment le faire avancer

En attendant je crois qu'on peut pin à doctrine/orm 2.16+, si ça marche

[florimondmanca]

Finalement c'est doctrine/orm 2.15.5 qui a l'air d'être fautif

La version 2.15.4 fonctionne, la 2.15.5 non car elle se met à utiliser la nouvelle API "platformOptions" et c'est ça qui fait que doctrine-postgis ne suit plus (car il utilisait l'ancienne API "customSchemaOptions")

Je pin à < 2.15.5 et ça devrait aller

[florimondmanca]

On progresse, on dirait qu'il ne reste plus qu'à corriger les tests E2E.

[mmarchois]

Super que tu aies pu trouver l'origine !

[florimondmanca]

Bon, comme les premiers fix que j'ai fait dans le code, ça a aussi l'air d'être un problème avec des changements sur les fr-radio-rich

On a une erreur "element intercepts pointer events" dans regulation_order.page.js:65

J'ai pas encore trouvé comment résoudre

J'ai essayé

• .check({ force: true })
• .click({ force: true })
• .click({ force: true })
• .locator('label').filter({ hasText: 'Tous les véhicules' }).first().click()
• .click() avec un datest-testid...

Rien n'a fonctionné

[florimondmanca]

J'ai dû mettre des data-testid sur les div avec le fr-radio-rich, impossible de faire en sorte que Playwright clique sur l'option en faisant référence "sémantique" au radio (getByRole, getByLabel, etc)

Normalement les tests E2E vont passer cette fois

On aura alors un déploiement Scalingo qu'on peut recetter en profondeur

Edit : tristesse.

[florimondmanca]

La CI échoue désormais à cause d'erreurs 500 (donc côté Symfony) que je n'ai pas en local

J'ai récupéré le test.log sur la CI

On voit ces deux erreurs "critiques"

[2023-09-19T09:04:16.497200+00:00] php.CRITICAL: Uncaught Error: Cannot initialize readonly property Symfony\Component\ExpressionLanguage\Node\ConstantNode::$isNullSafe from scope Symfony\Component\VarExporter\Internal\Hydrator {"exception":"[object] (Error(code: 0): Cannot initialize readonly property Symfony\\Component\\ExpressionLanguage\\Node\\ConstantNode::$isNullSafe from scope Symfony\\Component\\VarExporter\\Internal\\Hydrator at /home/runner/work/dialog/dialog/vendor/symfony/var-exporter/Internal/Hydrator.php:63)"} []
[2023-09-19T09:04:16.506515+00:00] request.CRITICAL: Uncaught PHP Exception Error: "Cannot initialize readonly property Symfony\Component\ExpressionLanguage\Node\ConstantNode::$isNullSafe from scope Symfony\Component\VarExporter\Internal\Hydrator" at /home/runner/work/dialog/dialog/vendor/symfony/var-exporter/Internal/Hydrator.php line 63 {"exception":"[object] (Error(code: 0): Cannot initialize readonly property Symfony\\Component\\ExpressionLanguage\\Node\\ConstantNode::$isNullSafe from scope Symfony\\Component\\VarExporter\\Internal\\Hydrator at /home/runner/work/dialog/dialog/vendor/symfony/var-exporter/Internal/Hydrator.php:63)"} []

Vu que ça mentionne ExpressionLanguage, c'est peut-être un bug déclenché par les contraintes de type 'expression' dans SaveRegulationGeneralInfoCommand.xml... À voir si c'est un bug connu de Symfony...

[florimondmanca]

Bon, j'ai très envie d'abandonner la màj côté Symfony car même en pinnant un bon paquet de dépendances, ce problème obscur d'erreurs 500 sur la CI se produit toujours.

Je n'ai pas réussi à trouver d'où il venait, et je n'ai pas trouvé de ticket ouvert qui parlerait de ça. Y'a seulement symfony/symfony#46841 mais l'OP prétend qu'à l'époque en mettant à jour var-exporter de 6.0.6 à 6.1.* ça avait réglé le problème. Ici même en pinnant var-exporter à la version qui fonctionne actuellement, on a quand même le pb. Alors je suis un peu perdu.

On pourrait cependant garder la màj des dépendances NPM, en espérant qu'aucune ne nécessite de mettre aussi à jour côté Symfony (Stimulus, etc).

@mmarchois Un avis ?

[florimondmanca]

Voilà encore autre chose... J'ai push un nouveau commit mais GitHub ne l'affiche pas dans cette PR. 😂 4134736

[florimondmanca]

Bon, on dirait qu'effectivement le problème est causé par l'une ou plusieurs des dépendances PHP mises à jour, et ce n'est pas par une dépendance Symfony puisqu'en pinnant Symfony on a aussi le problème.

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (541a5e9) 100.00% compared to head (dab069b) 100.00%.

Additional details and impacted files

@@             Coverage Diff             @@
##                main      #490   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity       587       587           
===========================================
  Files            120       120           
  Lines           2533      2533           
===========================================
  Hits            2533      2533           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[florimondmanca]

@mmarchois OK la PR est prête à review, au final il n'y a que les dépendances NPM qui sont mises à jour.

Je propose d'attendre encore pour la màj des dépendances PHP... Sauf si on arrive à trouver le coupable des erreurs 500, mais j'y ai déjà presque passé la journée, je me dis qu'il faut arrêter les frais !

[florimondmanca]

Déploiement Scalingo HS sur main (mais il était passé sur la branche)

/build/e9d22590-26fa-43d1-ad3b-7aaaa55eaa9a/bin/jq: line 1: syntax error near unexpected token newline' /build/e9d22590-26fa-43d1-ad3b-7aaaa55eaa9a/bin/jq: line 1: '
composer.json is not a valid JSON file.

J'essaie de relancer