test(planning): make the surface ui_journey checks wait for the state they assert
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 63/100
Línea de trabajo
Start with plugins/planning/surface/surface.test.sh and inspect the failing ui_journey.2 and ui_journey.13 checks to see what state each asserts and what it currently waits for. Replace timing-dependent waits with waits tied to the asserted state, then run the script 20 consecutive times under load; done means all 20 runs pass.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
plugins/planning/surface/surface.test.sh fails sometimes with no change to the files it covers. While verifying #7009, three runs on the same head gave three different results:
- run 1: exit 1, failing
ui_journey.2("Answer again moves focus to the first choice"); - run 2: exit 1, failing
ui_journey.2andui_journey.13; - run 3: exit 0 (PASS=480).
No file under plugins/planning/surface changed between the runs, and the failing checks differ from run to run. That points to a browser timing race in the UI journey checks: a focus assertion running before the page has settled.
Fix
Find what each failing check waits on and make it wait for the state it asserts. For example, wait for the focused element or a settled-render signal instead of a fixed delay, so the result does not depend on machine load.
Done when
surface.test.shpasses 20 consecutive runs on a loaded machine (for example, while another test suite runs in parallel).
Refs: #7009
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 7 h 42 min
- PR fusionados (30 d)
- 923
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 melodic-software/claude-code-plugins
-
priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
melodic-software/claude-code-plugins#7351 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
priority: high
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
melodic-software/claude-code-plugins#7312 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#7288 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
knowledge: docpage-digest dual verification defines no verdict vocabulary or pass/fail thresholdAbiertoneeds-human
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
melodic-software/claude-code-plugins#7286 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#7282 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de melodic-software/claude-code-plugins
Issues similares
-
[Bug]: armbian-firstlogin: preset Wi-Fi key with " or \ breaks netplan configPosiblemente ocupada @jfcabral la tomó hoy. AbiertoBug Good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
armbian/build#10995 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
lane:repo
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
SPARDiffData/differential-data-collection#42 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
alunduil/alunduil-chezmoi#923 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
xonsh/xontrib-abbrevs#11 ·