No dupe jsdoc for assignment inside an initializer (microsoft#24973)

sandersn · web-flow · commit 4db1c132b7ce · 2018-06-14T15:32:10.000-07:00
diff --git a/src/parser/utilities.ts b/src/parser/utilities.ts
@@ -2097,7 +2097,12 @@ namespace ts {
             if (isBinaryExpression(node) && node.operatorToken.kind === SyntaxKind.EqualsToken ||
                 isBinaryExpression(parent) && parent.operatorToken.kind === SyntaxKind.EqualsToken ||
                 node.kind === SyntaxKind.PropertyAccessExpression && node.parent && node.parent.kind === SyntaxKind.ExpressionStatement) {
-                getJSDocCommentsAndTagsWorker(parent);
+                if (isBinaryExpression(parent)) {
+                    getJSDocCommentsAndTagsWorker(parent.parent);
+                }
+                else {
+                    getJSDocCommentsAndTagsWorker(parent);
+                }
             }
 
             // Pull parameter comments from declaring function as well
diff --git a/tests/baselines/reference/noDuplicateJsdoc1.errors.txt b/tests/baselines/reference/noDuplicateJsdoc1.errors.txt
@@ -0,0 +1,9 @@
+tests/cases/conformance/jsdoc/a.js(2,11): error TS2304: Cannot find name 'b'.
+
+
+==== tests/cases/conformance/jsdoc/a.js (1 errors) ====
+    /** doc */
+    const a = b = () => 0;
+              ~
+!!! error TS2304: Cannot find name 'b'.
+    
diff --git a/tests/baselines/reference/noDuplicateJsdoc1.symbols b/tests/baselines/reference/noDuplicateJsdoc1.symbols
@@ -0,0 +1,5 @@
+=== tests/cases/conformance/jsdoc/a.js ===
+/** doc */
+const a = b = () => 0;
+>a : Symbol(a, Decl(a.js, 1, 5))
+
diff --git a/tests/baselines/reference/noDuplicateJsdoc1.types b/tests/baselines/reference/noDuplicateJsdoc1.types
@@ -0,0 +1,9 @@
+=== tests/cases/conformance/jsdoc/a.js ===
+/** doc */
+const a = b = () => 0;
+>a : () => number
+>b = () => 0 : () => number
+>b : any
+>() => 0 : () => number
+>0 : 0
+
diff --git a/tests/cases/conformance/jsdoc/noDuplicateJsdoc1.ts b/tests/cases/conformance/jsdoc/noDuplicateJsdoc1.ts
@@ -0,0 +1,6 @@
+// @allowJs: true
+// @checkJs: true
+// @noEmit: true
+// @Filename: a.js
+/** doc */
+const a = b = () => 0;
