Merge pull request #56 from Berkmann18/dev

fix: fixes special characters

Author: Berkmann18

index.js

@@ -57,6 +57,8 @@ const ops = (err) => ['+', '-', '*', '/', '>', '<', '~', '|', '&', '^'].includes
 
 const extraBrackets = (err) => err.found === '}';
 
+const specialChar = (err) => err.found === '"';
+
 const runFixer = ({ verbose, lines, start, err }) => {
   /* eslint-disable security/detect-object-injection */
   let fixedData = [...lines];
@@ -84,6 +86,8 @@ const runFixer = ({ verbose, lines, start, err }) => {
     fixedData = fixer.fixOpConcat({ start, fixedData, verbose });
   } else if (extraBrackets(err)) {
     fixedData = fixer.fixExtraCurlyBrackets({ start, fixedData, verbose });
+  } else if (specialChar(err)) {
+    fixedData = fixer.fixSpecialChar({ start, fixedData, verbose });
   } else throw new Error(`Unsupported issue: ${err.message} (please open an issue at the repo)`);
   return fixedData;
 };

src/__tests__/index.js

@@ -411,3 +411,23 @@ describe('multi rounds', () => {
     });
   });
 });
+
+describe('special chars', () => {
+  it('tab', () => {
+    shouldHaveChanged('tab', {
+      Test: '\t'
+    });
+  });
+
+  it('formatted tab', () => {
+    shouldHaveChanged('tabs', {
+      Test: '\t'
+    });
+  });
+
+  it('new line', () => {
+    shouldHaveChanged('newLines', {
+      Broken: '\n'
+    });
+  });
+});

src/fixer.js

@@ -156,7 +156,7 @@ const fixOpConcat = ({ start, fixedData, verbose }) => {
   return fixedData;
 };
 
-function fixExtraCurlyBrackets({ start, fixedData, verbose }) {
+const fixExtraCurlyBrackets = ({ start, fixedData, verbose }) => {
   if (verbose) psw(chalk.magenta('Extra curly brackets'));
 
   const targetLine = start.line - 1;
@@ -175,7 +175,30 @@ function fixExtraCurlyBrackets({ start, fixedData, verbose }) {
 
   fixedData[targetLine] = fixedLine;
   return fixedData;
-}
+};
+
+const fixSpecialChar = ({ start, fixedData, verbose }) => {
+  if (verbose) psw(chalk.magenta('Special character'));
+  const targetLine = start.line - 1;
+  const brokenLine = fixedData[targetLine];
+
+  let fixedLine = brokenLine
+    .replace(/\f/g, '\\f')
+    .replace(/\n/g, '\\n')
+    .replace(/\r/g, '\\r')
+    .replace(/\t/g, '\\t');
+
+  if (brokenLine.endsWith('"') && brokenLine[start.column] === undefined) {
+    if (verbose) psw(chalk.magenta('New line'));
+    const removedIndex = targetLine + 1;
+    const continuation = fixedData[removedIndex];
+    fixedLine = `${brokenLine}\\n${continuation}`;
+    fixedData.splice(removedIndex, 1);
+  }
+
+  fixedData[targetLine] = fixedLine;
+  return fixedData;
+};
 
 module.exports = {
   fixExtraChar,
@@ -186,5 +209,6 @@ module.exports = {
   fixCurlyBrackets,
   fixComment,
   fixOpConcat,
-  fixExtraCurlyBrackets
+  fixExtraCurlyBrackets,
+  fixSpecialChar
 };

test/samples/newLines.json

@@ -0,0 +1,2 @@
+{"Broken": "
+"}

test/samples/tab.json

@@ -0,0 +1 @@
+{"Test": "	"}

test/samples/tabs.json

@@ -0,0 +1,3 @@
+{
+  "Test": "	"
+}