Merge pull request #1731 from jdi-testing/issue_1588-escaping-special…

…-text-locator-characters

Issue 1588: escaping special text locator characters (addional fix)

manifest.json

@@ -3,7 +3,7 @@
   "name": "JDN — Page Object Generator",
   "description": "JDN – helps Test Automation Engineer to create Page Objects in the test automation framework and speed up test development",
   "devtools_page": "index.html",
-  "version": "3.15.29",
+  "version": "3.15.30",
   "icons": {
     "128": "icon128.png"
   },

package.json

@@ -1,6 +1,6 @@
 {
   "name": "jdn-ai-chrome-extension",
-  "version": "3.15.29",
+  "version": "3.15.30",
   "description": "jdn-ai chrome extension",
   "scripts": {
     "start": "webpack --watch --env devenv",

src/features/locators/utils/escapeLocatorString.ts

@@ -7,6 +7,7 @@ export const fullEscapeLocatorString = (str: string = ''): string => {
     .replaceAll(/\\/g, '\\\\') // escape backslash
     .replaceAll(/\"/g, '\\"') // escape double quotes
     .replaceAll(/\'/g, "\\'") // escape single quotes
+    .replaceAll(/\b/g, '\\b') // escape backspaces
     .replaceAll(/\t/g, '\\t') // escape tabs
     .replaceAll(/\n/g, '\\n') // escape newlines
     .replaceAll(/\r/g, '\\r') // escape carriage returns

src/features/pageObjects/utils/pageObjectTemplate.ts

@@ -49,13 +49,13 @@ export const getPageObjectTemplateForJdi = (
   const { name: className, library } = pageObject;
 
   const locatorsCode = locators.map((locator) => {
-    const locatorEscaped = locator.locatorValue.output?.replace(/\\/g, '\\\\').replace(/"/g, '\\"');
+    const locatorOutput = locator.locatorValue.output;
     const { locatorType } = pageObject;
     const currentLocatorType = locator.locatorType || locatorType || LocatorType.xPath;
     return `    ${locator.annotationType}(${getLocatorPrefix(
       locator.annotationType,
       currentLocatorType,
-    )}"${locatorEscaped}")\n    public ${locator.type} ${locator.name};`;
+    )}"${locatorOutput}")\n    public ${locator.type} ${locator.name};`;
   });
 
   const hasFindByAnnotationType: boolean = hasAnnotationType(locators, AnnotationType.FindBy);