Raghav-Mah3shwari opened a new pull request, #1194: URL: https://github.com/apache/knox/pull/1194
What changes were proposed in this pull request? This PR adds integration tests under .github/workflows/tests that run against the Knox gateway in the workflow Docker Compose stack: test_health.py — test_health_metrics_returns_json checks that GET /gateway/health/v1/metrics?pretty=true returns 200, a JSON Content-Type, and a parseable JSON object body. test_knoxauth_preauth_and_paths.py — Covers KnoxLDAP preauth (401 without credentials / bad credentials; POST with valid credentials) and extauthz behavior when an additional path segment is not ignored (expects 404 vs treating it like the base extauthz URL). test_remoteauth_extauthz_additional_path.py — Covers RemoteAuth extauthz: success with valid Basic auth and X-Knox-Actor-ID, additional path ignored when configured, 401 for bad credentials, and current behavior (500) when the Authorization header is missing. 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: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
