MPT-19487 unskip chat message tests by removing skip markers

[jentyk]

Closes MPT-19487

• Removed @pytest.mark.skip decorators from chat message tests in both test_async_messages.py and test_sync_messages.py
• Re-enabled three previously skipped tests: test_create_chat_message, test_update_chat_message_visibility, and test_delete_chat_message
• Tests are now active for execution following completion of MPT-19124

[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: 03128860-8f7a-41ac-8a51-0b387831b22c

📥 Commits

Reviewing files that changed from the base of the PR and between 82dc28b and bf16db9.

📒 Files selected for processing (2)

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

---

📝 Walkthrough

Walkthrough

This PR removes @pytest.mark.skip decorators from three chat message tests across async and sync test files. The decorators included a reference to ticket MPT-19124. No changes to test logic, assertions, or function signatures were made.

Changes

Cohort / File(s)	Summary
Chat message test unskipping tests/e2e/helpdesk/chats/messages/test_async_messages.py, tests/e2e/helpdesk/chats/messages/test_sync_messages.py	Removed @pytest.mark.skip(...) decorators from test_create_chat_message, test_update_chat_message_visibility, and test_delete_chat_message in both files, enabling these three tests to run.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• MPT-19124 mark chat tests as skipped pending MPT-19124 completion #240: Directly inverse relation — this PR added the same skip decorators to these test functions that the current PR is removing.
• MPT-19486 refactor fixtures and unskip chat attachment tests #268: Modified the chat_messages_service and async_chat_messages_service fixtures that these re-enabled tests depend on.
• MPT-19488 merge chat message fixtures and unskip link tests #269: Updated message test fixtures in tests/e2e/helpdesk/chats/messages/conftest.py that support the tests being enabled in this PR.

🚥 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 at the beginning.
Test Coverage Required	✅ Passed	The PR modifies code files in mpt_api_client/ directory and includes changes in tests/ folder, satisfying the pass condition.
Single Commit Required	✅ Passed	The PR contains exactly one commit titled 'test(helpdesk): unskip chat message tests by removing skip markers' (commit hash bf16db9).

_{✏️ 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