fix(e2e): docs_site test intermittently times out
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 85/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- github-actions, nodejs
- Área
- ci-cd, testing-qa
Línea de trabajo
El timeout está en e2e/test/commands/create/docs_site/docs_site_test.dart; la solución consiste en aumentarlo a Duration(minutes: 5), igual que en la prueba flame_game. Opcionalmente, añade --no-audit --no-fund --prefer-offline a la llamada de npm install que ejecuta la prueba. Ejecuta ese archivo de pruebas en e2e para confirmar que pasa y comprueba que el error de teardown Directory not empty ya no aparece.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Dart
- Estrellas
- 2.4k
- Forks
- 244
- Merge medio
- 1 d 4 h
- PR fusionados (30 d)
- 25
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de VeryGoodOpenSource/very_good_cli
-
fix(mcp): test timeout_seconds does not kill a hung test run, which blocks every later tool callPosiblemente ocupada @unicoderbot la tomó hoy. Abiertobug
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
VeryGoodOpenSource/very_good_cli#1765 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
VeryGoodOpenSource/very_good_cli#1762 · 1 comentario · 4 reacciones ·
Los mantenedores suelen responder en 1 día
-
feat: merge lcov reports from sharded test runsQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertofeature
Dificultad 5/5 Más de una semana Aptitud para principiantes 42/100
VeryGoodOpenSource/very_good_cli#1754 ·
Los mantenedores suelen responder en 1 día
-
feature
Dificultad 5/5 Más de una semana Aptitud para principiantes 38/100
VeryGoodOpenSource/very_good_cli#1695 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
feat(mcp): stream progress notifications for long-running toolsQuizá libre de nuevo @vgvbot la tomó hace 64 días y no hay ningún pull request abierto. Abiertofeature
VeryGoodOpenSource/very_good_cli#1616 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de VeryGoodOpenSource/very_good_cli
Issues similares
-
backlog:low bug localization
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
simonoppowa/OpenNutriTracker#1366 ·
Los mantenedores suelen responder en 1 día
-
backlog documentation good first issue priority: medium
Dificultad 2/5 Medio día Aptitud para principiantes 78/100
-
coverage 1.16.0: format_coverage fails on the `{}` that `dart test --coverage` writes for fully skipped suitesPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
The icons in "Tracks"Abiertofeature
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
finamp-app/finamp#1804 ·
Los mantenedores suelen responder en 9 días
-
fyi-ecosystem platform-ios team-ios
Dificultad 1/5 Menos de una hora Aptitud para principiantes 70/100
flutter/flutter#194348 · 1 comentario ·
Los mantenedores suelen responder en 1 día