Skip to content

Enable running tests for folders in tests explorer#15758

Merged
karthiknadig merged 3 commits into
microsoft:mainfrom
vladimir-kotikov:folders_tests
Apr 21, 2021
Merged

Enable running tests for folders in tests explorer#15758
karthiknadig merged 3 commits into
microsoft:mainfrom
vladimir-kotikov:folders_tests

Conversation

@vladimir-kotikov

Copy link
Copy Markdown

This seems to be quite obvious but useful addition which I constantly miss. The support for running tests for folders is already present in test runners, just package.json needs to be slightly tweaked.

@karthiknadig

Copy link
Copy Markdown
Member

@vladimir-kotikov Thanks for the PR. Can you create a issue for this, and add a news item so you get the credit for this fix.

@kimadeline

Copy link
Copy Markdown

Hi @vladimir-kotikov, thank you for your contribution! Could you also attach some screenshots or gifs of the feature you added?

@vladimir-kotikov

Copy link
Copy Markdown
Author

Created discussion to track this: #15869
Also here's a short video to demonstrate how it works

tests_folder

@karthiknadig @kimadeline

@kimadeline kimadeline added the skip package*.json package.json and package-lock.json don't both need updating label Apr 6, 2021
@karthiknadig karthiknadig added the skip tests Updates to tests unnecessary label Apr 6, 2021
Comment thread news/1 Enhancements/15862.md Outdated

@karthiknadig karthiknadig left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just one comment to ensure you get the credit for this.

Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
@karthiknadig karthiknadig merged commit 75c321c into microsoft:main Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip package*.json package.json and package-lock.json don't both need updating skip tests Updates to tests unnecessary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants