Add more tests and comments

(cherry picked from commit bf897c2)

Author: zhengbli

src/compiler/checker.ts

@@ -15375,6 +15375,8 @@ namespace ts {
                     case SpecialPropertyAssignmentKind.ThisProperty:
                     case SpecialPropertyAssignmentKind.ModuleExports:
                         return getSymbolOfNode(entityName.parent.parent);
+                    default:
+                        // Fall through if it is not a special property assignment
                 }
             }
 

tests/cases/fourslash/renameCrossJsTs01.ts

@@ -0,0 +1,12 @@
+/// <reference path='fourslash.ts'/>
+
+// @allowJs: true
+// @Filename: a.js
+////exports.[|area|] = function (r) { return r * r; }
+
+// @Filename: b.ts
+////import { [|area|] } from './a';
+////var t = /**/[|area|](10);
+
+goTo.marker();
+verify.renameLocations( /*findInStrings*/ false, /*findInComments*/ false);

tests/cases/fourslash/renameCrossJsTs02.ts

@@ -0,0 +1,12 @@
+/// <reference path='fourslash.ts'/>
+
+// @allowJs: true
+// @Filename: a.js
+////exports./**/[|area|] = function (r) { return r * r; }
+
+// @Filename: b.ts
+////import { [|area|] } from './a';
+////var t = [|area|](10);
+
+goTo.marker();
+verify.renameLocations( /*findInStrings*/ false, /*findInComments*/ false);

tests/cases/fourslash/renameJsExports.ts …sts/cases/fourslash/renameJsExports01.tstests/cases/fourslash/renameJsExports.ts renamed to tests/cases/fourslash/renameJsExports01.ts tests/cases/fourslash/renameJsExports.ts renamed to tests/cases/fourslash/renameJsExports01.ts

@@ -9,4 +9,4 @@
 ////var t = mod./**/[|area|](10);
 
 goTo.marker();
-verify.renameLocations(false, false);
+verify.renameLocations( /*findInStrings*/ false, /*findInComments*/ false);

tests/cases/fourslash/renameJsExports02.ts

@@ -0,0 +1,12 @@
+/// <reference path='fourslash.ts'/>
+
+// @allowJs: true
+// @Filename: a.js
+////exports./**/[|area|] = function (r) { return r * r; }
+
+// @Filename: b.js
+////var mod = require('./a');
+////var t = mod.[|area|](10);
+
+goTo.marker();
+verify.renameLocations( /*findInStrings*/ false, /*findInComments*/ false);

…s/fourslash/renameJsPrototypeProperty.ts …fourslash/renameJsPrototypeProperty01.tstests/cases/fourslash/renameJsPrototypeProperty.ts renamed to tests/cases/fourslash/renameJsPrototypeProperty01.ts tests/cases/fourslash/renameJsPrototypeProperty.ts renamed to tests/cases/fourslash/renameJsPrototypeProperty01.ts

@@ -9,4 +9,4 @@
 ////t./**/[|x|] = 11;
 
 goTo.marker();
-verify.renameLocations(false, false);
+verify.renameLocations( /*findInStrings*/ false, /*findInComments*/ false);

tests/cases/fourslash/renameJsPrototypeProperty02.ts

@@ -0,0 +1,12 @@
+/// <reference path='fourslash.ts'/>
+
+// @allowJs: true
+// @Filename: a.js
+////function bar() {
+////}
+////bar.prototype./**/[|x|] = 10;
+////var t = new bar();
+////t.[|x|] = 11;
+
+goTo.marker();
+verify.renameLocations( /*findInStrings*/ false, /*findInComments*/ false);

…/cases/fourslash/renameJsThisProperty.ts …ases/fourslash/renameJsThisProperty01.tstests/cases/fourslash/renameJsThisProperty.ts renamed to tests/cases/fourslash/renameJsThisProperty01.ts tests/cases/fourslash/renameJsThisProperty.ts renamed to tests/cases/fourslash/renameJsThisProperty01.ts

@@ -9,4 +9,4 @@
 ////t./**/[|x|] = 11;
 
 goTo.marker();
-verify.renameLocations(false, false);
+verify.renameLocations( /*findInStrings*/ false, /*findInComments*/ false);

tests/cases/fourslash/renameJsThisProperty02.ts

@@ -0,0 +1,12 @@
+/// <reference path='fourslash.ts'/>
+
+// @allowJs: true
+// @Filename: a.js
+////function bar() {
+////    this./**/[|x|] = 10;
+////}
+////var t = new bar();
+////t.[|x|] = 11;
+
+goTo.marker();
+verify.renameLocations( /*findInStrings*/ false, /*findInComments*/ false);