Skip to content

test: integration suite against real GLPI 10, 11 and 12 in CI - #6

Merged
giovanny07 merged 1 commit into
mainfrom
test/integration
Oct 13, 2026
Merged

giovanny07 merged 1 commit into
mainfrom
test/integration

Conversation

@giovanny07

Copy link
Copy Markdown
Owner

Runs both clients (sync and async) against official glpi/glpi containers on every PR, every push to main and once a week. The matrix uses floating tags 10.0, 11.0 and 12.0, so each run tests the latest patch of each branch.

Each of the four real bugs fixed in 1.5.0 has a test here that fails against the old behaviour:

  • HL API pagination past 100 items;
  • get_version() returning 0.0.0;
  • search() returning 404;
  • legacy add_sub_item failing with a permission error.

Verified locally on fresh containers (podman), same script as CI: 10.0.28: 6 passed (HL API tests skipped); 11.0.11: 20 passed; 12.0.0: 27 passed. GLPI setup takes about 1 minute per version.

The suite is skipped without GLPI_IT_URL, so plain pytest is unchanged (336 passed, 13 skipped).

🤖 Generated with Claude Code

- tests/integration: legacy and High-Level API clients (sync + async)
  against a live server: version, session helpers, pagination past the
  HL API default limit of 100, ticket lifecycle with Followup/Task/
  Solution sub-items, search engine and RSQL filter, purge -> 404,
  SoftwareLicense route; HL API runs on every version the server offers
  (default, 2.3, 3.0). Skipped unless GLPI_IT_URL is set.
- .github/scripts/glpi_it_setup.sh + glpi_oauth_client.php: prepare a
  fresh glpi/glpi container (APIs, open API client, OAuth client).
- .github/workflows/integration.yaml: matrix glpi/glpi:10.0, 11.0, 12.0
  on PRs, pushes to main and weekly, so new GLPI patch releases that
  break the client show up within a week.

Verified locally on fresh containers: 10.0.28 6 passed (HL API skipped),
11.0.11 20 passed, 12.0.0 27 passed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Giovanny Rodriguez <giovanny.rodriguez@imagunet.com>
@giovanny07
giovanny07 merged commit 45ac3a8 into main Oct 13, 2026
8 checks passed
@giovanny07
giovanny07 deleted the test/integration branch October 13, 2026 17:27
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