fliped around logic for Execute and ExecuteAsync, Execute now passes …

…through to ExecuteAsync
kelmelzer · Jul 19, 2024 · 446d42d · 446d42d
1 parent 5317408
commit 446d42d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/Appium.Net/Appium.Net.csproj b/src/Appium.Net/Appium.Net.csproj
@@ -60,7 +60,7 @@
     </PackageReference>
     <PackageReference Include="Selenium.Support" Version="4.23.0" />
     <PackageReference Include="Selenium.WebDriver" Version="4.23.0" />
-    <PackageReference Include="System.Drawing.Common" Version="8.0.6" />
+    <PackageReference Include="System.Drawing.Common" Version="8.0.7" />
   </ItemGroup>
   <ItemGroup>
     <Folder Include="Properties\" />

diff --git a/src/Appium.Net/Appium/Service/AppiumCommandExecutor.cs b/src/Appium.Net/Appium/Service/AppiumCommandExecutor.cs
@@ -54,14 +54,16 @@ internal AppiumCommandExecutor(AppiumLocalService service, TimeSpan timeForTheSe
             ClientConfig = clientConfig;
         }
 
-        public Response Execute(Command commandToExecute)
+        public Response Execute(Command commandToExecute) => this.ExecuteAsync(commandToExecute).GetAwaiter().GetResult();
+
+        public async Task<Response> ExecuteAsync(Command commandToExecute)
         {
             Response result = null;
 
             try
             {
                 bool newSession = HandleNewSessionCommand(commandToExecute);
-                result = RealExecutor.Execute(commandToExecute);
+                result = await RealExecutor.ExecuteAsync(commandToExecute);
                 if (newSession)
                 {
                     RealExecutor = UpdateExecutor(result, RealExecutor);
@@ -80,8 +82,6 @@ public Response Execute(Command commandToExecute)
             }
         }
 
-        public Task<Response> ExecuteAsync(Command commandToExecute) => Task.Run(() => this.Execute(commandToExecute));
-
         /// <summary>
         /// Handles a new session command, starts the service, and modifies the HTTP request header if necessary.
         /// </summary>