MPT-19480 update parameter type and skip markers in tests

[jentyk]

Closes MPT-19480

• Change parameter fixture type from "String" to "SingleLineText" in E2E helpdesk parameter tests (tests/e2e/helpdesk/parameter_groups/parameters/conftest.py)
• Update module-level skip reason to "Unskip after MPT-19967 fixed" across E2E parameter-related test suites:
  • tests/e2e/helpdesk/parameters/test_sync_parameters.py
  • tests/e2e/helpdesk/parameters/test_async_parameters.py
  • tests/e2e/helpdesk/parameter_groups/parameters/test_sync_parameters.py
  • tests/e2e/helpdesk/parameter_groups/parameters/test_async_parameters.py
  • tests/e2e/helpdesk/parameter_groups/test_sync_parameter_groups.py
  • tests/e2e/helpdesk/parameter_groups/test_async_parameter_groups.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: 7528b1a7-1b34-4b80-b610-dba8157a9596

📥 Commits

Reviewing files that changed from the base of the PR and between 4b815f6 and 1b693c7.

📒 Files selected for processing (5)

• tests/e2e/helpdesk/parameter_groups/parameters/conftest.py
• tests/e2e/helpdesk/parameter_groups/parameters/test_async_parameters.py
• tests/e2e/helpdesk/parameter_groups/parameters/test_sync_parameters.py
• tests/e2e/helpdesk/parameter_groups/test_async_parameter_groups.py
• tests/e2e/helpdesk/parameter_groups/test_sync_parameter_groups.py

✅ Files skipped from review due to trivial changes (5)

• tests/e2e/helpdesk/parameter_groups/parameters/conftest.py
• tests/e2e/helpdesk/parameter_groups/test_sync_parameter_groups.py
• tests/e2e/helpdesk/parameter_groups/test_async_parameter_groups.py
• tests/e2e/helpdesk/parameter_groups/parameters/test_async_parameters.py
• tests/e2e/helpdesk/parameter_groups/parameters/test_sync_parameters.py

---

📝 Walkthrough

Walkthrough

Changed a helpdesk parameter fixture's "type" from "String" to "SingleLineText" and updated module-level pytest skip reason strings from "Skipped per MPT-18373 request" to "Unskip after MPT-19967 fixed" across four E2E test modules.

Changes

Cohort / File(s)	Summary
Fixture Data tests/e2e/helpdesk/parameter_groups/parameters/conftest.py	Updated parameter fixture parameter_data["type"] from "String" to "SingleLineText", affecting fixtures that consume it.
Test Skip Reasons tests/e2e/helpdesk/parameter_groups/parameters/test_async_parameters.py, tests/e2e/helpdesk/parameter_groups/parameters/test_sync_parameters.py, tests/e2e/helpdesk/parameter_groups/test_async_parameter_groups.py, tests/e2e/helpdesk/parameter_groups/test_sync_parameter_groups.py	Replaced module-level pytest.mark.skip reason "Skipped per MPT-18373 request" with "Unskip after MPT-19967 fixed" while retaining pytest.mark.flaky markers.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• MPT-18372 Add helpdesk parameter groups API services with unit and e2e test coverage #246: Edits the same E2E test fixtures/files and makes matching parameter fixture and skip-reason updates.
• MPT-19481 update skip markers and parameter type in tests #292: Makes the same change to the parameter fixture type and updates related pytest skip markers.
• MPT-19481 add skip marker with reason for parameter test #293: Modifies module-level pytest markers for parameter-related tests with the same skip-reason string.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Jira Issue Key In Title	✅ Passed	PR title contains exactly one Jira issue key (MPT-19480) in the required MPT-XXXX format at the beginning.
Test Coverage Required	✅ Passed	All 5 modified files are test files located in tests/ folder; no code files outside tests/ were modified.
Single Commit Required	✅ Passed	The PR contains exactly one commit (1b693c7: 'test(helpdesk): update parameter type and skip markers in tests'). The git log shows a single commit between main and the PR branch.

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