entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

js: pin happy-dom version (#87584) #484

Merged
csechet merged 1 commits from wip/87584-tests-unitaires-js-encore-casses into main 2024-02-29 17:50:10 +01:00
Conversation 0 Commits 1 Files Changed 1 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
setup-vitest.sh
View File

@ -3,4 +3,4 @@
pip install $*
nodeenv --prebuilt --python-virtualenv
source $VIRTUAL_ENV/bin/activate # source again to activate npm from env
npm install -g vitest@"<1.1.0" happy-dom
npm install -g vitest@"<1.1.0" happy-dom@"=13.4.0"
fpeters commented 2024-02-29 08:00:55 +01:00
Review
Copy Link

Il n'y a pas un truc genre "!=13.5.0" (en supposant que c'est ce numéro de version qui est cassé), qui ferait qu'une correction arrivant ça passe automatiquement dessus ?

Ou c'est futile et il faut se rendre à l'évidence que sur du js la seule possibilité est de fixer toutes les versions ?

(aussi, https://jenkins.entrouvert.org/job/gitea/job/passerelle/job/main/455/consoleFull je ne vois pas d'échec et je ne vois pas non plus d'affichage des versions utilisées, ça serait utile.)

Il n'y a pas un truc genre "!=13.5.0" (en supposant que c'est ce numéro de version qui est cassé), qui ferait qu'une correction arrivant ça passe automatiquement dessus ? Ou c'est futile et il faut se rendre à l'évidence que sur du js la seule possibilité est de fixer toutes les versions ? (aussi, https://jenkins.entrouvert.org/job/gitea/job/passerelle/job/main/455/consoleFull je ne vois pas d'échec et je ne vois pas non plus d'affichage des versions utilisées, ça serait utile.)
csechet commented 2024-02-29 10:21:14 +01:00
Review
Copy Link

(aussi, https://jenkins.entrouvert.org/job/gitea/job/passerelle/job/main/455/consoleFull je ne vois pas d'échec et je ne vois pas non plus d'affichage des versions utilisées, ça serait utile.)

Le problème est ici : https://jenkins.entrouvert.org/job/gitea/job/passerelle/job/wip%252F86853-interface-lecteur/10/console, il apparaissait sur la branche suite à la modification des tests, je pense.

Il n'y a pas un truc genre "!=13.5.0" (en supposant que c'est ce numéro de version qui est cassé), qui ferait qu'une correction arrivant ça passe automatiquement dessus ?

Ou c'est futile et il faut se rendre à l'évidence que sur du js la seule possibilité est de fixer toutes les versions ?

Ce qui commence à m'inquiéter, c'est qu'en un mois, les tests on été cassés deux fois, par une release de vitest et une de happy-dom. Dans cette situation, je trouve plus sûr de garder une version avec laquelle on sait que les tests actuels fonctionnent, plutôt qu'ils se mettent à rater parce que les développeurs de ces librairies font n'importe quoi.

Dans un contexte normal, on aurait un package-lock.json qui fait ça, je pense que c'est la solution la plus perenne.

> (aussi, https://jenkins.entrouvert.org/job/gitea/job/passerelle/job/main/455/consoleFull je ne vois pas d'échec et je ne vois pas non plus d'affichage des versions utilisées, ça serait utile.) Le problème est ici : https://jenkins.entrouvert.org/job/gitea/job/passerelle/job/wip%252F86853-interface-lecteur/10/console, il apparaissait sur la branche suite à la modification des tests, je pense. > Il n'y a pas un truc genre "!=13.5.0" (en supposant que c'est ce numéro de version qui est cassé), qui ferait qu'une correction arrivant ça passe automatiquement dessus ? > > Ou c'est futile et il faut se rendre à l'évidence que sur du js la seule possibilité est de fixer toutes les versions ? Ce qui commence à m'inquiéter, c'est qu'en un mois, les tests on été cassés deux fois, par une release de vitest et une de happy-dom. Dans cette situation, je trouve plus sûr de garder une version avec laquelle on sait que les tests actuels fonctionnent, plutôt qu'ils se mettent à rater parce que les développeurs de ces librairies font n'importe quoi. Dans un contexte normal, on aurait un package-lock.json qui fait ça, je pense que c'est la solution la plus perenne.