Implemented fix for quotemark rule (palantir#1790)

src/rules/quotemarkRule.ts

@@ -116,7 +116,12 @@ class QuotemarkWalker extends Lint.RuleWalker {
                     ? Rule.SINGLE_QUOTE_FAILURE
                     : Rule.DOUBLE_QUOTE_FAILURE;
 
-                this.addFailure(this.createFailure(position, width, failureMessage));
+                const newText = expectedQuoteMark
+                    + text.slice(1, -1).replace(new RegExp(expectedQuoteMark, "g"), `\\${expectedQuoteMark}`)
+                    + expectedQuoteMark;
+
+                const fix = new Lint.Fix(Rule.metadata.ruleName, [ new Lint.Replacement(position, width, newText) ]);
+                this.addFailure(this.createFailure(position, width, failureMessage, fix));
             }
         }
 

test/rules/quotemark/double-avoid-escape/test.ts.fix

@@ -0,0 +1,4 @@
+var single = "single";
+    var doublee = "married";
+var singleWithinDouble = "'singleWithinDouble'";
+var doubleWithinSingle = '"doubleWithinSingle"';

test/rules/quotemark/double/test.ts.fix

@@ -0,0 +1,4 @@
+var single = "single";
+    var doublee = "married";
+var singleWithinDouble = "'singleWithinDouble'";
+var doubleWithinSingle = "\"doubleWithinSingle\"";

test/rules/quotemark/jsx-double/test.tsx.fix

@@ -0,0 +1,4 @@
+import * as React from 'react';
+
+export const a = (
+    <div className="class" id="id" data-a={'a' + 'b'}></div>

test/rules/quotemark/jsx-single/test.tsx.fix

@@ -0,0 +1,4 @@
+import * as React from "react";
+
+export const a = (
+    <div className='class' id='id' data-a={"a" + "b"}></div>

test/rules/quotemark/single-avoid-escape/test.ts.fix

@@ -0,0 +1,4 @@
+var single = 'single';
+    var doublee = 'married';
+var singleWithinDouble = "'singleWithinDouble'";
+var doubleWithinSingle = '"doubleWithinSingle"';

test/rules/quotemark/single/test.ts.fix

@@ -0,0 +1,4 @@
+var single = 'single';
+    var doublee = 'married';
+var singleWithinDouble = '\'singleWithinDouble\'';
+var doubleWithinSingle = '"doubleWithinSingle"';