microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
162.38k stars 28.62k forks source link

[Test UI] [Feature Request] Allow message for skipped() test function #133198

Open JustinGrote opened 2 years ago

JustinGrote commented 2 years ago

@connor4312 Issue Type: Feature Request

In Pester, tests can be skipped/inconclusive for reasons, for instance, because other tests precluded the need, or a flaw in a test made this test be inconclusive, or because a dependent test did not complete successfully.

Today I have to express these as error messages, I would prefer to be able to express it as part of the skipped() method using the same nomenclature as error, and also have a highlight for skipped messages that is a warning color or a gray color. image

VS Code version: Code - Insiders 1.61.0-insider (0c83ca1ced83259971d80890d93abbe19cbfc992, 2021-09-15T05:16:32.507Z) OS version: Windows_NT x64 10.0.22000 Restricted Mode: No

Extensions (45) Extension|Author (truncated)|Version ---|---|--- vscode-log-viewer|ber|0.12.2 markdown-checkbox|bie|0.3.2 vscode-opennewinstance|chr|0.0.12 esbuild-problem-matchers|con|0.0.2 vscode-eslint|dba|2.1.25 vscode-generate-getter-setter|DSK|0.5.0 gitlens|eam|11.6.0 filter-lines|ear|1.0.0 EditorConfig|Edi|0.16.4 prettier-vscode|esb|8.1.0 copilot|Git|1.4.2678 remotehub|Git|0.14.0 vscode-pull-request-github|Git|0.30.0 todo-tree|Gru|0.0.214 vscode-test-explorer|hbe|2.21.1 vscode-drawio|hed|1.6.2 change-case|hjd|1.0.2 git-graph|mhu|1.30.0 vscode-bicep|ms-|0.4.613 vscode-docker|ms-|1.16.1 csharp|ms-|1.23.15 vscode-dotnet-runtime|ms-|1.2.0 remote-containers|ms-|0.195.0 remote-ssh-nightly|ms-|2021.9.19620 remote-wsl|ms-|0.58.2 azure-account|ms-|0.9.8 powershell-preview|ms-|2021.9.1 test-adapter-converter|ms-|0.1.2 vscode-paste-image|mus|1.0.4 indent-rainbow|ode|8.0.0 docthis|oou|0.8.2 vscode-jest|Ort|4.1.2 pester-test|psp|0.0.0-source quicktype|qui|12.0.46 vscode-commons|red|0.0.6 vscode-yaml|red|0.22.0 vscode-sort-json|ric|1.20.0 even-better-toml|tam|0.14.2 vscode-inline-values-powershell|Tyl|0.0.5 errorlens|use|3.4.0 vscode-icons|vsc|11.6.0 codetour|vsl|0.0.58 gistfs|vsl|0.3.0 gitmoji-vscode|Vtr|1.0.7 better-align|wwm|1.1.6
jdneo commented 2 years ago

+1 for this.

Maybe the message can be an optional parameter. And make some UI highlight to tell the user that this test case is skipped. Currently, the skipped item will only have a skipped status icon in test explorer. But in the editor gutter, it still displays as a run button, which is a little bit confusing to users.

image

BDisp commented 2 months ago

Maybe a good solution is to add a "Show Only Skipped Tests" item on the dropdown when the "More Filters..." icon is clicked. Thanks.

image