Repository navigation
[🚀 Feature]: [dotnet] Revisit unit tests infrastructure #15536
Description
Feature and motivation
Current situation is:
Usage example
Concerns:
- It could be one test project because Chrome/Edge/Firefox drivers live in one project supported by Selenium (like built-in classes).
- Default namespace in test project is
OpenQA.Seleniumwhen usually it should beOpenQA.Selenium.Tests. I don't understand why it is unusual. - Avoid any
staticcontext (EnvironmentManagerclass) - it will allow to look at parallelization. My computer has 6/12 cores, I want to utilize all of them when running Selenium unit tests. - Provide a possibility to override driver options, like required in [dotnet] [bidi] Support WebExtension module #15850
Reacted by Michael Render
Activity
RenderMichael commentedon Apr 2, 2025
I don't have much of the historical context surrounding the design decisions of the test projects. There are some clear reasons I see for a heavily configuration-based
EnvironmentManager. For example, theEnvironmentManager.Instanceproperty is accessible in theAssemblyFixturetypes.At the same time, I see some room for optimization.
Notably, the above
AssemblyFixture's one-time setup starts the web server and the remote server, but not the driver. The one-time teardown closes whatever driver may be opened, likely as a precaution. I fully believe we can move the "current driver" management to the test-side.Rather than using
appconfig.json(and all theRunfiles.Create()andrunfiles.Rlocationceremony surrounding that), we can giveDriverTestFixturea method like so:I can experiment with some potential solutions.
nvborisenko commentedon Apr 14, 2025
Historical decision is very clear: reusing browser across 100 tests is faster than initiating new clean browser per test. I would agree at that long time ago, as a quick approach to run tests. But now I strongly disagree. Reusing shared context breaks main principle of testing: isolation. Tests should not be dependent on context, on browser state in our case. (here I could put a link to ignored test which should not be ignored). This is why we currently have
NeedsFreshDriverAttribute, meaning we have to keep in mind the state of browser, shared between tests. And it is bad in my opinion.Tests isolation has more priority than tests execution duration. Let's say that current total duration is 20mins, introducing tests isolation will increase total duration to 40 mins. Not a problem at all, I don't see big diff between 40 mins and 20 mins. Insight:
BiDitests are executed in parallel fully isolated: 100 tests in 1.5 minutes on my machine.So my recommendation is also very clear: be isolated and parallelizable. Good news it is achievable.
github-actions commentedon Feb 7, 2026
This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 14 days.
nvborisenko commentedon Jun 28, 2026
Next big step is to move NUnit infrastructure into dedicated reusable project.
nvborisenko commentedon Oct 1, 2026
NUnit infra is in dedicated project (reusable), test fixtures are parallelizable. I am confident now how it works - in IDE just run test - it runs fast.
@YevgeniyShunevych @RenderMichael any leftovers you wanted to bring before closing this chapter (issue)?
YevgeniyShunevych commentedon Oct 1, 2026
I have nothing to mention.
nvborisenko commentedon Oct 1, 2026
Sold. Thank you all for good collaboration. Next improvements will be tweak-tuning. Closing this chapter as completed.