Ignore Element key name change (#221)

* Added logic for ignore region * added test case * Added validation in tests * Possible concurrent thread fix 1 * Possible concurrent thread fix 2 * Possible concurrent thread fix 3 * Possible concurrent thread fix 4 * Possible concurrent thread fix 5 * Fix ignore region logic
percy · Jun 21, 2023 · 752a877 · 752a877
1 parent 3b78466
commit 752a877
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/main/java/io/percy/selenium/Percy.java b/src/main/java/io/percy/selenium/Percy.java
@@ -222,7 +222,8 @@ public void screenshot(String name, Map<String, Object> options) throws Unsuppor
 
         if (options.containsKey(ignoreElementKey)) {
             List<String> ignoreElementIds =  getElementIdFromElement((List<RemoteWebElement>) options.get(ignoreElementKey));
-            options.replace(ignoreElementKey, ignoreElementIds);
+            options.remove(ignoreElementKey);
+            options.put("ignore_region_elements", ignoreElementIds);
         }
 
         // Build a JSON object to POST back to the agent node process