Fix issue microsoft#22923

Author: akhomchenko

src/compiler/checker.ts

@@ -16410,7 +16410,13 @@ namespace ts {
                 errorInfo = chainDiagnosticMessages(errorInfo, Diagnostics.Property_0_does_not_exist_on_type_1_Did_you_mean_2, declarationNameToString(propNode), typeToString(containingType), suggestion);
             }
             else {
-                errorInfo = chainDiagnosticMessages(errorInfo, Diagnostics.Property_0_does_not_exist_on_type_1, declarationNameToString(propNode), typeToString(containingType));
+                const promisedType = getPromisedTypeOfPromise(containingType);
+                if (promisedType && getPropertyOfType(promisedType, propNode.escapedText)) {
+                    errorInfo = chainDiagnosticMessages(errorInfo, Diagnostics.Property_0_does_not_exist_on_type_1_Did_you_forget_to_await_the_1, declarationNameToString(propNode), typeToString(containingType));
+                }
+                else {
+                    errorInfo = chainDiagnosticMessages(errorInfo, Diagnostics.Property_0_does_not_exist_on_type_1, declarationNameToString(propNode), typeToString(containingType));
+                }
             }
             diagnostics.add(createDiagnosticForNodeFromMessageChain(propNode, errorInfo));
         }

src/compiler/diagnosticMessages.json

@@ -2000,7 +2000,10 @@
         "category": "Error",
         "code": 2569
     },
-
+    "Property '{0}' does not exist on type '{1}'. Did you forget to await the '{1}'?": {
+        "category": "Error",
+        "code": 2570
+    },
     "JSX element attributes type '{0}' may not be a union type.": {
         "category": "Error",
         "code": 2600

tests/baselines/reference/nonexistentPropertyAvailableOnPromisedType.errors.txt

@@ -0,0 +1,10 @@
+tests/cases/compiler/nonexistentPropertyAvailableOnPromisedType.ts(2,7): error TS2570: Property 'toLowerCase' does not exist on type 'Promise<string>'. Did you forget to await the 'Promise<string>'?
+
+
+==== tests/cases/compiler/nonexistentPropertyAvailableOnPromisedType.ts (1 errors) ====
+    function f(x: Promise<string>) {
+        x.toLowerCase();
+          ~~~~~~~~~~~
+!!! error TS2570: Property 'toLowerCase' does not exist on type 'Promise<string>'. Did you forget to await the 'Promise<string>'?
+    }
+    

tests/baselines/reference/nonexistentPropertyAvailableOnPromisedType.js

@@ -0,0 +1,10 @@
+//// [nonexistentPropertyAvailableOnPromisedType.ts]
+function f(x: Promise<string>) {
+    x.toLowerCase();
+}
+
+
+//// [nonexistentPropertyAvailableOnPromisedType.js]
+function f(x) {
+    x.toLowerCase();
+}

tests/baselines/reference/nonexistentPropertyAvailableOnPromisedType.symbols

@@ -0,0 +1,10 @@
+=== tests/cases/compiler/nonexistentPropertyAvailableOnPromisedType.ts ===
+function f(x: Promise<string>) {
+>f : Symbol(f, Decl(nonexistentPropertyAvailableOnPromisedType.ts, 0, 0))
+>x : Symbol(x, Decl(nonexistentPropertyAvailableOnPromisedType.ts, 0, 11))
+>Promise : Symbol(Promise, Decl(lib.d.ts, --, --))
+
+    x.toLowerCase();
+>x : Symbol(x, Decl(nonexistentPropertyAvailableOnPromisedType.ts, 0, 11))
+}
+

tests/baselines/reference/nonexistentPropertyAvailableOnPromisedType.types

@@ -0,0 +1,13 @@
+=== tests/cases/compiler/nonexistentPropertyAvailableOnPromisedType.ts ===
+function f(x: Promise<string>) {
+>f : (x: Promise<string>) => void
+>x : Promise<string>
+>Promise : Promise<T>
+
+    x.toLowerCase();
+>x.toLowerCase() : any
+>x.toLowerCase : any
+>x : Promise<string>
+>toLowerCase : any
+}
+

tests/baselines/reference/nonexistentPropertyUnavailableOnPromisedType.errors.txt

@@ -0,0 +1,10 @@
+tests/cases/compiler/nonexistentPropertyUnavailableOnPromisedType.ts(2,7): error TS2339: Property 'toLowerCase' does not exist on type 'Promise<number>'.
+
+
+==== tests/cases/compiler/nonexistentPropertyUnavailableOnPromisedType.ts (1 errors) ====
+    function f(x: Promise<number>) {
+        x.toLowerCase();
+          ~~~~~~~~~~~
+!!! error TS2339: Property 'toLowerCase' does not exist on type 'Promise<number>'.
+    }
+    

tests/baselines/reference/nonexistentPropertyUnavailableOnPromisedType.js

@@ -0,0 +1,10 @@
+//// [nonexistentPropertyUnavailableOnPromisedType.ts]
+function f(x: Promise<number>) {
+    x.toLowerCase();
+}
+
+
+//// [nonexistentPropertyUnavailableOnPromisedType.js]
+function f(x) {
+    x.toLowerCase();
+}

tests/baselines/reference/nonexistentPropertyUnavailableOnPromisedType.symbols

@@ -0,0 +1,10 @@
+=== tests/cases/compiler/nonexistentPropertyUnavailableOnPromisedType.ts ===
+function f(x: Promise<number>) {
+>f : Symbol(f, Decl(nonexistentPropertyUnavailableOnPromisedType.ts, 0, 0))
+>x : Symbol(x, Decl(nonexistentPropertyUnavailableOnPromisedType.ts, 0, 11))
+>Promise : Symbol(Promise, Decl(lib.d.ts, --, --))
+
+    x.toLowerCase();
+>x : Symbol(x, Decl(nonexistentPropertyUnavailableOnPromisedType.ts, 0, 11))
+}
+

tests/baselines/reference/nonexistentPropertyUnavailableOnPromisedType.types

@@ -0,0 +1,13 @@
+=== tests/cases/compiler/nonexistentPropertyUnavailableOnPromisedType.ts ===
+function f(x: Promise<number>) {
+>f : (x: Promise<number>) => void
+>x : Promise<number>
+>Promise : Promise<T>
+
+    x.toLowerCase();
+>x.toLowerCase() : any
+>x.toLowerCase : any
+>x : Promise<number>
+>toLowerCase : any
+}
+

tests/cases/compiler/nonexistentPropertyAvailableOnPromisedType.ts

@@ -0,0 +1,3 @@
+function f(x: Promise<string>) {
+    x.toLowerCase();
+}

tests/cases/compiler/nonexistentPropertyUnavailableOnPromisedType.ts

@@ -0,0 +1,3 @@
+function f(x: Promise<number>) {
+    x.toLowerCase();
+}