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

Test adapter loading strategy #3380

Merged
merged 10 commits into from
Feb 18, 2022
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixed typos in code.
  • Loading branch information
Haplois committed Feb 16, 2022
commit 12c12b0928debe191efdfbcb6b4ddb3702e39e4b
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,12 @@ namespace Microsoft.VisualStudio.TestPlatform.ObjectModel
internal enum TestAdapterLoadingStrategy
{
/// <summary>
/// A strategy not defined, Test Platfrom will load adapters normally.
/// A strategy not defined, Test Platform will load adapters normally.
/// </summary>
Default = 0b0000_0000_0000_0000,

/// <summary>
/// Test Plarform will only load adapters specified by /TestAdapterPath (or RunConfiguration.TestAdaptersPaths node).
/// Test Platform will only load adapters specified by /TestAdapterPath (or RunConfiguration.TestAdaptersPaths node).
/// If a specific adapter path is provided, adapter will be loaded; if a directory path is provided adapters directly in that folder will be loaded.
/// If no adapter path is specified, test run will fail.
/// This will imply /InIsolation switch and force the tests to be run in an isolated process.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -162,9 +162,7 @@ public void Initialize(string argument)
}

if (string.IsNullOrWhiteSpace(argument))
{


{
InitializeDefaultStrategy();
return;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ namespace Microsoft.VisualStudio.TestPlatform.CommandLine.Processors;
/// --Port
///
/// Help
/// -Help
/// --Help
/// </summary>
internal enum HelpContentPriority
{
Expand Down
8 changes: 4 additions & 4 deletions src/vstest.console/Resources/Resources.resx
Original file line number Diff line number Diff line change
Expand Up @@ -564,14 +564,14 @@
<value>--TestAdapterLoadingStrategy|/TestAdapterLoadingStrategy:&lt;strategy&gt;
This affects adapter loading behavior.

Currently supported behaviours:
- Explicit: Test Plarform will only load adapters specified by /TestAdapterPath (or RunConfiguration.TestAdaptersPaths node).
Currently supported behaviors:
- Explicit: Test Platform will only load adapters specified by /TestAdapterPath (or RunConfiguration.TestAdaptersPaths node).
If a specific adapter path is provided, adapter will be loaded; if a directory path is provided adapters directly in that folder will be loaded, unless Recursive option is also specified.
If no adapter path is specified, test run will fail.
This will imply /InIsolation switch and force the tests to be run in an isolated process.

- Default: Test Platfrom will load adapters is if this argument has not beed specified.
It will pick up extensions from next to source, provided aditional adapter paths and from the default directory.
- Default: Test Platform will load adapters is if this argument has not been specified.
It will pick up extensions from next to source, provided additional adapter paths and from the default directory.

- DefaultRuntimeProviders: Load default runtime providers shipped with Test Platform.
If this is not specified when "Explicit" option is set, a test host provider need to be specified explicitly.
Expand Down