diff --git a/plugins/chat/test/javascripts/acceptance/user-status-on-mentions-test.js b/plugins/chat/test/javascripts/acceptance/user-status-on-mentions-test.js index bf7e925bb86..f28184c6991 100644 --- a/plugins/chat/test/javascripts/acceptance/user-status-on-mentions-test.js +++ b/plugins/chat/test/javascripts/acceptance/user-status-on-mentions-test.js @@ -5,7 +5,7 @@ import { publishToMessageBus, query, } from "discourse/tests/helpers/qunit-helpers"; -import { skip } from "qunit"; +import { skip, test } from "qunit"; import { click, triggerEvent, visit, waitFor } from "@ember/test-helpers"; import pretender, { response } from "discourse/tests/helpers/create-pretender"; @@ -56,6 +56,7 @@ acceptance("Chat | User status on mentions", function (needs) { id: channelId, chatable_id: 1, chatable_type: "Category", + title: "A category channel", meta: { message_bus_last_ids: {}, can_delete_self: true }, current_user_membership: { following: true }, chatable: { id: 1 }, @@ -117,11 +118,6 @@ acceptance("Chat | User status on mentions", function (needs) { statusSelector(mentionedUser2.username), mentionedUser2.status ); - await assertStatusTooltipIsRendered( - assert, - statusSelector(mentionedUser2.username), - mentionedUser2.status - ); }); skip("just posted messages | it updates status on mentions", async function (assert) { @@ -135,7 +131,6 @@ acceptance("Chat | User status on mentions", function (needs) { const selector = statusSelector(mentionedUser2.username); await waitFor(selector); assertStatusIsRendered(assert, selector, newStatus); - await assertStatusTooltipIsRendered(assert, selector, newStatus); }); skip("just posted messages | it deletes status on mentions", async function (assert) { @@ -165,11 +160,6 @@ acceptance("Chat | User status on mentions", function (needs) { statusSelector(mentionedUser3.username), mentionedUser3.status ); - await assertStatusTooltipIsRendered( - assert, - statusSelector(mentionedUser3.username), - mentionedUser3.status - ); }); skip("edited messages | it updates status on mentions", async function (assert) { @@ -186,7 +176,6 @@ acceptance("Chat | User status on mentions", function (needs) { const selector = statusSelector(mentionedUser3.username); await waitFor(selector); assertStatusIsRendered(assert, selector, newStatus); - await assertStatusTooltipIsRendered(assert, selector, newStatus); }); skip("edited messages | it deletes status on mentions", async function (assert) { @@ -206,7 +195,7 @@ acceptance("Chat | User status on mentions", function (needs) { assert.dom(selector).doesNotExist("status is deleted"); }); - skip("deleted messages | it shows status on mentions", async function (assert) { + test("deleted messages | it shows status on mentions", async function (assert) { await visit(`/chat/c/-/${channelId}`); await deleteMessage(".chat-message-content"); @@ -217,14 +206,9 @@ acceptance("Chat | User status on mentions", function (needs) { statusSelector(mentionedUser1.username), mentionedUser1.status ); - await assertStatusTooltipIsRendered( - assert, - statusSelector(mentionedUser1.username), - mentionedUser1.status - ); }); - skip("deleted messages | it updates status on mentions", async function (assert) { + test("deleted messages | it updates status on mentions", async function (assert) { await visit(`/chat/c/-/${channelId}`); await deleteMessage(".chat-message-content"); @@ -237,10 +221,9 @@ acceptance("Chat | User status on mentions", function (needs) { const selector = statusSelector(mentionedUser1.username); await waitFor(selector); assertStatusIsRendered(assert, selector, newStatus); - await assertStatusTooltipIsRendered(assert, selector, newStatus); }); - skip("deleted messages | it deletes status on mentions", async function (assert) { + test("deleted messages | it deletes status on mentions", async function (assert) { await visit(`/chat/c/-/${channelId}`); await deleteMessage(".chat-message-content"); @@ -255,7 +238,7 @@ acceptance("Chat | User status on mentions", function (needs) { assert.dom(selector).doesNotExist("status is deleted"); }); - skip("restored messages | it shows status on mentions", async function (assert) { + test("restored messages | it shows status on mentions", async function (assert) { await visit(`/chat/c/-/${channelId}`); await deleteMessage(".chat-message-content"); @@ -266,14 +249,9 @@ acceptance("Chat | User status on mentions", function (needs) { statusSelector(mentionedUser1.username), mentionedUser1.status ); - await assertStatusTooltipIsRendered( - assert, - statusSelector(mentionedUser1.username), - mentionedUser1.status - ); }); - skip("restored messages | it updates status on mentions", async function (assert) { + test("restored messages | it updates status on mentions", async function (assert) { await visit(`/chat/c/-/${channelId}`); await deleteMessage(".chat-message-content"); @@ -286,10 +264,9 @@ acceptance("Chat | User status on mentions", function (needs) { const selector = statusSelector(mentionedUser1.username); await waitFor(selector); assertStatusIsRendered(assert, selector, newStatus); - await assertStatusTooltipIsRendered(assert, selector, newStatus); }); - skip("restored messages | it deletes status on mentions", async function (assert) { + test("restored messages | it deletes status on mentions", async function (assert) { await visit(`/chat/c/-/${channelId}`); await deleteMessage(".chat-message-content"); @@ -315,27 +292,6 @@ acceptance("Chat | User status on mentions", function (needs) { ); } - async function assertStatusTooltipIsRendered(assert, selector, status) { - await triggerEvent(selector, "mouseenter"); - - assert.equal( - document - .querySelector(".user-status-tooltip-description") - .textContent.trim(), - status.description, - "status description is correct" - ); - - assert.ok( - document.querySelector( - `.user-status-message-tooltip img[alt='${status.emoji}']` - ), - "status emoji is correct" - ); - - await triggerEvent(selector, "mouseleave"); - } - async function deleteMessage(messageSelector) { await triggerEvent(query(messageSelector), "mouseenter"); await click(".more-buttons .select-kit-header-wrapper");