Fix #4514: [quotemark: backtick] Incorrectly flags string literals th…

…at must use single/double quotes (#4535)

* Fix #4514

* make comment more specific

* lint fixes

src/rules/quotemarkRule.ts

@@ -14,8 +14,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
-import { isNoSubstitutionTemplateLiteral, isSameLine, isStringLiteral } from "tsutils";
+import {
+    isImportDeclaration,
+    isNoSubstitutionTemplateLiteral,
+    isPropertyAssignment,
+    isSameLine,
+    isStringLiteral,
+} from "tsutils";
 import * as ts from "typescript";
 
 import * as Lint from "../index";
@@ -122,6 +127,14 @@ function walk(ctx: Lint.WalkContext<Options>) {
                     : options.quotemark;
             const actualQuotemark = sourceFile.text[node.end - 1];
 
+            // Don't use backticks instead of single/double quotes when it breaks TypeScript syntax.
+            if (
+                expectedQuotemark === "`" &&
+                (isImportDeclaration(node.parent) || isPropertyAssignment(node.parent))
+            ) {
+                return;
+            }
+
             if (actualQuotemark === expectedQuotemark) {
                 return;
             }

test/rules/quotemark/backtick/test.ts.fix

@@ -1,3 +1,5 @@
+import { Something } from "some-package"
+
 var single = `single`;
     var double = `married`;
 var singleWithinDouble = `'singleWithinDouble'`;
@@ -7,3 +9,5 @@ var tabNewlineWithinSingle = `tab\tNewline\nWithinSingle`;
 
 // "avoid-template" option is not set.
 `foo`;
+
+const object = { "kebab-case": 3 }

test/rules/quotemark/backtick/test.ts.lint

@@ -1,3 +1,5 @@
+import { Something } from "some-package"
+
 var single = 'single';
              ~~~~~~~~  [' should be `]
     var double = "married";
@@ -13,3 +15,5 @@ var tabNewlineWithinSingle = 'tab\tNewline\nWithinSingle';
 
 // "avoid-template" option is not set.
 `foo`;
+
+const object = { "kebab-case": 3 }