PowerShell / PowerShell/PowerShellEditorServices
Trying to "wait for debugger" in tests doesn't work
Nessuno ha ancora preso questa issue.
- Lingua principale
- C#
- Stelle
- 767
- Fork
- 266
- Merge medio
- 3g 16h
- PR unite (30g)
- 1
Descrizione
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).
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Esamina le righe relative all’aggancio del debugger in LanguageServerTests.cs e DebugServerTests.cs, iniziando dalla posizione collegata in LanguageServerTests.cs. Riproduci il comportamento in VSCode e confronta le finestre di dialogo degli errori native e la continuazione dell’esecuzione dei test con il workflow di debug previsto. Il lavoro è completato quando il comportamento obsoleto è stato rimosso o sostituito con la pausa proposta del test-runner e l’identificazione del processo, e i test forniscono un punto di aggancio utilizzabile.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- csharp, powershell, vscode
- Ambito
- testing-qa
- Tipo di issue
- Refactoring
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100