fix: add double quotes for coping locator output & add semicolons in …

…Colorized Jdi String
jdi-testing · Dec 6, 2023 · aa11323 · aa11323
1 parent 1f1cd45
commit aa11323
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 20 deletions.
diff --git a/src/common/utils/copyToClipboard.ts b/src/common/utils/copyToClipboard.ts
@@ -1,21 +1,3 @@
-export const copyToClipboard = (value: string | string[]) => {
-  const clipboard = navigator.clipboard;
-  const valueString: string = valueToString(value);
-
-  if (!clipboard) {
-    console.error('Clipboard API is not supported in this browser.');
-    return;
-  }
-
-  // eslint-disable-next-line prettier/prettier
-  clipboard
-    .writeText(valueString)
-    .catch((error) => {
-    console.warn('Error copying text to clipboard:', error);
-    deprecatedCopyToClipboard(valueString);
-  });
-};
-
 const valueToString = (value: string | string[]) => {
   if (Array.isArray(value)) {
     return value.join('\n');
@@ -37,3 +19,21 @@ const deprecatedCopyToClipboard = (value: string) => {
     console.warn('Second Error copying text to clipboard:', err);
   }
 };
+
+export const copyToClipboard = (value: string | string[]) => {
+  const clipboard = navigator.clipboard;
+  const valueString: string = valueToString(value);
+
+  if (!clipboard) {
+    console.error('Clipboard API is not supported in this browser.');
+    return;
+  }
+
+  // eslint-disable-next-line prettier/prettier
+  clipboard
+    .writeText(valueString)
+    .catch((error) => {
+    console.warn('Error copying text to clipboard:', error);
+    deprecatedCopyToClipboard(valueString);
+  });
+};
diff --git a/src/features/locators/utils/locatorOutput.tsx b/src/features/locators/utils/locatorOutput.tsx
@@ -95,7 +95,7 @@ export const getLocatorString = (
   type: ElementLibrary | ElementClass,
   name: string,
 ): string => {
-  const locatorOutput = `'${locator.output}'`;
+  const locatorOutput = `"${locator.output}"`;
 
   return `${annotationType}(${getLocatorPrefix(annotationType, locatorType)}${locatorOutput})\npublic ${type} ${name};`;
 };
@@ -117,7 +117,7 @@ export const renderColorizedJdiString = (
       <br />
       <span className="jdn__locator_item-type">public</span>
       <span>&nbsp;{type}&nbsp;</span>
-      {name}
+      {name};
     </>
   );
 };