fix(e2e): docs_site test intermittently times out
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 85/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- github-actions, nodejs
- Ambito
- ci-cd, testing-qa
Direzione di ricerca
Il timeout si trova in e2e/test/commands/create/docs_site/docs_site_test.dart; la correzione consiste nel portarlo a Duration(minutes: 5), in linea con il test flame_game. Facoltativamente, aggiungi --no-audit --no-fund --prefer-offline alla chiamata npm install eseguita dal test. Esegui quel file di test in e2e per confermare che passa e verifica che l'errore di teardown Directory not empty non compaia più.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
The create docs_site e2e test (e2e/test/commands/create/docs_site/docs_site_test.dart) intermittently times out on main.
Failing run: https://github.com/VeryGoodOpenSource/very_good_cli/actions/runs/37757567540/job/113245773965
TimeoutException after 0:02:00.000000: Test timed out after 2 minutes.
FileSystemException: Deletion failed, path = '/tmp/MCMMJN' (OS Error: Directory not empty, errno = 39)
The failing commit (#1756) is unrelated to docs_site, and the following run on main (37779724241) passed on the same code, so this is flakiness, not a regression.
Root cause
The test has a 2-minute timeout, but it normally takes almost that long:
| Run | Result | "Run Tests (e2e)" step duration |
|---|---|---|
| 35327672012 | ✅ | 1m52s |
| 35616186431 | ✅ | 1m47s |
| 36122147301 | ✅ | 1m49s |
| 37757567540 | ❌ | 2m15s (timeout) |
Most of the time goes to npm install. The generated project has no package-lock.json, and CI has no npm cache, so every run installs ~1,300 packages fresh from the registry. Timing each step locally:
| Step | Duration |
|---|---|
npm install |
72s |
npm run format |
1s |
npm run lint |
1s |
npm run build |
8s |
On CI the time depends on registry speed, so the test sometimes runs past the limit.
The Directory not empty error is a side effect: the test teardown deletes the temp directory while npm is still writing to it after the timeout.
The 2-minute timeout was copied from the Dart/Flutter e2e tests. The other tests that install heavy dependencies already use longer limits: flame_game has 5 minutes and flutter_plugin has 8.
Proposed fix
- Raise the timeout in
docs_site_test.darttoDuration(minutes: 5), matchingflame_game. - Optionally, speed up the install with
npm install --no-audit --no-fund --prefer-offline. Caching npm throughactions/setup-nodewon't work as-is, because it needs a lockfile and the template doesn't ship one.
- Lingua principale
- Dart
- Stelle
- 2.4k
- Fork
- 244
- Merge medio
- 1g 8h
- PR unite (30g)
- 27
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di VeryGoodOpenSource/very_good_cli
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
VeryGoodOpenSource/very_good_cli#1762 · 1 commento · 4 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
feat: merge lcov reports from sharded test runsForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Apertafeature
Difficoltà 5/5 Più di una settimana Idoneità per principianti 42/100
VeryGoodOpenSource/very_good_cli#1754 ·
I maintainer di solito rispondono entro 1 giorno
-
feature
Difficoltà 5/5 Più di una settimana Idoneità per principianti 38/100
VeryGoodOpenSource/very_good_cli#1695 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
feat(mcp): stream progress notifications for long-running toolsForse di nuovo libera @vgvbot l’ha presa 62 giorni fa e non c’è nessuna pull request aperta. Apertafeature
VeryGoodOpenSource/very_good_cli#1616 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
feature
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
VeryGoodOpenSource/very_good_cli#1363 · 4 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di VeryGoodOpenSource/very_good_cli
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
lugassawan/swe-workbench#799 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
epam/dmtools-dart#385 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
commetchat/commet#1085 ·
I maintainer di solito rispondono entro 1 giorno
-
first-timers-only good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
Raza-learner/Runmote#11 ·