PowerShell / PowerShell/PowerShellEditorServices
Trying to "wait for debugger" in tests doesn't work
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C#
- Estrellas
- 767
- Forks
- 266
- Merge medio
- 3 d 16 h
- PR fusionados (30 d)
- 1
Descripción
There are lines in both LanguageServerTests.cs and DebugServerTests.cs that can be uncommented to enable debugger attachment:
However, trying these in VSCode, I just get a bunch of error dialogs pop up in the machine's native UI and the tests just plough on and fail. I'm guessing this option is meant for Visual Studio usage, rather than VSCode.
I'm thinking we should retire this functionality and instead add a compile-time-conditional block with something like a Console.ReadKey() that reports the process ID of the test runner (that's how I tend to do it at the moment at least).
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.
Línea de trabajo
Inspecciona las líneas de conexión del depurador en LanguageServerTests.cs y DebugServerTests.cs, comenzando por la ubicación enlazada de LanguageServerTests.cs. Reproduce el comportamiento en VSCode y compara los diálogos de error nativos y la continuación de la ejecución de las pruebas con el flujo de trabajo de depuración previsto. Se considera terminado cuando el comportamiento obsoleto se haya eliminado o sustituido por la pausa propuesta del ejecutor de pruebas y la identificación del proceso, y las pruebas proporcionen un punto de conexión utilizable.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- csharp, powershell, vscode
- Área
- testing-qa
- Tipo de issue
- Refactorización
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100