The GitHub Actions job "Tests (AMD)" on airflow.git/fix/fab-session-cookie-bytes-64921 has failed. Run started by GitHub user Sriniketh24 (triggered by eladkal).
Head commit for run: 41aac1614d218cab10cbf7234abe0d4703766fcb / Sriniketh24 <[email protected]> Fix test_session mock to account for SessionExemptMixin in MRO The tests patched `__mro__[1]` (SessionExemptMixin) but `AirflowSecureCookieSessionInterface.save_session()` calls `super()` which chains through SessionExemptMixin before reaching SecureCookieSessionInterface. This caused two failures: 1. SessionExemptMixin.save_session accesses `flask.request.path` — added a `patch("flask.request")` context manager. 2. `patch.object` replaces the class method with a MagicMock whose `side_effect` does NOT receive `self` — changed the lambda signature to `*args, **kwargs`. 3. Changed patch target from `__mro__[1]` to `__mro__[2]` to patch SecureCookieSessionInterface directly, letting SessionExemptMixin run its real check so the bytes-to-str wrapper is exercised. Report URL: https://github.com/apache/airflow/actions/runs/26383130170 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
