MPT-19481 add skip marker with reason for parameter test

[jentyk]

Closes MPT-19481

• Added skip marker with reason to tests/e2e/helpdesk/parameters/test_sync_parameters.py module-level pytestmark, skipping all tests in the file until MPT-19967 is fixed while retaining the flaky designation

[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: 16312a6d-b91e-40c9-afbb-8b32f56346cb

📥 Commits

Reviewing files that changed from the base of the PR and between 3f49d0d and 3c5b69c.

📒 Files selected for processing (1)

• tests/e2e/helpdesk/parameters/test_sync_parameters.py

---

📝 Walkthrough

Walkthrough

Added a skip marker to the module-level pytestmark in a test file, causing all tests in the module to be skipped pending resolution of issue MPT-19967, while retaining the existing flaky designation.

Changes

Cohort / File(s)	Summary
Test Configuration tests/e2e/helpdesk/parameters/test_sync_parameters.py	Modified module-level pytest markers to add pytest.mark.skip(reason="Unskip after MPT-19967 fixed") alongside existing pytest.mark.flaky, causing all tests in this module to skip until the referenced issue is resolved.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• MPT-19481 update skip markers and parameter type in tests #292: Directly modifies the same test file's module-level pytest markers configuration.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Jira Issue Key In Title	✅ Passed	PR title contains Jira issue key MPT-19481 in correct format at the beginning.
Test Coverage Required	✅ Passed	The pull request modifies a test file in tests/e2e/helpdesk/parameters/ folder, satisfying the check requirement that test files must be included or modified.
Single Commit Required	✅ Passed	The PR contains exactly one commit (3c5b69c) that adds a skip marker with reason to a test file, matching requirements.

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