Skip to content

Feat: add Jest regression test for dhtmlx-gantt upgrades#220

Open
Rom1-B wants to merge 1 commit into
mainfrom
feat_jest
Open

Feat: add Jest regression test for dhtmlx-gantt upgrades#220
Rom1-B wants to merge 1 commit into
mainfrom
feat_jest

Conversation

@Rom1-B

@Rom1-B Rom1-B commented Jul 10, 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
  • Dependabot bumps of dhtmlx-gantt can silently break the Gantt view without showing up in the diff (v10.0.0 turned out to make window.gantt undefined at runtime).
    This adds a Jest test loading the actual built bundle (public/lib/libs.js) and asserting the gantt API surface used by gantt-helper.js is present, so the shared plugin CI catches this class of regression automatically.
    The plugin relies on GLPI core's own jest/jest-environment-jsdom install (via the CI's ../../node_modules/.bin/jest fallback), so no new devDependency was added.

Screenshots (if appropriate):

@Rom1-B Rom1-B requested a review from stonebuzz July 10, 2026 14:46
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.

1 participant