remove hash from invalid locator

jdi-testing · Jul 27, 2023 · fd872bf · fd872bf
1 parent d2184a9
commit fd872bf
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/src/common/components/notification/useNotifications.tsx b/src/common/components/notification/useNotifications.tsx
@@ -56,7 +56,6 @@ export const useNotifications = (container?: HTMLElement | null) => {
     notification[type]({
       message: _message,
       description,
-      duration: 500,
       getContainer: () => container || document.body,
       placement: "bottom",
       bottom,

diff --git a/src/features/locators/locators.slice.ts b/src/features/locators/locators.slice.ts
@@ -23,6 +23,7 @@ import {
   LocatorCalculationPriority,
   JDNHash,
   LocatorValue,
+  LocatorValidationWarnings,
 } from "./types/locator.types";
 import { checkLocatorsValidityReducer } from "./reducers/checkLocatorValidity.thunk";
 import { LocatorType } from "../../common/types/common";
@@ -73,6 +74,8 @@ const locatorsSlice = createSlice({
         newValue.locator.xPath = locator;
       }
 
+      if (rest.message === LocatorValidationWarnings.NotFound) newValue.jdnHash = "";
+
       locatorsAdapter.upsertOne(state, newValue);
     },
     changeIdentificationStatus(state, { payload }: PayloadAction<IdentificationStatus>) {

diff --git a/src/pageServices/contentScripts/selectable.js b/src/pageServices/contentScripts/selectable.js
@@ -38,11 +38,13 @@ export const selectable = () => {
         });
       } else query = `[id='${payload.jdnHash}']`;
 
-      this.foreach(document.querySelectorAll(query), (element) => {
-        if (element.classList.contains(opt.selectedClass)) {
-          element.classList.remove(opt.selectedClass);
-        }
-      });
+      if (query.length) {
+        this.foreach(document.querySelectorAll(query), (element) => {
+          if (element.classList.contains(opt.selectedClass)) {
+            element.classList.remove(opt.selectedClass);
+          }
+        });
+      }
     };
 
     Selectables.prototype.setSelect = function (payload) {