MPT-19479 E2E: Helpdesk Chats core

[jentyk]

Closes MPT-19479

• Added test package structure for Helpdesk channels, messages, parameter groups, and queues with __init__.py files
• Enabled core Helpdesk chat e2e tests by removing skip decorators from test_get_chat, test_list_chats, and test_update_chat in both sync and async test suites
• Added skip decorators to channel message "not found" tests pending 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: 14db56ae-93bd-423d-a35d-0fc44661e554

📥 Commits

Reviewing files that changed from the base of the PR and between 3d19416 and a732c0d.

📒 Files selected for processing (9)

• tests/e2e/helpdesk/channels/__init__.py
• tests/e2e/helpdesk/channels/messages/__init__.py
• tests/e2e/helpdesk/channels/messages/test_async_messages.py
• tests/e2e/helpdesk/channels/messages/test_sync_messages.py
• tests/e2e/helpdesk/chats/test_async_chats.py
• tests/e2e/helpdesk/chats/test_sync_chats.py
• tests/e2e/helpdesk/parameter_groups/__init__.py
• tests/e2e/helpdesk/parameter_groups/parameters/__init__.py
• tests/e2e/helpdesk/queues/__init__.py

💤 Files with no reviewable changes (2)

• tests/e2e/helpdesk/chats/test_async_chats.py
• tests/e2e/helpdesk/chats/test_sync_chats.py

---

📝 Walkthrough

Walkthrough

This PR establishes test package hierarchies for helpdesk channels, parameter groups, and queues by adding empty __init__.py files, while selectively re-enabling three previously skipped chat tests and marking specific channel message tests as skipped pending MPT-19124 completion.

Changes

Cohort / File(s)	Summary
Test Package Initialization tests/e2e/helpdesk/channels/__init__.py, tests/e2e/helpdesk/channels/messages/__init__.py, tests/e2e/helpdesk/parameter_groups/__init__.py, tests/e2e/helpdesk/parameter_groups/parameters/__init__.py, tests/e2e/helpdesk/queues/__init__.py	Added empty __init__.py files to establish Python package structure for new test module hierarchies.
Channel Message Test Skip Markers tests/e2e/helpdesk/channels/messages/test_async_messages.py, tests/e2e/helpdesk/channels/messages/test_sync_messages.py	Added @pytest.mark.skip decorator to test_list_channel_messages_not_found tests with reason "Unskip after MPT-19124 completed".
Chat Test Unskip Markers tests/e2e/helpdesk/chats/test_async_chats.py, tests/e2e/helpdesk/chats/test_sync_chats.py	Removed @pytest.mark.skip decorators from test_get_chat, test_list_chats, and test_update_chat, enabling these tests to execute.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• MPT-19124 mark chat tests as skipped pending MPT-19124 completion #240: Directly modifies the same chat test modules with skip decorator changes affecting test execution control flow.
• MPT-18373 Add helpdesk parameter definition services and test scaffolding #244: Adds helpdesk parameter services and e2e tests that utilize the new parameter_groups test package structure established in this PR.
• MPT-18930 Add helpdesk channel API services and test coverage #248: Introduces new helpdesk channels and messages e2e test modules that depend on the package initialization files added by 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 in the correct format (MPT-19479) at the beginning.
Test Coverage Required	✅ Passed	All modified files are within the tests/ folder. No code files outside tests/ directory have been modified.
Single Commit Required	✅ Passed	The pull request contains exactly one commit (a732c0d) with message 'test(helpdesk): unskip chat e2e tests', meeting the single commit requirement.

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