Skip to content
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

Add Testing user guide outline #7310

Merged
merged 9 commits into from
Jun 14, 2024
Merged

Add Testing user guide outline #7310

merged 9 commits into from
Jun 14, 2024

Conversation

ntrogh
Copy link
Contributor

@ntrogh ntrogh commented May 24, 2024

Fixes #7208

@ntrogh ntrogh requested a review from isidorn May 24, 2024 13:05
@ntrogh
Copy link
Contributor Author

ntrogh commented May 24, 2024

@isidorn I created an outline for the Testing user guide. I'll continue to further complete the doc. Welcome your feedback.

@isidorn
Copy link
Contributor

isidorn commented May 28, 2024

Sorry for the slow response. I just checked this out and I like this outline!
The only part that I think is super important and I do not know if you covered is the "testing getting started". Do we plan to have a paragraph in the intro on how to actually setup testing?

And I suggest to share with Connor when possible to also get his feedback.

@ntrogh
Copy link
Contributor Author

ntrogh commented May 28, 2024

The only part that I think is super important and I do not know if you covered is the "testing getting started". Do we plan to have a paragraph in the intro on how to actually setup testing?

@isidorn I plan to have the high-level getting started experience as part of the intro. The actual details are extension/language specific. If this information becomes too long, I might move it into a separate section.

I'll start adding some more details, and will also include Connor in the next review cycle.

@isidorn
Copy link
Contributor

isidorn commented May 28, 2024

Sounds good. Thank you very much 🙏

docs/editor/testing.md Outdated Show resolved Hide resolved
isidorn
isidorn previously approved these changes May 31, 2024
@ntrogh ntrogh requested a review from connor4312 May 31, 2024 13:29
@ntrogh
Copy link
Contributor Author

ntrogh commented May 31, 2024

@connor4312 I've started working on a user guide about Testing. Appreciate your perspectives on the current article - it's about 80% ready. Is the outline and structure ok, are there things missing or incorrect? Thanks!

docs/editor/testing.md Outdated Show resolved Hide resolved

The Test Explorer view provides a centralized place to manage and run your tests. You can access the Test Explorer view by selecting the beaker icon in the Activity Bar or by using the **Testing: Focus on Test Explorer View** command in the Command Palette (`kb(workbench.action.showCommands)`).

If you have a project with tests, the Test Explorer view automatically discovers and lists the tests in your workspace. By default, the discovered tests are displayed in a tree view in the Test Explorer. The tree view matches the hierarchical structure of your tests, making it easy to navigate and run your tests.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's also a little bit of "it depends" here, for example the Python extension requires users to run a configure command before it shows tests. (A button to do so is shown in the welcome view if there are no tests in the workspace yet.) The user might need to consult the readme of their extension to set up tests.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a note at the end of this section.

docs/editor/testing.md Outdated Show resolved Hide resolved

## Task integration

Tasks in VS Code can be configured to run scripts and start processes within VS Code, without having to enter a command line or write new code. In VS Code, you can define a default test task that runs your tests, and optionally create keyboard shortcuts to run the tests.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently tasks don't have special integration into VS Code's testing functionality, so running tests in a task won't update VS Code's UI.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Included this in the note that extension can implement integrated test automation features.

docs/editor/testing.md Outdated Show resolved Hide resolved
@ntrogh ntrogh marked this pull request as ready for review June 7, 2024 11:56
@ntrogh ntrogh merged commit 9648869 into main Jun 14, 2024
2 checks passed
@ntrogh ntrogh deleted the testing-user-guide branch June 14, 2024 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Testing user guide article
3 participants