Merge pull request #1078 from yileicn/master

modify LocateElement logger level

Author: Oceania2018

src/Infrastructure/BotSharp.Abstraction/Browsing/Models/ElementLocatingArgs.cs

@@ -30,7 +30,7 @@ public class ElementLocatingArgs
 
     public bool FailIfMultiple { get; set; }
     [JsonPropertyName("ignore_if_not_found")]
-    public bool IgnoreIfNotFound { get; set; }
+    public bool IgnoreIfNotFound { get; set; } = true;
 
     /// <summary>
     /// Draw outline around the element

src/Plugins/BotSharp.Plugin.WebDriver/Drivers/PlaywrightDriver/PlaywrightWebDriver.LocateElement.cs

@@ -23,30 +23,35 @@ public async Task<BrowserActionResult> LocateElement(MessageInfo message, Elemen
         }
         ILocator locator = page.Locator("body");
         int count = 0;
+        var keyword = string.Empty;
 
         // check if selector is specified
         if (location.Selector != null)
         {
+            keyword = location.Selector;
             locator = locator.Locator(location.Selector);
             count = await locator.CountAsync();
         }
 
         if (location.Tag != null)
         {
+            keyword = location.Tag;
             locator = page.Locator(location.Tag);
             count = await locator.CountAsync();
         }
 
         // try attribute
         if (!string.IsNullOrEmpty(location.AttributeName))
         {
-            locator = locator.Locator($"[{location.AttributeName}='{location.AttributeValue}']");
+            keyword = $"[{location.AttributeName}='{location.AttributeValue}']";
+            locator = locator.Locator(keyword);
             count = await locator.CountAsync();
         }
 
         // Retrieve the page raw html and infer the element path
         if (!string.IsNullOrEmpty(location.Text))
         {
+            keyword = location.Text;
             var text = location.Text.Replace("(", "\\(").Replace(")", "\\)");
             var regexExpression = location.MatchRule.ToLower() switch
             {
@@ -69,6 +74,7 @@ public async Task<BrowserActionResult> LocateElement(MessageInfo message, Elemen
 
         if (location.Index >= 0)
         {
+            keyword = $"Index:{location.Index}";
             locator = locator.Nth(location.Index);
             count = await locator.CountAsync();
         }
@@ -77,12 +83,12 @@ public async Task<BrowserActionResult> LocateElement(MessageInfo message, Elemen
         {
             if (location.IgnoreIfNotFound)
             {
-                result.Message = $"Can't locate element by keyword {location.Text} and Ignored";
+                result.Message = $"Can't locate element by keyword {keyword} and Ignored";
                 _logger.LogWarning(result.Message);
             }
             else
             {
-                result.Message = $"Can't locate element by keyword {location.Text}";
+                result.Message = $"Can't locate element by keyword {keyword}";
                 _logger.LogError(result.Message);
             }