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
csechet commented 2024-02-28 18:57:27 +01:00
Owner
Copy Link
No description provided.
csechet added 1 commit 2024-02-28 18:57:27 +01:00
csechet force-pushed wip/87584-tests-unitaires-js-encore-casses from 8318abe990 to 36a283b6f9 2024-02-28 18:58:09 +01:00 Compare
fpeters approved these changes 2024-02-29 08:00:59 +01:00
setup-vitest.sh
@ -4,3 +4,3 @@
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
Owner
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
Author
Owner
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.
csechet merged commit 96f992e11d into main 2024-02-29 17:50:10 +01:00
csechet deleted branch wip/87584-tests-unitaires-js-encore-casses 2024-02-29 17:50:11 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#484
No description provided.
Delete Branch "wip/87584-tests-unitaires-js-encore-casses"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?