Merge pull request microsoft#16488 from Microsoft/add-jsdoc-to-eof-token

sandersn · web-flow · commit 2495e6780b9f · 2017-06-13T11:14:52.000-07:00
Add jsdoc to EOF token to catch missed `@typedef`s
diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
@@ -685,7 +685,7 @@ namespace ts {
 
             sourceFile.statements = parseList(ParsingContext.SourceElements, parseStatement);
             Debug.assert(token() === SyntaxKind.EndOfFileToken);
-            sourceFile.endOfFileToken = <EndOfFileToken>parseTokenNode();
+            sourceFile.endOfFileToken = addJSDocComment(parseTokenNode() as EndOfFileToken);
 
             setExternalModuleIndicator(sourceFile);
 
diff --git a/tests/baselines/reference/jsdocTypeDefAtStartOfFile.symbols b/tests/baselines/reference/jsdocTypeDefAtStartOfFile.symbols
@@ -0,0 +1,12 @@
+=== tests/cases/conformance/jsdoc/dtsEquivalent.js ===
+/** @typedef {{[k: string]: any}} AnyEffect */
+No type information for this code./** @typedef {number} Third */
+No type information for this code.=== tests/cases/conformance/jsdoc/index.js ===
+/** @type {AnyEffect} */
+let b;
+>b : Symbol(b, Decl(index.js, 1, 3))
+
+/** @type {Third} */
+let c;
+>c : Symbol(c, Decl(index.js, 3, 3))
+
diff --git a/tests/baselines/reference/jsdocTypeDefAtStartOfFile.types b/tests/baselines/reference/jsdocTypeDefAtStartOfFile.types
@@ -0,0 +1,12 @@
+=== tests/cases/conformance/jsdoc/dtsEquivalent.js ===
+/** @typedef {{[k: string]: any}} AnyEffect */
+No type information for this code./** @typedef {number} Third */
+No type information for this code.=== tests/cases/conformance/jsdoc/index.js ===
+/** @type {AnyEffect} */
+let b;
+>b : { [k: string]: any; }
+
+/** @type {Third} */
+let c;
+>c : number
+
diff --git a/tests/cases/conformance/jsdoc/jsdocTypeDefAtStartOfFile.ts b/tests/cases/conformance/jsdoc/jsdocTypeDefAtStartOfFile.ts
@@ -0,0 +1,11 @@
+// @allowJs: true
+// @checkJs: true
+// @noEmit: true
+// @Filename: dtsEquivalent.js
+/** @typedef {{[k: string]: any}} AnyEffect */
+/** @typedef {number} Third */
+// @Filename: index.js
+/** @type {AnyEffect} */
+let b;
+/** @type {Third} */
+let c;
