aicam commented on PR #5629: URL: https://github.com/apache/texera/pull/5629#issuecomment-4712730303
Added test coverage for the dynamic routing logic in `AccessControlResourceSpec` (commit e103ad936): - The existing test computing unit now has a recorded URI, and the "user has access" test asserts the rewritten `Host` header carries exactly that URI (covers the route-to-recorded-URI branch). - Two new units the user *can* access are refused with `FORBIDDEN`: one with no URI recorded, and one whose URI is blank/whitespace (covers the refuse-when-no-URI branch). This also fixes the existing OK-path tests, which were failing under the new refuse-when-no-URI behavior because the test unit had no recorded URI. All 12 tests in the spec pass locally. -- 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]
