PowerShell / PowerShell/PowerShellEditorServices

Trying to "wait for debugger" in tests doesn't work

Offen
#676 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Area-Test Issue-Bug
Vorherrschende Sprache
C#
Sterne
767
Forks
266
Ø Merge
3 T. 16 Std.
Gemergte PRs (30 T.)
1

Beschreibung

There are lines in both LanguageServerTests.cs and DebugServerTests.cs that can be uncommented to enable debugger attachment:

https://github.com/PowerShell/PowerShellEditorServices/blob/4bb71bd93320ba9b2105e3b5d2007f09f0f4a959/test/PowerShellEditorServices.Test.Host/LanguageServerTests.cs#L53

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).

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Untersuche die Zeilen zur Debugger-Anbindung in LanguageServerTests.cs und DebugServerTests.cs, beginnend mit der verlinkten Stelle in LanguageServerTests.cs. Reproduziere das Verhalten in VSCode und vergleiche die nativen Fehlerdialoge sowie die fortgesetzte Testausführung mit dem vorgesehenen Debugging-Workflow. Als erledigt gilt die Aufgabe, wenn das veraltete Verhalten entfernt oder durch die vorgeschlagene Pause des Test-Runners und die Prozessidentifizierung ersetzt wurde und die Tests eine nutzbare Möglichkeit zum Anhängen bieten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp, powershell, vscode
Bereich
testing-qa
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.