fix(57022): Rename of string literal property name doesn't rename all occurences (#57201)

Author: Zzzen

src/services/findAllReferences.ts

@@ -757,7 +757,7 @@ interface PrefixAndSuffix {
     readonly suffixText?: string;
 }
 function getPrefixAndSuffixText(entry: Entry, originalNode: Node, checker: TypeChecker, quotePreference: QuotePreference): PrefixAndSuffix {
-    if (entry.kind !== EntryKind.Span && isIdentifier(originalNode)) {
+    if (entry.kind !== EntryKind.Span && (isIdentifier(originalNode) || isStringLiteralLike(originalNode))) {
         const { node, kind } = entry;
         const parent = node.parent;
         const name = originalNode.text;

tests/baselines/reference/renameStringPropertyNames2.baseline.jsonc

@@ -0,0 +1,10 @@
+// === findRenameLocations ===
+// === /tests/cases/fourslash/renameStringPropertyNames2.ts ===
+// type Props = {
+//   <|[|fooRENAME|]: boolean;|>
+// }
+// 
+// <|let { [|fooRENAME|]: foo/*END SUFFIX*/ }: Props = null as any;|>
+// foo;
+// 
+// let asd: Props = { <|"[|fooRENAME|]"/*RENAME*/: true|> }; // rename foo here

tests/cases/fourslash/renameStringPropertyNames2.ts

@@ -0,0 +1,12 @@
+/// <reference path="fourslash.ts" />
+
+////type Props = {
+////  foo: boolean;
+////}
+////
+////let { foo }: Props = null as any;
+////foo;
+////
+////let asd: Props = { "foo"/**/: true }; // rename foo here
+
+verify.baselineRename("", {});