MockContext does not honor "warn=False" which is the default behaviour
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Start by tracing MockContext handling of ctx.run with a Result whose exited value is nonzero, paying attention to the default warn=False behavior described in the issue. Reproduce the scenario in the MockContext tests and verify that a nonzero exit causes the expected failure unless warning behavior is explicitly enabled.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I just ran into an issue where my test wasn't catching a bug I had because of this.
I gracefully catch errors:
result = ctx.run("my command", hide=False, echo=True)
if result.exited:
... do stuff
Now I forgot to add warn=True here, but wrote my tests with MockContext:
ctx = MockContext(
run={
"my command": Result("Mock output", exited=1),
}
)
... and checked the "do stuff" after the command returned a non-zero exit code worked as expected
My tests were passing without issue because MockContext isn't honoring the default warn=False. I found in production the error was not being caught because I forgot to set warn=True.
What should have happened was:
- MockContext sees
warn=Falseand pyinvoke immediately exits after the command returns a non-zero exit code - My tests fail and I catch the misconfiguration
- Lenguaje dominante
- Python
- Estrellas
- 4.8k
- Forks
- 412
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 pyinvoke/invoke
-
[Security] Shell injection via Context.cd() path argument — metacharacters not escaped (CWE-78) Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Todos los issues de pyinvoke/invoke
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
anthropics/skills#1811 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
speaches-ai/speaches#678 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
datalayer/mcp-compose#42 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
conda-forge/spacy-feedstock#177 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
UKGovernmentBEIS/inspect_evals#2523 ·