Test execution timeout it not properly detected
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
Línea de trabajo
Comienza reproduciendo el fallo con Additional Test Arguments establecido en --timeout=1; después ejecuta codeql test run --format=json e inspecciona cómo la extensión gestiona failureStage EXECUTION, el array messages vacío y el archivo actual. Verifica que la pestaña Testing deje de ejecutarse y que el timeout aparezca en la salida de CodeQL Tests, en lugar de aparecer únicamente en el archivo .actual.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
When test execution times out (after 5 minutes by default), the extension does not seem to notice that properly:
- The "Testing" tab still shows the test as running
- The timeout error appears (only) in the
.actualfile instead of the "CodeQL Tests" output (similar to #819)
Version
Extension:
CodeQL extension version: 1.6.12
CodeQL CLI version: 2.10.4
Platform: win32 x64
VS Code:
Version: 1.71.0 (user setup)
Commit: 784b0177c56c607789f9638da7b6bf3230d47a8c
Date: 2022-09-01T07:36:10.600Z
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: 10.2.154.15-electron.0
OS: Windows_NT x64 10.0.19043
Sandboxed: No
To reproduce
- Open the extension settings
- Edit "Additional Test Arguments" and add
"--timeout=1" - Run a test
❌ The extension does not properly handle the test failure
(remember to revert your settings changes)
Expected behavior
- The extension should properly detect that the test failed
- The timeout error message should appear in the "CodeQL Tests" output and not in the
.actualfile
Additional context
The .actual file contains:
Timeout (5m0s) in api-visibility-issues#09cf2679::getATypeExposingElement#6#fffffff
(with api-visibility-issues being the query name)
Running the test with codeql test run --format=json yields the following response (might helpful to adjust the parsing logic in the extension?):
{
"test" : "<test-dir>/api-visibility-issues.qlref",
"pass" : false,
"failureStage" : "EXECUTION",
"messages" : [ ],
"evaluationMs" : 305698,
"expected" : "<test-dir>/api-visibility-issues.expected",
"actual" : "<test-dir>/api-visibility-issues.actual",
"diff" : [
"--- expected",
"+++ actual",
"@@ -1,30 +1,1 @@",
"-| ...",
"+Timeout (5m0s) in api-visibility-issues#09cf2679::getATypeExposingElement#6#fffffff"
]
}
(slightly minified)
- Lenguaje dominante
- TypeScript
- Estrellas
- 539
- Forks
- 240
- Merge medio
- 2 d 6 h
- PR fusionados (30 d)
- 29
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 github/vscode-codeql
-
bug VSCode
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
github/vscode-codeql#4471 · 1 comentario ·
-
bug enhancement VSCode
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
github/vscode-codeql#592 · 2 comentarios ·
-
enhancement VSCode
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
github/vscode-codeql#4432 · 1 comentario ·
-
bug VSCode
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
github/vscode-codeql#4356 ·
-
bug VSCode
Dificultad 3/5 1-2 días Aptitud para principiantes 64/100
github/vscode-codeql#4339 · 1 comentario ·
Todos los issues de github/vscode-codeql
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
safetrustcr/dApp-SafeTrust#426 ·
-
area:workflow bug ready-for-agent
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
fil-donadoni/tolaria#4409 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Fission-AI/OpenSpec#1960 ·
-
Add dependabot Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
corsairdev/corsair#1764 ·