[dotnet] Use selenium manager in tests (#11871)

nvborisenko · diemol · web-flow · commit 5055da92b764 · 2023-04-19T13:51:07.000+02:00
* Copy manager binaries to tests output * Use selenium manager implicitly to execute tests * Trigger tests again * Revert "Trigger tests again" This reverts commit 6dde845. * Fix flaky ShouldBeAbleToUploadTheSameFileTwice tests * Remove preinstalled drivers * Remove all known pre-installed drivers * Damn powershell * Print verbose messages while deleting drivers * Return back globalization namespace --------- Co-authored-by: Diego Molina <diemol@users.noreply.github.com>
diff --git a/.github/workflows/ci-dotnet.yml b/.github/workflows/ci-dotnet.yml
@@ -31,10 +31,8 @@ jobs:
         include:
           - driver: Chrome
             browser: chrome
-            location: C:\SeleniumWebDrivers\ChromeDriver
           - driver: Firefox
             browser: chrome
-            location: C:\SeleniumWebDrivers\GeckoDriver
     with:
       name: Browser Tests (${{ matrix.driver }}, ${{ matrix.framework }})
       browser: ${{ matrix.browser }}
@@ -44,6 +42,8 @@ jobs:
       run: |
         bazel build //dotnet/test/common:${{ matrix.browser }}
         $env:ACTIVE_DRIVER_CONFIG = '${{ matrix.driver }}'
-        $env:DRIVER_SERVICE_LOCATION = '${{ matrix.location }}'
+        rm "$env:ChromeWebDriver" -r -v
+        rm "$env:EdgeWebDriver" -r -v
+        rm "$env:GeckoWebDriver" -r -v
         cd dotnet
         dotnet test test/common/WebDriver.Common.Tests.csproj --framework ${{ matrix.framework }}
diff --git a/dotnet/src/webdriver/SeleniumManager.cs b/dotnet/src/webdriver/SeleniumManager.cs
@@ -16,12 +16,13 @@
 // limitations under the License.
 // </copyright>
 
+using Newtonsoft.Json;
+using OpenQA.Selenium.Internal;
 using System;
 using System.Collections.Generic;
 using System.Diagnostics;
-using Newtonsoft.Json;
-using OpenQA.Selenium.Internal;
 using System.Globalization;
+using System.Reflection;
 
 #if !NET45 && !NET46 && !NET47
 using System.Runtime.InteropServices;
@@ -37,6 +38,7 @@ namespace OpenQA.Selenium
     /// </summary>
     public static class SeleniumManager
     {
+        private static string basePath = System.IO.Path.GetDirectoryName(Assembly.GetExecutingAssembly().Location);
         private static string binary;
 
         /// <summary>
@@ -140,7 +142,7 @@ private static string Binary
         private static string RunCommand(string fileName, string arguments)
         {
             Process process = new Process();
-            process.StartInfo.FileName = fileName;
+            process.StartInfo.FileName = $"{basePath}/{fileName}";
             process.StartInfo.Arguments = arguments;
             process.StartInfo.UseShellExecute = false;
             process.StartInfo.RedirectStandardOutput = true;
diff --git a/dotnet/src/webdriver/build/Selenium.WebDriver.targets b/dotnet/src/webdriver/build/Selenium.WebDriver.targets
@@ -1,20 +1,24 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
 
+  <PropertyGroup>
+    <Selenium_Manager_BinariesRootPath Condition="$(Selenium_Manager_BinariesRootPath) == ''">$(MSBuildThisFileDirectory)..\manager</Selenium_Manager_BinariesRootPath>
+  </PropertyGroup>
+
   <ItemGroup>
-    <None Include="$(MSBuildThisFileDirectory)..\manager\linux\selenium-manager">
+    <None Include="$(Selenium_Manager_BinariesRootPath)\linux\selenium-manager">
       <Link>selenium-manager\linux\%(Filename)%(Extension)</Link>
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
       <Visible>False</Visible>
     </None>
 
-    <None Include="$(MSBuildThisFileDirectory)..\manager\macos\selenium-manager">
+    <None Include="$(Selenium_Manager_BinariesRootPath)\macos\selenium-manager">
       <Link>selenium-manager\macos\%(Filename)%(Extension)</Link>
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
       <Visible>False</Visible>
     </None>
 
-    <None Include="$(MSBuildThisFileDirectory)..\manager\windows\selenium-manager.exe">
+    <None Include="$(Selenium_Manager_BinariesRootPath)\windows\selenium-manager.exe">
       <Link>selenium-manager\windows\%(Filename)%(Extension)</Link>
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
       <Visible>False</Visible>
diff --git a/dotnet/test/common/FormHandlingTests.cs b/dotnet/test/common/FormHandlingTests.cs
@@ -218,6 +218,9 @@ public void ShouldBeAbleToUploadTheSameFileTwice()
 
                 uploadElement.SendKeys(inputFile.FullName);
                 uploadElement.Submit();
+
+                // Explicitly wait next page to be loaded, Firefox is not handling elements submitting
+                WaitFor(() => driver.Url.EndsWith("resultPage.html"), "We are not redirected to the resultPage after submitting web element");
             }
 
             inputFile.Delete();
diff --git a/dotnet/test/common/WebDriver.Common.Tests.csproj b/dotnet/test/common/WebDriver.Common.Tests.csproj
@@ -63,4 +63,10 @@
     <Exec Command="$(ProjectDir)WebDriver.Common.Tests.csproj.prebuild.sh $(ProjectDir)" />
   </Target>
 
+  <!-- It is automatically imported when Selenium.WebDriver consumed via nuget package -->
+  <PropertyGroup>
+    <Selenium_Manager_BinariesRootPath>..\..\..\common\manager</Selenium_Manager_BinariesRootPath>
+  </PropertyGroup>
+  <Import Project="..\..\src\webdriver\build\Selenium.WebDriver.targets" />
+
 </Project>

Original file line number	Diff line number	Diff line change
`@@ -218,6 +218,9 @@ public void ShouldBeAbleToUploadTheSameFileTwice()`
`218`	`218`
`219`	`219`	`uploadElement.SendKeys(inputFile.FullName);`
`220`	`220`	`uploadElement.Submit();`
	`221`	`+`
	`222`	`+ // Explicitly wait next page to be loaded, Firefox is not handling elements submitting`
	`223`	`+ WaitFor(() => driver.Url.EndsWith("resultPage.html"), "We are not redirected to the resultPage after submitting web element");`
`221`	`224`	`}`
`222`	`225`
`223`	`226`	`inputFile.Delete();`