Skip to content

KNOX-3300: Add workflow HTTP integration tests for health, KnoxLDAP, KNOXTOKEN, and RemoteAuth#1202

Open
Raghav-Mah3shwari wants to merge 1 commit intoapache:masterfrom
Raghav-Mah3shwari:KNOX-3300
Open

KNOX-3300: Add workflow HTTP integration tests for health, KnoxLDAP, KNOXTOKEN, and RemoteAuth#1202
Raghav-Mah3shwari wants to merge 1 commit intoapache:masterfrom
Raghav-Mah3shwari:KNOX-3300

Conversation

@Raghav-Mah3shwari
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?
This pull request adds Python HTTP integration tests under .github/workflows/tests that run against a live Knox Gateway (and LDAP) in Docker Compose as part of CI.

Coverage includes:

  • Health topology: ping (GET/POST), metrics (with/without pretty), gateway-status, registry section shapes, HSTS / Cache-Control where applicable, unknown topology 404.
  • KnoxLDAP topology: KNOX-AUTH-SERVICE preauth/extauthz (401/404/200 cases, LDAP group headers), KNOXTOKEN JWKS and token v1/v2, JWT-shaped access_token.
  • RemoteAuth topology: pre (GET/POST, guest/admin, bad credentials, missing credentials error path), extauthz including ignore.additional.path behavior.
  • Global config: HSTS on a 404 path 1.
  • Supporting CI/build/compose updates (if included in the same PR) wire the tests image, optional test bind-mount compose override, and workflow steps to build Knox and run pytest with JUnit output

How was this patch tested?
Built the workflow Knox image and ran the compose stack, then executed the integration test container (same flow as .github/workflows/tests/README.md).

Command (from the repo root, paths as in the README):

docker compose -f ./.github/workflows/compose/docker-compose.yml up --build --exit-code-from tests tests

Confirmed pytest collects and passes the new cases together with existing workflow tests (health ping, LDAP auth service, configs, remote auth, etc.).

Integration Tests
Yes. New/updated tests live under .github/workflows/tests:

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