update forslash test

breaking changes fourslash test

Temporarily retain the before fix code for easy comparison

Author: jeffy-g

src/services/completions.ts

@@ -855,7 +855,7 @@ namespace ts.Completions {
                 // When completion is requested without "@", we will have check to make sure that
                 // there are no comments prefix the request position. We will only allow "*" and space.
                 // e.g
-                //   /** |c| /*
+                //   /** |c| */
                 //
                 //   /**
                 //     |c|
@@ -868,6 +868,19 @@ namespace ts.Completions {
                 //   /**
                 //    *         |c|
                 //    */
+                /* https://coderwall.com/p/zbc2zw/the-comment-toggle-trick
+                if (sourceFile.text.charCodeAt(position - 1) === CharacterCodes.at) {
+                    // The current position is next to the '@' sign, when no tag name being provided yet.
+                    // Provide a full list of tag names
+                    return { kind: CompletionDataKind.JsDocTagName };
+                }
+                else {
+                    const lineStart = getLineStartPositionForPosition(position, sourceFile);
+                    if (!(sourceFile.text.substring(lineStart, position).match(/[^\*|\s|(/\*\*)]/))) {
+                        return { kind: CompletionDataKind.JsDocTag };
+                    }
+                }
+                /*/
                 const lineStart = getLineStartPositionForPosition(position, sourceFile);
                 // jsdoc tag will be listed if there is more than one whitespace after "*"
                 const match = /^\s*(?:[*\s]+(?=\s)|\/\*\*)?\s+(@)?$/.exec(
@@ -880,6 +893,7 @@ namespace ts.Completions {
                         kind: match[1] ? CompletionDataKind.JsDocTagName: CompletionDataKind.JsDocTag
                     };
                 }
+                //*/
             }
 
             // Completion should work inside certain JsDoc tags. For example:

tests/cases/fourslash/completionsJsdocTag.ts

@@ -11,32 +11,64 @@
 ////function a0(s: string) {}
 ////
 
+/////**@/*2*/ */
+////const some = 0;
+////
+
 verify.completions({ marker: "", includes: { name: "@property", text: "@property", kind: "keyword" } });
 
 
 //
 // test for src/services/completions.ts#getCompletionData.insideComment.hasDocComment (#37546)
 //
+goTo.marker("2");
+// line 1: [/**@|c|]
+// before the fix, jsdoc tag names was listed (but no longer appears
+// jsdoc tag names are still listed after the fix.
+// however this behavior does not by getCompletionData.insideComment.hasDocComment clause
+verify.completions({
+    triggerCharacter: "@",
+    includes: ["abstract", "access"]
+});
+// line 1: [/**@|c|] -> [/**|c|]
+// before the fix, jsdoc tags was listed but no longer appears
+edit.backspace(1);
+verify.completions({
+    exact: []
+});
+
+goTo.marker("1");
 // line 2: [ +|c|]
 verify.completions({
-    marker: "1",
+    // marker: "1",
     exact: []
 });
 // line 2: [ +|c|] -> [ +@|c|]
 // before the fix, jsdoc tag names was listed but no longer appears
 edit.insert("@");
 verify.completions({
-    marker: "1",
+    // marker: "1", // marker is invalid
+    triggerCharacter: "@",
+    exact: []
+});
+
+// line 2: [ +@|c|] -> [ * ### jsdoc @|c|]
+// before the fix, jsdoc tag names was listed but no longer appears
+edit.replaceLine(1, " * ### jsdoc @");
+verify.completions({
+    triggerCharacter: "@",
     exact: []
 });
+
 // line 2: [ +@|c|] -> [ *|c|]
 // before the fix, jsdoc tags was listed but no longer appears
 edit.replaceLine(1, " *");
 verify.completions({
     exact: []
 });
+
 // line 2: [ *|c|] -> [ *@|c|]
-// this behavior does not by getCompletionData.insideComment.hasDocComment section
+// this behavior does not by getCompletionData.insideComment.hasDocComment clause
 edit.insert("@");
 verify.completions({
     triggerCharacter: "@",