Skip to content

Fix: fill mandatory ticket content in escalation test - #504

Merged
MyvTsv merged 1 commit into
mainfrom
fix_phpunit
Sep 23, 2026
Merged

MyvTsv merged 1 commit into
mainfrom
fix_phpunit

Conversation

@Rom1-B

@Rom1-B Rom1-B commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes #N/A
  • testTaskGroupEscalation created its ticket with empty content, which GLPI core rejects as a missing mandatory field once the test reaches the "solved ticket" step.
  • This left an unhandled session error message and made the two data sets covering solve/reject-solution escalation fail.
  • The ticket now gets non-empty content so those data sets pass again; no plugin behavior changed.

Screenshots (if appropriate):

@MyvTsv
MyvTsv merged commit 9b9f293 into main Sep 23, 2026
7 checks passed
@MyvTsv
MyvTsv deleted the fix_phpunit branch September 23, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants