improve tests and parser

Author: Kingwl

src/compiler/parser.ts

@@ -7588,7 +7588,7 @@ namespace ts {
                     return lookAhead(() => {
                         parseOptional(SyntaxKind.OpenBraceToken);
                         return isIdentifier();
-                    })
+                    });
                 }
 
                 function parseBracketNameInPropertyAndParamTag(): { name: EntityName, isBracketed: boolean } {

tests/cases/fourslash/jsDocSee1.ts

@@ -5,7 +5,7 @@
 //// }
 
 //// namespace NS {
-////     export interface [| /*def2*/Bar|] {
+////     export interface [|/*def2*/Bar|] {
 ////         baz: Foo
 ////     }
 //// }
@@ -16,8 +16,26 @@
 //// /** @see {NS./*use2*/[|Bar|]} ns.bar*/
 //// const b = ""
 
+//// /** @see /*use3*/[|Foo|] f1*/
+//// const c = ""
+
+//// /** @see NS./*use4*/[|Bar|] ns.bar*/
+//// const [|/*def3*/d|] = ""
+
+//// /** @see /*use5*/[|d|] dd*/
+//// const e = ""
+
 goTo.marker("use1");
 verify.goToDefinitionIs("def1");
 
 goTo.marker("use2");
 verify.goToDefinitionIs("def2");
+
+goTo.marker("use3");
+verify.goToDefinitionIs("def1");
+
+goTo.marker("use4");
+verify.goToDefinitionIs("def2");
+
+goTo.marker("use5");
+verify.goToDefinitionIs("def3");

tests/cases/fourslash/jsDocSee2.ts

@@ -0,0 +1,43 @@
+///<reference path="fourslash.ts" />
+
+//// /** @see {/*use1*/[|foooo|]} unknown reference*/
+//// const a = ""
+
+//// /** @see {/*use2*/[|@bar|]} invalid tag*/
+//// const b = ""
+
+//// /** @see /*use3*/[|foooo|] unknown reference without brace*/
+//// const c = ""
+
+//// /** @see /*use4*/[|@bar|] invalid tag without brace*/
+//// const [|/*def1*/d|] = ""
+
+//// /** @see {/*use5*/[|d@fff|]} partial reference */
+//// const e = ""
+
+//// /** @see /*use6*/[|@@@@@@|] total invalid tag*/
+//// const f = ""
+
+//// /** @see d@{/*use7*/[|fff|]} partial reference */
+//// const g = ""
+
+goTo.marker("use1");
+verify.goToDefinition([]);
+
+goTo.marker("use2");
+verify.goToDefinition([]);
+
+goTo.marker("use3");
+verify.goToDefinition([]);
+
+goTo.marker("use4");
+verify.goToDefinition([]);
+
+goTo.marker("use5");
+verify.goToDefinitionIs("def1");
+
+goTo.marker("use6");
+verify.goToDefinition([]);
+
+goTo.marker("use7");
+verify.goToDefinition([]);