Skip to content

Reorder revoke-token from API integration test - #37696

Merged
jotacarma90 merged 1 commit into
4.14.8from
enhancement/37677-api-integration-revoke-token-and-remove-wazuh-dbd
Jul 17, 2026
Merged

jotacarma90 merged 1 commit into
4.14.8from
enhancement/37677-api-integration-revoke-token-and-remove-wazuh-dbd

Conversation

@Antoniogm03

@Antoniogm03 Antoniogm03 commented Jul 15, 2026

Copy link
Copy Markdown
Member

Description

test_security_PUT_endpoints.tavern.yaml runs PUT /security/user/revoke as the 4th of 9 test blocks. Revoking invalidates the shared test_login_token, so every stage that reuses it afterwards (DELETE /security/user/authenticate, PUT /security/users/{user_id}, .../run_as, PUT /security/config, PUT /security/config Check) fails with 401 Invalid token once strict auth validation landed in 4.14.7 (#37034). The equivalent fix was applied on main/5.0.0 in #35040 (move revoke to the end) but was never backported to 4.14.x.

Proposed Changes

  • Move the PUT /security/user/revoke block to the end of api/test/integration/test_security_PUT_endpoints.tavern.yaml, after PUT /security/config (Check), matching the ordering already in place on main/5.0.0 since API ITs order changed to revoke token at last #35040.
  • No URLs, headers or assertions changed, only block order.

Review Checklist

  • Code changes reviewed
  • Relevant evidence provided
  • Tests cover the new functionality
  • Configuration changes documented
  • Developer documentation reflects the changes
  • Meets requirements and/or definition of done
  • No unresolved dependencies with other issues

Closes #37677
Closes #37676

@Antoniogm03
Antoniogm03 changed the base branch from main to 4.14.8 July 15, 2026 10:31
@Antoniogm03
Antoniogm03 marked this pull request as ready for review July 15, 2026 10:41
@Antoniogm03 Antoniogm03 self-assigned this Jul 15, 2026
@jotacarma90

Copy link
Copy Markdown
Member

Verification 🟢

The 4.x API integration suite is executed by Jenkins from the wazuh-qa-automation copy of the tests, so the revoke reorder was verified there via the equivalent change (wazuh/wazuh-qa-automation#8183) — build #4:

Suite Before After
test_security_PUT_endpoints.tavern.yaml 4 passed / 5 failed (401 cascade after revoke) 9 passed
test_manager_endpoints.tavern.yaml 35 passed / 1 failed ✅ 36 passed

🟢 SUCCESS (Test files 2/0, individual tests 45/0). This PR keeps the wazuh/wazuh copy of the test in sync with the same reorder.

@jotacarma90
jotacarma90 merged commit 849a33a into 4.14.8 Jul 17, 2026
9 of 10 checks passed
@jotacarma90
jotacarma90 deleted the enhancement/37677-api-integration-revoke-token-and-remove-wazuh-dbd branch July 17, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

API integration: revoke-token ordering fix (#35015) not backported to 4.14.x; test_security_PUT fails on 4.14.7-rc1

2 participants