PowerShell / PowerShell/PowerShellEditorServices
Trying to "wait for debugger" in tests doesn't work
Personne n'a encore pris cette issue.
- Langage dominant
- C#
- Étoiles
- 767
- Forks
- 266
- Merge moyen
- 3 j 16 h
- PR mergées (30 j)
- 1
Description
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).
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Inspectez les lignes d’attachement du débogueur dans LanguageServerTests.cs et DebugServerTests.cs, en commençant par l’emplacement lié dans LanguageServerTests.cs. Reproduisez le comportement dans VSCode et comparez les boîtes de dialogue d’erreur natives ainsi que la poursuite de l’exécution des tests avec le workflow de débogage prévu. La tâche est terminée lorsque le comportement obsolète est supprimé ou remplacé par la mise en pause proposée du test-runner et l’identification du processus, et que les tests fournissent un point d’attachement utilisable.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- csharp, powershell, vscode
- Domaine
- testing-qa
- Type d'issue
- Refactorisation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100