Repository navigation
fix: #ENABLING-1202 retire la dépendance directe à axios, adopte le bridge http d'entcore-toolkit - #389
Merged
Conversation
pascalsaussier-edifice
requested review from
benjaminperez,
damienromito,
jcbe-ode and
nabil-mansouri
September 16, 2026 15:51
damienromito
approved these changes
Sep 16, 2026
damienromito
left a comment
damienromito
left a comment
Member
There was a problem hiding this comment.
testé en local sans soucis
…ridge http d'entcore-toolkit
Migre les 4 modules de ce repo (presences, incidents, massmailing,
statistics-presences) vers entcore-toolkit@2.0.0-dev.1 : 75 fichiers
source migrés (imports/types axios -> http/HttpResponse/HttpError/
HttpRequestConfig), tous des usages réels (aucun import mort trouvé
cette fois) ; axios et axios-mock-adapter retirés du package.json.
Découverte majeure en cours de migration, systémique sur ~10 autres
repos à venir (edt, calendar, vie-scolaire, form, et la moitié des
CGI) : 12 fichiers de tests jest mockaient axios directement
(axios-mock-adapter). Vérifié empiriquement (AxiosError: Network
Error) que ça ne fonctionne plus une fois axios retiré : le bridge
`http` d'entcore-toolkit embarque sa propre instance axios bundlée
via esbuild, distincte de celle de l'app - mocker l'axios de l'app
n'intercepte jamais les appels du bridge. Corrigé en remplaçant le
mock par `jest.mock('entcore-toolkit', ...)` sur `http` (get/post/
put/delete/postFile/putFile en jest.fn()), avec un petit helper
partagé (test-utils/httpMock.ts, alias @Test-Utils) pour reconstruire
la forme d'une HttpResponse. Les assertions sur l'URL/le corps de
requête sont préservées via toHaveBeenCalledWith (équivalent du
matching d'URL de mock-adapter). 120 tests, 29 suites, tous verts.
Aucune classe de ce repo n'étend Selection<T> - non concerné par le
fix Selection.all/TS2610.
yarn.lock et package-lock.json sont dans le .gitignore de ce repo
(convention existante, pas de lockfile committé) - rien à committer
sur ce point.
Build réel vérifié : `gulp build` (4 bundles, aucune erreur) +
`yarn run build:sass`. Vérification runtime complémentaire faite via
US3-verification/ (Playwright) : les 4 checks du bridge http passent
(GET liste, POST création, erreur 500, upload FormData).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
jenkinsEdificePublic
force-pushed
the
develop-axios
branch
from
October 1, 2026 16:16
9ea8519 to
ef9289c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe your changes
Migre les 4 modules de ce repo (
presences,incidents,massmailing,statistics-presences) versentcore-toolkit@2.0.0-dev.1: 75 fichiers source migrés (imports/types axios → http/HttpResponse/HttpError/HttpRequestConfig), tous des usages réels (aucun import mort trouvé).axiosetaxios-mock-adapterretirés dupackage.json.Découverte majeure pendant cette migration, systémique sur ~10 autres repos à venir (
edt,calendar,vie-scolaire,form, et la moitié des CGI) : 12 fichiers de tests jest mockaientaxiosdirectement viaaxios-mock-adapter. Vérifié empiriquement (AxiosError: Network Error) que ça ne fonctionne plus une foisaxiosretiré : le bridgehttpd'entcore-toolkitembarque sa propre instance axios bundlée via esbuild, distincte de celle de l'app — mocker l'axios de l'app n'intercepte jamais les appels du bridge. Corrigé en remplaçant le mock parjest.mock('entcore-toolkit', ...)surhttp, avec un petit helper partagé (test-utils/httpMock.ts) pour reconstruire la forme d'uneHttpResponse. Les assertions sur l'URL/le corps de requête sont préservées viatoHaveBeenCalledWith.Aucune classe de ce repo n'étend
Selection<T>.yarn.lock/package-lock.jsonsont dans le.gitignorede ce repo (convention existante) — rien à committer sur ce point.Checklist tests
gulp build(4 bundles, aucune erreur) +yarn run build:sass.US3-verification/) : les 4 checks du bridgehttppassent (GET liste, POST création, erreur 500, upload FormData).Issue ticket number and link
ENABLING-1202 (US3, épic ENABLING-1199)
Checklist before requesting a review (magic string, indentation, comment/documentation...)
🤖 Generated with Claude Code