Skip to content

[dotnet] [test] Remove/move browser specific test projects#17281

Merged
nvborisenko merged 3 commits intoSeleniumHQ:trunkfrom
nvborisenko:dotnet-remove-specific-projects
Mar 31, 2026
Merged

[dotnet] [test] Remove/move browser specific test projects#17281
nvborisenko merged 3 commits intoSeleniumHQ:trunkfrom
nvborisenko:dotnet-remove-specific-projects

Conversation

@nvborisenko
Copy link
Copy Markdown
Member

Now it is part of the main test suite.

🔗 Related Issues

Contributes to #15536

💥 What does this PR do?

This pull request removes all browser-specific test projects and related files for Chrome, Edge, Firefox, IE, and Safari from the .NET test suite. The changes consolidate the test structure by eliminating redundant or unnecessary test projects, configuration files, and setup/teardown logic for these browsers.

The most important changes are:

Removal of browser-specific test projects and configuration:

  • Deleted the entire test projects for Chrome, Edge, Firefox, IE, and Safari, including their .csproj files, NUnit project files, and browser-specific test classes such as ChromeSpecificTests, EdgeSpecificTests, and SafariSpecificTests. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13]

Cleanup of assembly setup/teardown code:

  • Removed all AssemblyTeardown.cs files that handled test suite setup and teardown for each browser-specific test project.

Project and solution file updates:

  • Updated .bazelignore and Selenium.slnx to remove references to the deleted browser-specific test directories and projects. [1] [2]

💡 Additional Considerations

Just moved tests.

🔄 Types of changes

  • Cleanup (formatting, renaming)

@selenium-ci selenium-ci added C-dotnet .NET Bindings B-build Includes scripting, bazel and CI integrations labels Mar 30, 2026
@qodo-code-review
Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Consolidate .NET test suite by removing browser-specific test projects

🧪 Tests

Grey Divider

Walkthroughs

Description
• Consolidate .NET test suite by removing browser-specific test projects
• Move Firefox and IE tests into main webdriver test project
• Delete Chrome, Edge, Safari, Firefox, and IE test project files
• Update solution file and build configuration to reflect changes

Grey Divider

File Changes

1. dotnet/test/chrome/AssemblyTeardown.cs 🧪 Tests +0/-41

Remove Chrome test project assembly teardown

dotnet/test/chrome/AssemblyTeardown.cs


2. dotnet/test/chrome/ChromeSpecificTests.cs 🧪 Tests +0/-28

Remove Chrome-specific test class

dotnet/test/chrome/ChromeSpecificTests.cs


3. dotnet/test/edge/AssemblyTeardown.cs 🧪 Tests +0/-41

Remove Edge test project assembly teardown

dotnet/test/edge/AssemblyTeardown.cs


View more (23)
4. dotnet/test/edge/EdgeSpecificTests.cs 🧪 Tests +0/-28

Remove Edge-specific test class

dotnet/test/edge/EdgeSpecificTests.cs


5. dotnet/test/firefox/AssemblyTeardown.cs 🧪 Tests +0/-41

Remove Firefox test project assembly teardown

dotnet/test/firefox/AssemblyTeardown.cs


6. dotnet/test/ie/AssemblyTeardown.cs 🧪 Tests +0/-41

Remove IE test project assembly teardown

dotnet/test/ie/AssemblyTeardown.cs


7. dotnet/test/safari/AssemblyTeardown.cs 🧪 Tests +0/-41

Remove Safari test project assembly teardown

dotnet/test/safari/AssemblyTeardown.cs


8. dotnet/test/safari/SafariSpecificTests.cs 🧪 Tests +0/-28

Remove Safari-specific test class

dotnet/test/safari/SafariSpecificTests.cs


9. dotnet/test/webdriver/Firefox/FirefoxDriverServiceTests.cs 🧪 Tests +2/-5

Update namespace and remove unused imports

dotnet/test/webdriver/Firefox/FirefoxDriverServiceTests.cs


10. dotnet/test/webdriver/Firefox/FirefoxDriverTests.cs 🧪 Tests +2/-7

Update namespace and clean up imports

dotnet/test/webdriver/Firefox/FirefoxDriverTests.cs


11. dotnet/test/webdriver/Firefox/FirefoxProfileManagerTests.cs 🧪 Tests +2/-2

Update namespace and remove unused imports

dotnet/test/webdriver/Firefox/FirefoxProfileManagerTests.cs


12. dotnet/test/webdriver/Firefox/FirefoxProfileTests.cs 🧪 Tests +2/-3

Update namespace and remove unused imports

dotnet/test/webdriver/Firefox/FirefoxProfileTests.cs


13. dotnet/test/webdriver/IE/IeSpecificTests.cs 🧪 Tests +2/-6

Update namespace and clean up imports

dotnet/test/webdriver/IE/IeSpecificTests.cs


14. .bazelignore ⚙️ Configuration changes +0/-10

Remove browser-specific test project directories

.bazelignore


15. dotnet/Selenium.slnx ⚙️ Configuration changes +0/-5

Remove browser-specific test projects from solution

dotnet/Selenium.slnx


16. dotnet/test/chrome/Selenium.WebDriver.Chrome.Tests.csproj 🧪 Tests +0/-18

Delete Chrome test project file

dotnet/test/chrome/Selenium.WebDriver.Chrome.Tests.csproj


17. dotnet/test/chrome/Selenium.WebDriver.Chrome.Tests.nunit 🧪 Tests +0/-7

Delete Chrome NUnit configuration file

dotnet/test/chrome/Selenium.WebDriver.Chrome.Tests.nunit


18. dotnet/test/edge/Selenium.WebDriver.Edge.Tests.csproj 🧪 Tests +0/-23

Delete Edge test project file

dotnet/test/edge/Selenium.WebDriver.Edge.Tests.csproj


19. dotnet/test/edge/Selenium.WebDriver.Edge.Tests.nunit 🧪 Tests +0/-7

Delete Edge NUnit configuration file

dotnet/test/edge/Selenium.WebDriver.Edge.Tests.nunit


20. dotnet/test/firefox/Selenium.WebDriver.Firefox.Tests.csproj 🧪 Tests +0/-24

Delete Firefox test project file

dotnet/test/firefox/Selenium.WebDriver.Firefox.Tests.csproj


21. dotnet/test/firefox/Selenium.WebDriver.Firefox.Tests.nunit 🧪 Tests +0/-7

Delete Firefox NUnit configuration file

dotnet/test/firefox/Selenium.WebDriver.Firefox.Tests.nunit


22. dotnet/test/ie/Selenium.WebDriver.IE.Tests.csproj 🧪 Tests +0/-25

Delete IE test project file

dotnet/test/ie/Selenium.WebDriver.IE.Tests.csproj


23. dotnet/test/ie/Selenium.WebDriver.IE.Tests.nunit 🧪 Tests +0/-7

Delete IE NUnit configuration file

dotnet/test/ie/Selenium.WebDriver.IE.Tests.nunit


24. dotnet/test/safari/Selenium.WebDriver.Safari.Tests.csproj 🧪 Tests +0/-24

Delete Safari test project file

dotnet/test/safari/Selenium.WebDriver.Safari.Tests.csproj


25. dotnet/test/safari/Selenium.WebDriver.Safari.Tests.nunit 🧪 Tests +0/-7

Delete Safari NUnit configuration file

dotnet/test/safari/Selenium.WebDriver.Safari.Tests.nunit


26. dotnet/test/webdriver/Selenium.WebDriver.Tests.csproj 🧪 Tests +4/-0

Add IE folder to main test project

dotnet/test/webdriver/Selenium.WebDriver.Tests.csproj


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown
Contributor

qodo-code-review bot commented Mar 30, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

@nvborisenko nvborisenko merged commit 3753c6b into SeleniumHQ:trunk Mar 31, 2026
18 of 19 checks passed
@nvborisenko nvborisenko deleted the dotnet-remove-specific-projects branch March 31, 2026 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B-build Includes scripting, bazel and CI integrations C-dotnet .NET Bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants