Merge pull request #7483 from dibarbet/fix_hover_tests

Fix hover tests to actually run and include correct content

Author: dibarbet

test/integrationTests/hover.integration.tests.ts renamed to test/integrationTests/hover.integration.test.ts

@@ -5,6 +5,7 @@
 
 import * as vscode from 'vscode';
 import * as path from 'path';
+import { EOL } from 'os';
 import testAssetWorkspace from './testAssets/testAssetWorkspace';
 import { activateCSharpExtension, closeAllEditorsAsync, openFileInWorkspaceAsync } from './integrationHelpers';
 import { describe, beforeAll, beforeEach, afterAll, test, expect, afterEach } from '@jest/globals';
@@ -35,8 +36,7 @@ describe(`[${testAssetWorkspace.description}] Hover Tests`, () => {
             )
         );
 
-        const expected =
-            '```csharp\r\nbool testissue.Compare(int gameObject, string tagName)\r\n```\r\n  \r\nA cref&nbsp;testissue  \r\n**strong text**  \r\n_italic text_  \r\n<u>underline text</u>  \r\n  \r\n•&nbsp;Item 1\\.  \r\n•&nbsp;Item 2\\.  \r\n  \r\n[link text](https://google.com)  \r\n  \r\nRemarks are cool too\\.  \r\n  \r\nReturns:  \r\n&nbsp;&nbsp;a string  \r\n  \r\nExceptions:  \r\n&nbsp;&nbsp;NullReferenceException  \r\n';
+        const expected = `\`\`\`csharp${EOL}bool testissue.Compare(int gameObject, string tagName)${EOL}\`\`\`${EOL}  ${EOL}A cref&nbsp;testissue  ${EOL}**strong text**  ${EOL}_italic text_  ${EOL}<u>underline text</u>  ${EOL}  ${EOL}•&nbsp;Item 1\\.  ${EOL}•&nbsp;Item 2\\.  ${EOL}  ${EOL}[link text](https://google.com)  ${EOL}  ${EOL}Remarks are cool too\\.  ${EOL}  ${EOL}Returns:  ${EOL}&nbsp;&nbsp;a string  ${EOL}  ${EOL}Exceptions:  ${EOL}&nbsp;&nbsp;NullReferenceException  ${EOL}`;
 
         expect(hovers.length).toEqual(1);
         expect((hovers[0].contents[0] as vscode.MarkdownString).value).toEqual(expected);

test/integrationTests/testAssets/slnWithCsproj/src/app/hover.cs

@@ -1,16 +1,39 @@
 using System;
 namespace Test
 {
-   class testissue
-   {
-       ///<summary>Checks if object is tagged with the tag.</summary>
-       /// <param name="gameObject">The game object.</param>
-       /// <param name="tagName">Name of the tag.</param>
-       /// <returns>Returns <c>true</c> if object is tagged with tag.</returns>
-       
-       public static bool Compare(int gameObject,string tagName)
-       {
-           return true;
-       }
-   }
+    class testissue
+    {
+        /// <summary>
+        /// A cref <see cref="testissue"/>
+        /// <br/>
+        /// <strong>strong text</strong>
+        /// <br/>
+        /// <em>italic text</em>
+        /// <br/>
+        /// <u>underline text</u>
+        /// <para>
+        /// <list type='bullet'>
+        /// <item>
+        /// <description>Item 1.</description>
+        /// </item>
+        /// <item>
+        /// <description>Item 2.</description>
+        /// </item>
+        /// </list>
+        /// <a href = "https://google.com" > link text</a>
+        /// </para>
+        /// </summary>
+        /// <exception cref='System.NullReferenceException'>
+        /// Oh no!
+        /// </exception>
+        /// <param name='i'>an int</param>
+        /// <returns>a string</returns>
+        /// <remarks>
+        /// Remarks are cool too.
+        /// </remarks>
+        public static bool Compare(int gameObject,string tagName)
+        {
+            return true;
+        }
+    }
 }