-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Categorise test cases into test suite(s) #247
Conversation
Meson test cases may belong to test suite(s). If so, in VSCode's "Testing" feature from the Activity Bar, add the test suite(s) as a parent item and add the test cases as children. The user has the choice to run all tests belonging to a test suite. Will pass the test suite to the meson test command --suite. The option --suite is needed to run test cases that the meson config purposely excludes (for example, test suite "flaky"). Fixes mesonbuild#245 Quote the "test case" in the meson command, so can run test cases with a space in the name. The quotes are needed because `cp.ExecFileOptions` sets `shell=true`. Fixes mesonbuild#256
Thanks for the PR. I will try to test and review it this week. |
What do you think of #237? Should I merge that one first? |
Thanks for explaining the limitations. I see that there is this function. Do we have access to the test runs, so if someone runs a suite, we can properly annotate the test? https://vscode-api.js.org/interfaces/vscode.TestRun.html#passed |
I'm working on resolving the limitations. Closing this PR until I resolve. |
Alright. Really appreciate you looking into this more! |
Meson test cases may belong to test suite(s). If so, in VSCode's "Testing" feature from the Activity Bar, add the test suite(s) as a parent item and add the test cases as children.
The user has the choice to run all tests belonging to a test suite. Will pass the test suite to the meson test command --suite. The option --suite is needed to run test cases that the meson config purposely excludes (for example, test suite "flaky").
Fixes #245
Quote the "test case" in the meson command, so can run test cases with a space in the name. The quotes are needed because
cp.ExecFileOptions
setsshell=true
.Fixes #256