Make sure we have a node for the error in cases of special property assignment

Author: mhegazy

src/compiler/checker.ts

@@ -20530,7 +20530,7 @@ namespace ts {
                             errorMessage = Diagnostics.Class_0_defines_instance_member_accessor_1_but_extended_class_2_defines_it_as_instance_member_function;
                         }
 
-                        error(derived.valueDeclaration.name, errorMessage, typeToString(baseType), symbolToString(base), typeToString(type));
+                        error(derived.valueDeclaration.name || derived.valueDeclaration, errorMessage, typeToString(baseType), symbolToString(base), typeToString(type));
                     }
                 }
             }

tests/baselines/reference/checkJsFiles_noErrorLocation.errors.txt

@@ -0,0 +1,25 @@
+tests/cases/compiler/a.js(14,5): error TS2424: Class 'A' defines instance member function 'foo', but extended class 'B' defines it as instance member property.
+
+
+==== tests/cases/compiler/a.js (1 errors) ====
+    // @ts-check
+    class A {
+      constructor() {
+    
+      }
+      foo() {
+        return 4;
+      }
+    }
+    
+    class B extends A {
+      constructor() {
+        super();
+        this.foo = () => 3;
+        ~~~~~~~~~~~~~~~~~~
+!!! error TS2424: Class 'A' defines instance member function 'foo', but extended class 'B' defines it as instance member property.
+      }
+    }
+    
+    const i = new B();
+    i.foo();

tests/cases/compiler/checkJsFiles_noErrorLocation.ts

@@ -0,0 +1,24 @@
+// @allowJs: true
+// @checkJs: true
+// @noEmit: true
+
+// @fileName: a.js
+// @ts-check
+class A {
+  constructor() {
+
+  }
+  foo() {
+    return 4;
+  }
+}
+
+class B extends A {
+  constructor() {
+    super();
+    this.foo = () => 3;
+  }
+}
+
+const i = new B();
+i.foo();