MPT-19487 unskip chat messages e2e tests

[jentyk]

Closes MPT-19487

• Unskipped test_list_chat_messages end-to-end test in tests/e2e/helpdesk/chats/messages/test_async_messages.py
• Unskipped test_list_chat_messages end-to-end test in tests/e2e/helpdesk/chats/messages/test_sync_messages.py

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 0d48bc51-962c-4e47-8528-c726882122eb

📥 Commits

Reviewing files that changed from the base of the PR and between 9ef1497 and 314ba5b.

📒 Files selected for processing (2)

• tests/e2e/helpdesk/chats/messages/test_async_messages.py
• tests/e2e/helpdesk/chats/messages/test_sync_messages.py

💤 Files with no reviewable changes (2)

• tests/e2e/helpdesk/chats/messages/test_async_messages.py
• tests/e2e/helpdesk/chats/messages/test_sync_messages.py

---

📝 Walkthrough

Walkthrough

This pull request removes the @pytest.mark.skip decorators from test_list_chat_messages in two end-to-end test files for helpdesk chat messaging. Previously skipped pending completion of task MPT-19124, these tests are now enabled to run.

Changes

Cohort / File(s)	Summary
Test Skip Removal tests/e2e/helpdesk/chats/messages/test_async_messages.py, tests/e2e/helpdesk/chats/messages/test_sync_messages.py	Removed @pytest.mark.skip(reason="Unskip after MPT-19124 completed") decorator from test_list_chat_messages in both async and sync message test files, enabling these E2E tests to execute.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• MPT-19124 mark chat tests as skipped pending MPT-19124 completion #240: Directly reverts the skip decorator on test_list_chat_messages in the same message test files, indicating completion of the referenced task.
• MPT-19479 E2E: Helpdesk Chats core #249: Modifies pytest skip decorators across helpdesk E2E chat-related tests, showing a broader pattern of unskipping previously disabled tests in this module.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Jira Issue Key In Title	✅ Passed	The PR title contains exactly one Jira issue key (MPT-19487) in the required MPT-XXXX format.
Test Coverage Required	✅ Passed	PR only modifies test files in tests/ directory with no code file modifications, so the requirement to include tests when modifying code is not applicable.
Single Commit Required	✅ Passed	The PR contains exactly one commit (314ba5b: "test(helpdesk): unskip chat messages e2e tests") that removes @pytest.mark.skip decorators from helpdesk test files.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud