fix(39332): handle quotes preference in interface method signatures

Author: a-tarasyuk

src/services/codefixes/helpers.ts

@@ -121,19 +121,19 @@ namespace ts.codefix {
                 if (declarations.length === 1) {
                     Debug.assert(signatures.length === 1, "One declaration implies one signature");
                     const signature = signatures[0];
-                    outputMethod(signature, modifiers, name, ambient ? undefined : createStubbedMethodBody(preferences));
+                    outputMethod(preferences, signature, modifiers, name, ambient ? undefined : createStubbedMethodBody(preferences));
                     break;
                 }
 
                 for (const signature of signatures) {
                     // Need to ensure nodes are fresh each time so they can have different positions.
-                    outputMethod(signature, getSynthesizedDeepClones(modifiers, /*includeTrivia*/ false), getSynthesizedDeepClone(name, /*includeTrivia*/ false));
+                    outputMethod(preferences, signature, getSynthesizedDeepClones(modifiers, /*includeTrivia*/ false), getSynthesizedDeepClone(name, /*includeTrivia*/ false));
                 }
 
                 if (!ambient) {
                     if (declarations.length > signatures.length) {
                         const signature = checker.getSignatureFromDeclaration(declarations[declarations.length - 1] as SignatureDeclaration)!;
-                        outputMethod(signature, modifiers, name, createStubbedMethodBody(preferences));
+                        outputMethod(preferences, signature, modifiers, name, createStubbedMethodBody(preferences));
                     }
                     else {
                         Debug.assert(declarations.length === signatures.length, "Declarations and signatures should match count");
@@ -143,14 +143,15 @@ namespace ts.codefix {
                 break;
         }
 
-        function outputMethod(signature: Signature, modifiers: NodeArray<Modifier> | undefined, name: PropertyName, body?: Block): void {
-            const method = signatureToMethodDeclaration(context, signature, enclosingDeclaration, modifiers, name, optional, body, importAdder);
+        function outputMethod(preferences: UserPreferences, signature: Signature, modifiers: NodeArray<Modifier> | undefined, name: PropertyName, body?: Block): void {
+            const method = signatureToMethodDeclaration(context, preferences, signature, enclosingDeclaration, modifiers, name, optional, body, importAdder);
             if (method) addClassElement(method);
         }
     }
 
     function signatureToMethodDeclaration(
         context: TypeConstructionContext,
+        preferences: UserPreferences,
         signature: Signature,
         enclosingDeclaration: ClassLikeDeclaration,
         modifiers: NodeArray<Modifier> | undefined,
@@ -162,7 +163,8 @@ namespace ts.codefix {
         const program = context.program;
         const checker = program.getTypeChecker();
         const scriptTarget = getEmitScriptTarget(program.getCompilerOptions());
-        const signatureDeclaration = <MethodDeclaration>checker.signatureToSignatureDeclaration(signature, SyntaxKind.MethodDeclaration, enclosingDeclaration, NodeBuilderFlags.NoTruncation | NodeBuilderFlags.SuppressAnyReturnType, getNoopSymbolTrackerWithResolver(context));
+        const flags = NodeBuilderFlags.NoTruncation | NodeBuilderFlags.SuppressAnyReturnType | (preferences.quotePreference === "single" ? NodeBuilderFlags.UseSingleQuotesForStringLiteralType : 0);
+        const signatureDeclaration = <MethodDeclaration>checker.signatureToSignatureDeclaration(signature, SyntaxKind.MethodDeclaration, enclosingDeclaration, flags, getNoopSymbolTrackerWithResolver(context));
         if (!signatureDeclaration) {
             return undefined;
         }

tests/cases/fourslash/codeFixClassImplementInterface1_optionQuote.ts

@@ -1,20 +1,33 @@
 /// <reference path='fourslash.ts' />
 
 ////interface I {
-////    m(): void;
+////    a(): void;
+////    b(x: 'x', y: 'a' | 'b'): 'b';
+////
+////    c: 'c';
+////    d: { e: 'e'; };
 ////}
-////class C implements I {}
+////class Foo implements I {}
 
 verify.codeFix({
-    description: "Implement interface 'I'",
+    description: [ts.Diagnostics.Implement_interface_0.message, "I"],
     newFileContent:
 `interface I {
-    m(): void;
+    a(): void;
+    b(x: 'x', y: 'a' | 'b'): 'b';
+
+    c: 'c';
+    d: { e: 'e'; };
 }
-class C implements I {
-    m(): void {
+class Foo implements I {
+    a(): void {
+        throw new Error('Method not implemented.');
+    }
+    b(x: 'x', y: 'a' | 'b'): 'b' {
         throw new Error('Method not implemented.');
     }
+    c: 'c';
+    d: { e: 'e'; };
 }`,
     preferences: { quotePreference: "single" }
 });