Added node_modules path check on getTodoComments method.

Armando Aguirre · Armando Aguirre · commit 6f28f83f18de · 2017-07-17T17:35:46.000-07:00
diff --git a/src/services/services.ts b/src/services/services.ts
@@ -1834,7 +1834,7 @@ namespace ts {
             const fileContents = sourceFile.text;
             const result: TodoComment[] = [];
 
-            if (descriptors.length > 0) {
+            if (descriptors.length > 0 && !isNodeModulesFile(fileName)) {
                 const regExp = getTodoCommentsRegExp();
 
                 let matchArray: RegExpExecArray;
@@ -1958,6 +1958,12 @@ namespace ts {
                     (char >= CharacterCodes.A && char <= CharacterCodes.Z) ||
                     (char >= CharacterCodes._0 && char <= CharacterCodes._9);
             }
+
+            function isNodeModulesFile(path: string): boolean {
+                const node_modulesFolderName = "/node_modules/";
+
+                return path.indexOf(node_modulesFolderName) !== -1;
+            }
         }
 
         function getRenameInfo(fileName: string, position: number): RenameInfo {
diff --git a/tests/cases/fourslash/todoComments18.ts b/tests/cases/fourslash/todoComments18.ts
@@ -0,0 +1,6 @@
+// Tests node_modules name in file still gets todos.
+
+// @Filename: /node_modules_todotest0.ts
+//// // [|TODO|]
+
+verify.todoCommentsInCurrentFile(["TODO"]);
diff --git a/tests/cases/fourslash/todoComments19.ts b/tests/cases/fourslash/todoComments19.ts
@@ -0,0 +1,6 @@
+// Tests that todos are not found in node_modules folder.
+
+// @Filename: /node_modules/todotest0.ts
+//// // TODO
+
+verify.todoCommentsInCurrentFile(["TODO"]);
diff --git a/tests/cases/fourslash/todoComments20.ts b/tests/cases/fourslash/todoComments20.ts
@@ -0,0 +1,4 @@
+// @Filename: dir1/node_modules/todotest0.ts
+//// // TODO
+
+verify.todoCommentsInCurrentFile(["TODO"]);
