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

Pre-start testhosts #3666

Merged
merged 41 commits into from
Feb 23, 2023
Merged
Changes from 1 commit
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
257d214
Pre-start testhosts
nohwnd May 23, 2022
4091e46
Fix run logic
nohwnd May 23, 2022
a458d23
Fix scheduling for parallel run
nohwnd May 24, 2022
afe9408
Pre-start use first started
nohwnd May 25, 2022
95c771e
Fix build
Jul 28, 2022
ccf90ac
Merge branch 'main' into dev/copoiena/pre-start-testhost
Jul 28, 2022
8fe0e2a
Fixed public API
Jul 28, 2022
fc8b79e
Fixed build
Jul 28, 2022
269a23e
Fixed build
Jul 28, 2022
4564baa
Fixed tests
Aug 4, 2022
73051d6
Merge branch 'main' into dev/copoiena/pre-start-testhost
Aug 4, 2022
224e105
Merged main
Jan 5, 2023
6648299
Quick & dirty test fix
Jan 9, 2023
469e047
Added comment
Jan 10, 2023
3552b3c
Reverted condition order
Jan 11, 2023
6769d82
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/ProxyExecuti…
cvpoienaru Jan 11, 2023
41a9c5d
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/ProxyExecuti…
cvpoienaru Jan 11, 2023
42fd8f2
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/Parallel/Par…
cvpoienaru Jan 11, 2023
4522650
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/Parallel/Par…
cvpoienaru Jan 11, 2023
920c6fb
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/Parallel/Par…
cvpoienaru Jan 11, 2023
a502498
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/ProxyExecuti…
cvpoienaru Jan 11, 2023
1233c92
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/ProxyDiscove…
cvpoienaru Jan 11, 2023
3e6b114
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/Parallel/Par…
cvpoienaru Jan 11, 2023
8e05fda
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/Parallel/Par…
cvpoienaru Jan 11, 2023
b598d0e
Code review fixes
Jan 12, 2023
cbea5d9
Used Task.Delay in test
Jan 19, 2023
fccc469
Fixed tests
Jan 19, 2023
d02345b
Replaced Debug.Write with EqtTrace.Verbose
Jan 19, 2023
f017807
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/Parallel/Par…
cvpoienaru Jan 19, 2023
eb88b48
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/Parallel/Par…
cvpoienaru Jan 19, 2023
26c721f
Update src/Microsoft.TestPlatform.CrossPlatEngine/Client/Parallel/Par…
cvpoienaru Jan 19, 2023
2ecc535
More Debug.WriteLine fixes
Jan 19, 2023
95a93e9
Merge branch 'pre-start-testhost' of https://github.com/nohwnd/vstest…
Jan 19, 2023
1548939
Removed log
Jan 19, 2023
a0cf1b7
Fixed tests
Feb 1, 2023
45dab9f
Merge branch 'main' into dev/copoiena/pre-start-testhost
Feb 1, 2023
03ee17f
Merge branch 'main' into pre-start-testhost
nohwnd Feb 14, 2023
844b82e
Fix acceptance tests
nohwnd Feb 14, 2023
d0c083d
Added comment about slot synchronization
Feb 22, 2023
b371799
Merge branch 'pre-start-testhost' of https://github.com/nohwnd/vstest…
Feb 22, 2023
6f720f6
Merge branch 'main' into dev/copoiena/pre-start-testhost
Feb 22, 2023
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
Removed log
  • Loading branch information
Codrin Poienaru committed Jan 19, 2023
commit 15489396b77b8b19b8b1a28869245f9beb48869d
Original file line number Diff line number Diff line change
Expand Up @@ -253,23 +253,16 @@ static DiscoveryCriteria NewDiscoveryCriteriaFromSourceAndSettings(IEnumerable<s
/// <param name="ProxyDiscoveryManager">Proxy discovery manager instance.</param>
private Task InitializeDiscoverTestsOnConcurrentManager(IProxyDiscoveryManager proxyDiscoveryManager, ITestDiscoveryEventsHandler2 eventHandler, DiscoveryCriteria discoveryCriteria)
{
try
// Kick off another discovery task for the next source
return Task.Run(() =>
{
// Kick off another discovery task for the next source
return Task.Run(() =>
{
EqtTrace.Verbose("ProxyParallelDiscoveryManager.InitializeDiscoverTestsOnConcurrentManager: Discovery preparation started.");
EqtTrace.Verbose("ProxyParallelDiscoveryManager.InitializeDiscoverTestsOnConcurrentManager: Discovery preparation started.");

proxyDiscoveryManager.Initialize(_skipDefaultAdapters);
proxyDiscoveryManager.InitializeDiscovery(discoveryCriteria, eventHandler, _skipDefaultAdapters);
proxyDiscoveryManager.Initialize(_skipDefaultAdapters);
proxyDiscoveryManager.InitializeDiscovery(discoveryCriteria, eventHandler, _skipDefaultAdapters);

EqtTrace.Verbose($"ProxyParallelDiscoveryManager.InitializeDiscoverTestsOnConcurrentManager: Init only: {discoveryCriteria.Sources.Single()}");
});
}
finally
{
EqtTrace.Verbose("ProxyParallelDiscoveryManager.InitializeDiscoverTestsOnConcurrentManager: No sources available for discovery.");
}
EqtTrace.Verbose($"ProxyParallelDiscoveryManager.InitializeDiscoverTestsOnConcurrentManager: Init only: {discoveryCriteria.Sources.Single()}");
});
}

/// <summary>
Expand Down