Fixed failing tests and lint. Added unit tests.

Author: Armando Aguirre

src/services/services.ts

@@ -1511,15 +1511,16 @@ namespace ts {
         function getReferences(fileName: string, position: number, options?: FindAllReferences.Options) {
             synchronizeHostData();
 
-            //Exclude default library when renaming as commonly user don't want to change that file.
+            // Exclude default library when renaming as commonly user don't want to change that file.
             let sourceFiles: SourceFile[] = [];
-            if (options.isForRename) {
-                for (let sourceFile of program.getSourceFiles()) {
+            if (options && options.isForRename) {
+                for (const sourceFile of program.getSourceFiles()) {
                     if (!program.isSourceFileDefaultLibrary(sourceFile)) {
                         sourceFiles.push(sourceFile);
                     }
                 }
-            } else {
+            }
+            else {
                 sourceFiles = program.getSourceFiles();
             }
 

tests/cases/fourslash/renameDefaultLibDontWork.ts

@@ -0,0 +1,11 @@
+/// <reference path='fourslash.ts' />
+
+// Tests that tokens found on the default library are not renamed.
+// "test" is a comment on the default library.
+
+// @Filename: file1.ts
+//// var [|test|] = "foo";
+//// console.log([|test|]);
+
+const ranges = test.ranges();
+verify.renameLocations(ranges[0], { findInComments: true, ranges });