Re-enable mention intermittent failed cases

change/@ni-nimble-components-3798a360-e7c9-425c-b1b9-df4b1015ea4a.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Re-enable mention intermittently failed test cases",
+  "packageName": "@ni/nimble-components",
+  "email": "123377167+aagash-ni@users.noreply.github.com",
+  "dependentChangeType": "patch"
+}

packages/nimble-components/src/rich-text/editor/testing/rich-text-editor.pageobject.ts

@@ -248,10 +248,7 @@ export class RichTextEditorPageObject {
             .run();
         await waitForUpdatesAsync();
 
-        if (this.isMentionListboxOpened()) {
-            this.richTextEditorElement.tiptapEditor.commands.focus();
-            await waitForUpdatesAsync();
-        }
+        await this.focusEditorIfMentionListboxOpened();
     }
 
     public async setCursorPosition(position: number): Promise<void> {
@@ -439,6 +436,8 @@ export class RichTextEditorPageObject {
     }
 
     public async clickMentionListboxOption(index: number): Promise<void> {
+        await this.focusEditorIfMentionListboxOpened();
+
         const listOption = this.getAllListItemsInMentionBox()[index];
         listOption?.click();
         await waitForUpdatesAsync();
@@ -530,4 +529,11 @@ export class RichTextEditorPageObject {
         );
         return parserMentionConfig;
     }
+
+    private async focusEditorIfMentionListboxOpened(): Promise<void> {
+        if (this.isMentionListboxOpened()) {
+            this.richTextEditorElement.tiptapEditor.commands.focus();
+            await waitForUpdatesAsync();
+        }
+    }
 }

packages/nimble-components/src/rich-text/editor/tests/rich-text-editor-mention.spec.ts

@@ -1542,29 +1542,34 @@ describe('RichTextEditorMentionListbox', () => {
             expect(pageObject.isMentionListboxOpened()).toBeTrue();
         });
 
-        // Intermittent, see https://github.com/ni/nimble/issues/2219
-        xit('should show mention popup for multiple mention configuration elements', async () => {
+        it('should show mention popup for multiple mention configuration elements', async () => {
             await appendUserMentionConfiguration(element, [
                 { key: 'user:1', displayName: 'username1' },
                 { key: 'user:2', displayName: 'username2' }
             ]);
+
             await pageObject.setEditorTextContent('@');
-            expect(pageObject.isMentionListboxOpened()).toBeTrue();
+
             expect(pageObject.getMentionListboxItemsName()).toEqual([
                 'username1',
                 'username2'
             ]);
-            await pageObject.clickMentionListboxOption(0);
+            expect(pageObject.isMentionListboxOpened()).toBeTrue();
+
+            await pageObject.sliceEditorContent(0, 2);
+
             await appendTestMentionConfiguration(element, [
                 { key: 'test:1', displayName: 'testname1' },
                 { key: 'test:2', displayName: 'testname2' }
             ]);
+
             await pageObject.setEditorTextContent('!');
-            expect(pageObject.isMentionListboxOpened()).toBeTrue();
+
             expect(pageObject.getMentionListboxItemsName()).toEqual([
                 'testname1',
                 'testname2'
             ]);
+            expect(pageObject.isMentionListboxOpened()).toBeTrue();
         });
 
         it('mention listbox should be closed when cursor position is moved to start and configuration dynamically changes', async () => {

packages/nimble-components/src/rich-text/models/tests/markdown-serializer.spec.ts

@@ -337,12 +337,13 @@ Plain text 3`);
 <user:1> `);
         });
 
-        // Intermittent, see https://github.com/ni/nimble/issues/2219
-        xit('Mention node', async () => {
+        it('Mention node', async () => {
             await appendUserMentionConfiguration(element, [
                 { key: 'user:1', displayName: 'username1' }
             ]);
+
             await commitFirstMentionBoxOptionIntoEditor('@');
+
             expect(element.getMarkdown()).toEqual('<user:1> ');
         });