Raghav-Mah3shwari opened a new pull request, #1196: URL: https://github.com/apache/knox/pull/1196
[KNOX-1234](https://issues.apache.org/jira/browse/KNOX-1234) - Add integration tests for health API and KnoxLDAP preauth/extauthz paths What changes were proposed in this pull request? This pull request extends [.github/workflows/tests](vscode-file://vscode-app/Applications/Cursor.app/Contents/Resources/app/out/vs/code/electron-sandbox/workbench/.github/workflows/tests) and the Docker Compose–based CI path so Knox is exercised with additional automated integration coverage: test_health.py: Asserts /gateway/health/v1/metrics returns JSON (with and without pretty), checks top-level metric keys align with the Java health test expectations, and verifies /gateway/health/v1/ping responds with text/plain in addition to the existing ping and HSTS checks. test_knoxauth_preauth_and_paths.py: Covers KnoxLDAP preauth (401 without or with bad credentials; 200 for POST/GET with valid credentials; x-knox-actor-username and group headers for admin) and verifies extauthz does not swallow unknown path segments (404 for a non-existent subpath). 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]
